On Tue, 2011-04-26 at 16:37 +0200, Tigran Mkrtchyan wrote:
> On 04/26/2011 04:29 PM, Trond Myklebust wrote:
> > On Tue, 2011-04-26 at 16:03 +0200, Tigran Mkrtchyan wrote:
> >> I run 2.6.39-0.rc4.git2.0 build for fedora-16 build by redhat.
> >> Probably Steve and Bruce can comment on the patch set.
> >>
> >> My observation is that state id provided by client on read to
> >> DS dit not contain correct stateid:
> >>
> >> struct stateid4 {
> >> uint32_t seqid;
> >> opaque other[12];
> >> };
> >>
> >> The seqid equals to zero when client send it to DS which is not the
> >> value returned on OPEN.
> > Why do you think this is a bug? Section 8.2.2 of RFC5661 not only allows
> > this behaviour, it actually suggests it is appropriate for READ and
> > WRITE since it means that the server isn't forced to constantly check
> > the seqid value.
> Well, It's not a bug, but it's not the behavior we was testing at
> Connectathon.
> My expectation was that during interoperability testing we catch such
> changes.
My expectation is that everyone is coding to the spec, and not to
specific client or server behaviours. The main goal of interoperability
testing should be to discover deviations from the spec and/or incomplete
implementations.
Cheers
Trond
--
Trond Myklebust
Linux NFS client maintainer
NetApp
Trond.Myklebust@xxxxxxxxxx
www.netapp.com
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
