09-063r1 comment (retry)

[Penokie, George]

Formatted message: <a href="http://www.t10.org/cgi-bin/ac.pl?t=r&f=r0904011_f.htm">HTML-formatted message</a>

Craig,
There was a transcribe error on my part. Item 1 in section 5.6.5.3.1 should
have been stated as
1)  initialize and start the SNTT timer and the SNLT timer;
The sentence in section 5.6.5.4.3 " Receipt of the ALIGN Received (1) message
indicates that the connected phy has been able to achieve dword
synchronization with the previously negotiated settings." does not belong in
a transition section and will be deleted from that section. It will either
move to another section or be removed pending a discussion at the next
working group.
Bye for now,
George Penokie
LSI Corporation
3033 41st St. NW
Suite 100
Rochester, MN 55901
507-328-9017
george.penokie at lsi.com
________________________________
From: Craig Stoops [mailto:craig at expertio.com]
Sent: Tuesday, March 31, 2009 7:31 PM
To: t10 at t10.org
Cc: Penokie, George
Subject: 09-063r1 comment
Hi George,
In your posting of 09-063r1 today,  section 5.6.5.3 has the timer changed
|from SNLT to SNTT from the original 2008 proposal.
Questions:
1) Is this in fact correct that the timer is changed
2) If so, there is still a reference to SNLT in 5.6.5.3.3 (a), I assume this
should also be SNTT.
    If SNLT in this section is correct, then SNLT is missing initialization
(starting) and any consequent actions on SNLT expiring in the PS states.
Secondly, regarding section 5.6.5.4.3 "footnote" of:
"Receipt of the ALIGN Received (1) message indicates that the connected phy
has been able to achieve dword synchronization with the previously negotiated
settings."
Questions:
1) Does this only apply to the 5.6.5.4.3 transition and not to the 5.6.5.3.3
transition to ALIGN1 state? Ie ALIGN0 state can transition to ALIGN
regardless of DWS sync?
   If so, I would like to suggest that maybe just 5.6.5.4.3 "a" should just
read:
    "a) If the DWS is synced, and this state receives an ALIGN Received (1)
message before the SNTT timer expires..."
    because other places in the phy spec the ALIGN Received message does not
imply the DWS is synced. I think it would be clearer and more consistent.
Please advise
Thanks,
Craig Stoops
ExpertIO, Inc. - "Your Storage Protocol Verification Experts"
www.expertio.com<<a href="http://www.expertio.com&gt">http://www.expertio.com&gt;