attachment-0002

Hi,                                           Tuesday (29 November 2011)<br><br>High North has reviewed the System Object and System Control Service and<br>has comments.<br><br>These comments augment the Apple comments and try to avoid overlap.<br>
<br>Cheers,<br>- Ira<br><br>------------------------------------------------------------------------<br><br>     [Comments on &quot;wd-mfdsystemcontrolservicemodel10-20111102.pdf&quot;]<br><br>Global - fix title of MFD Model [PWG5108.1]<br>
- several different unrelated titles are used for [PWG5108.1],<br>  for example on lines 260-261.<br><br>Global - case of &quot;subunit&quot; versus &quot;Subunit&quot;<br>- inconsistent capitalization of singular and plural &quot;subunit&quot;<br>
  throughout this document<br>- my preference would be titlecase throughout (per MFD Model)<br><br>Global - terminology - clarity<br>- change &quot;Client&quot; to &quot;System Client&quot;<br>  (too ambiguous otherwise, with respective to Job service clients)<br>
<br>Line 14 - typo<br>- change &quot;[PWG51008.1]&quot; to &quot;[PWG5108.1]&quot;<br><br>Line 16 - typo<br>- change &quot;by n MFD.&quot; to &quot;by an MFD.&quot;<br><br>Line 17 - clarify<br>- change &quot;The model extension&quot;<br>
  to &quot;The [PWG5108.1] model extension&quot;<br><br>Line 22 - clarify<br>- change &quot;Alert Table&quot; to &quot;prtAlertTable&quot;<br><br>Line 22 - typo<br>- change &quot;visible to any&quot; to &quot;visible via any&quot;<br>
<br>Line 23 - clarify<br>- change &quot;Imaging Device wide usage counters, all the conditions, and<br>  every Subunits.&quot;<br>  to &quot;Imaging Device total usage counters, all conditions, and all<br>  Subunits.&quot;<br>
<br>Line 25-26 - clarify<br>- change &quot;to start services that have been shut down.&quot;<br>  to &quot;to start (create) services and to restart services that have been<br>  previously shut down.&quot;<br><br>Line 181 - scope error<br>
- change &quot;System Object&#39;s capabilities, configuration,&quot;<br>  to &quot;System Object&#39;s configuration,&quot;<br><br>Line 182 - missing word<br>- change &quot;to modify System Object&#39;s data.&quot;<br>  to &quot;to modify the System Object&#39;s data.&quot;<br>
<br>Line 188 - case error<br>- change &quot;Network Connected System Client&quot;<br>  to &quot;network-connected System Client&quot;<br><br>Line 189 - typo<br>- change &quot;services that protects&quot;<br>  to &quot;services that protect&quot;<br>
<br>Line 190 - clarify<br>- change &quot;access of&quot; to &quot;access to the&quot;<br><br>Line 198 - missing term<br>- define &quot;System Client&quot; (used in section 2, 4.1, 8.1, etc.)<br>  (simply using &quot;Client&quot; is far too ambiguous in this spec)<br>
<br>Line 200 - clarify<br>- change &quot;this Specification&quot;<br>  to &quot;System Object and System Control Service&quot;<br><br>Line 203 - clarify<br>- change &quot;for the Imaging Device System.&quot;<br>  to &quot;the SystemControlService of the Imaging Device.&quot;<br>
<br>Line 208 - clarify<br>- change &quot;this Specification&quot;<br>  to &quot;System Object and System Control Service&quot;<br><br>Line 211 - broken reference<br>- change &quot;The MFD Working Group charter [CHAR] defines the following as<br>
  out-of-scope:&quot;<br>  to &quot;The following are out-of-scope:&quot;<br>  (MFD charter is out-of-date and should NOT be referenced in a PWG CS)<br>  (it&#39;s also missing in current Informative References section)<br>
<br>Line 217 - clarify<br>- change &quot;top level objects&quot;<br>  to &quot;top level elements&quot;<br>  (SystemStatus is NOT an object - it&#39;s an element group)<br><br>Line 217-220 - clarify<br>- rewrite these sentences to stop confusing object w/ element group<br>
<br>Line 236 - clarify<br>- change &quot;System Model Overview&quot;<br>  to &quot;Imaging Device Model&quot;<br>  (title disagrees with first sentence)<br><br>Line 240 - clarify<br>- change &quot;SystemControlService Clients&quot;<br>
  to &quot;System Clients&quot;<br>  (the formal term for conformance)<br><br>Line 244 - clarify<br>- change &quot;there is only a single instance.&quot;<br>  to &quot;there can only be a single instance.&quot;<br><br>Line 245 - clarify<br>
- change &quot;no jobs coming in or output produced and no subordinate&quot;<br>  to &quot;no Jobs coming in or output produced and therefore no subordinate&quot;<br><br>Line 246 - clarify<br>- change &quot;entire system&quot; to &quot;entire Imaging Device&quot;<br>
<br>Line 258 - clarify<br>- change &quot;all the subunits that comprise&quot;<br>  to &quot;all the subunits configured on&quot;<br>  (the subunits, by themselves, do NOT comprise the Imaging Device)<br><br>Line 267 - typo<br>
- change &quot;provide descriptive&quot; to &quot;provides descriptive&quot;<br><br>Line 270 - typo<br>- change &quot;has extension&quot; to &quot;has an extension&quot;<br><br>Line 281-283 - clarify<br>- add complete table of rules for &quot;optimistic&quot; Service state rollup<br>
  (explain that non-Job services like Resource don&#39;t participate)<br>  (overlaps w/ Apple comment - but this has to be interoperable)<br><br>Line 288 - typo<br>- change &quot;2lists&quot; to &quot;2 lists&quot;<br><br>
Line 293 - ambiguous element name<br>- change &quot;ServiceInstance&quot; to &quot;ServiceSummary&quot;<br>  (consistent with &quot;ResourceSummary&quot;)<br>  (the actual service instances are in the Services container)<br>
  (also line 416 now uses &quot;ServiceSummary&quot;)<br><br>Line 296 - clarify<br>- change &quot;SystemControlService Theory of Operation&quot;<br>  to &quot;SystemControlService Model&quot;<br><br>Line 297 - redundancy<br>
- change &quot;Imaging Device System&quot; to &quot;Imaging Device&quot;<br><br>Line 298 - typo<br>- change &quot;SystemControlService model&quot; to &quot;SystemControlService models&quot;<br><br>Line 301 - clarify<br>- change &quot;comprise&quot; to &quot;are configured on&quot;<br>
  (the subunits, by themselves, do NOT comprise the Imaging Device)<br><br>Line 304 - typo<br>- change &quot;has extension&quot; to &quot;has an extension&quot;<br><br>Line 309 - model error in Figure 6<br>- change SystemControlServiceDescription to REQUIRED<br>
  (otherwise there is no language or charset for Status values)<br><br>Line 312 - typo<br>- change &quot;provide Descriptive&quot; to &quot;provides descriptive&quot;<br><br>Line 315 - typo<br>- change &quot;has extension&quot; to &quot;has an extension&quot;<br>
<br>Line 327 - typo<br>- change &quot;provide state&quot; to &quot;provides state&quot;<br><br>Line 338 - clarify<br>- change &quot;SystemControlService Interfaces&quot;<br>  to &quot;SystemControlService Operations&quot;<br>
<br>Line 339 - clarify<br>- change &quot;provides a set of service interfaces&quot;<br>  to &quot;provides a service interface&quot;<br>  (one interface - multiple operations - in formal model terminology)<br><br>Line 376 - clarify<br>
- change &quot;can no longer be instantiated.&quot;<br>  to &quot;can no longer be restarted.&quot;<br>  (delete removes the instance itself)<br><br>Line 382 - typo<br>- change &quot;State element.&quot; to &quot;State elements.&quot;<br>
<br>Line 387 - typo<br>- change &quot;State element.&quot; to &quot;State elements.&quot;<br><br>Line 394 - punctuation<br>- add missing period<br><br>Line 400 - punctuation<br>- delete extra period after &quot;InputChannelInterface&quot;<br>
<br>Line 401 - punctuation<br>- delete extra period after &quot;Marker)&quot;<br><br>Line 417 - typo<br>- change &quot;the service&#39;s&quot; to &quot;for the service&#39;s&quot;<br><br>Line 423 - clarify<br>- change &quot;transition the&quot; to &quot;transition each&quot;<br>
<br>Line 424 - clarify<br>- add &quot;This operation does not affect the SystemControlService.&quot;<br>  (appears in ResumeAllServices)<br><br>Line 430 - clarify<br>- change &quot;transition the&quot; to &quot;transition each&quot;<br>
<br>Line 467 - clarify<br>- change &quot;transitions every job based Services&quot;<br>  to &quot;transitions every job based Service&quot;<br><br>Line 468 - clarify<br>- move &quot;This operation does not affect the SystemControlService.&quot;<br>
  to end-of-paragraph (it&#39;s an odd infix at present)<br><br>Line 533 - clarify<br>- change &quot;Client Conformance Requirements&quot;<br>  to &quot;System Client Conformance Requirements&quot;<br><br>Line 534-576 - clarify<br>
- reorder lists - MUST, then SHOULD, then MAY requirements<br><br>Line 539 - clarify (w/ list reorder above)<br>- change &quot;When sending a request, a conforming client NEED NOT supply&quot;<br>  to &quot;A conforming client MAY supply&quot;<br>
<br>Line 540 - bogus conformance (w/ list reorder above)<br>- change &quot;OPTIONALLY supplied&quot; to &quot;OPTIONAL to supply&quot;<br><br>Line 544 - clarify (w/ list reorder above)<br>- change &quot;client implementation&quot; to &quot;conforming client implementation&quot;<br>
- change &quot;inter-operate&quot; to &quot;interoperate&quot; (one word)<br><br>Line 549, 552, 560, 569 - clarify<br>- change &quot;conforming implementations&quot; to<br>  &quot;conforming Imaging Device and SystemControlService implementations&quot;<br>
<br>------------------------------------------------------------------------<br><br clear="all">Ira McDonald (Musician / Software Architect)<br>Chair - Linux Foundation Open Printing WG<br>Secretary - IEEE-ISTO Printer Working Group<br>
Co-Chair - IEEE-ISTO PWG IPP WG<br>Co-Chair - TCG Trusted Mobility Solutions WG<br>Chair - TCG Embedded Systems Hardcopy SG<br>IETF Designated Expert - IPP &amp; Printer MIB<br>Blue Roof Music/High North Inc<br><a style="color:rgb(51, 51, 255)" href="http://sites.google.com/site/blueroofmusic" target="_blank">http://sites.google.com/site/blueroofmusic</a><br>
<a style="color:rgb(102, 0, 204)" href="http://sites.google.com/site/highnorthinc" target="_blank">http://sites.google.com/site/highnorthinc</a><br>mailto:<a href="mailto:blueroofmusic@gmail.com" target="_blank">blueroofmusic@gmail.com</a><br>
Winter  579 Park Place  Saline, MI  48176  734-944-0094<br>Summer  PO Box 221  Grand Marais, MI 49839  906-494-2434<div style="display:inline"></div><div style="display:inline"></div><div style="display:inline"></div><div>
</div><div></div><div></div><div></div><br>
<div style="visibility: hidden; left: -5000px;" id="avg_ls_inline_popup"></div>
<br />-- 
<br />This message has been scanned for viruses and
<br />dangerous content by
<a href="http://www.mailscanner.info/"><b>MailScanner</b></a>, and is
<br />believed to be clean.