Sorry for the delay. Returning unrequested attributes sounds like unlikely behavior, but may as well check for it, sure.... Applying. --b. On Thu, Aug 14, 2014 at 11:02:26AM +0530, Srikrishan Malik wrote: > Signed-off-by: Srikrishan Malik <srimalik@xxxxxxxxxx> > --- > nfs4.0/servertests/st_getattr.py | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/nfs4.0/servertests/st_getattr.py b/nfs4.0/servertests/st_getattr.py > index a55b923..6a4acdd 100644 > --- a/nfs4.0/servertests/st_getattr.py > +++ b/nfs4.0/servertests/st_getattr.py > @@ -43,10 +43,14 @@ def _try_empty(t, c, path): > def _try_supported(t, env, path): > c = env.c1 > mandatory = sum([attr.mask for attr in env.attr_info if attr.mandatory]) > + all = sum([attr.mask for attr in env.attr_info]) > supported = c.supportedAttrs(path) > if mandatory & supported != mandatory: > t.fail("GETATTR(FATTR4_SUPPORTED_ATTRS) did not return " > "all mandatory attributes") > + if supported > all: > + t.fail("GETATTR(FATTR4_SUPPORTED_ATTRS) returned more than expected " > + "supported attributes for the protocol") > > def _try_long(env, path): > c = env.c1 > -- > 1.7.1 > -- 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