Re: Questions about pynfs:testLargeData-WRT5

["J. Bruce Fields" <bfields@redhat.com>]

On Tue, Feb 13, 2018 at 11:15:40AM -0500, J. Bruce Fields wrote:
> WRT5 is wrong in any case, I'll fix it.

How about this?

--b.

commit 30f1fff66da3
Author: J. Bruce Fields <bfields@redhat.com>
Date:   Tue Feb 13 16:30:27 2018 -0500

    4.0 server tests: replace WRT5 by better tests
    
    This test is sending a write that (at least in the knfsd case) is larger
    than the server's maximum write size, and expecting it to succeed.
    That's weird.
    
    Remove WRT5 and replace it by two tests.  Both first query the maximum
    write size.  One does a large (but not too large) write and checks that
    the data was written correctly.  The other does a too-large write and
    expects nothing.  (A wide variety of server behavior would be in spec;
    we do this just in case a buggy server might crash.)
    
    Signed-off-by: J. Bruce Fields <bfields@redhat.com>

diff --git a/nfs4.0/servertests/st_write.py b/nfs4.0/servertests/st_write.py
index 710452efa11d..004562d2f52b 100644
--- a/nfs4.0/servertests/st_write.py
+++ b/nfs4.0/servertests/st_write.py
@@ -120,17 +120,22 @@ def testNoData(t, env):
     if compareTimes(time_prior,time_after) != 0:
         t.fail("WRITE with no data affected time_modify")
 
+#WRT5 permanently retired
+
 def testLargeData(t, env):
-    """WRITE with a large amount of data
+    """WRITE with the maximum size, READ it back and compare
 
     FLAGS: write read all
     DEPEND: MKFILE
-    CODE: WRT5
+    CODE: WRT5a
     """
     c = env.c1
     c.init_connection()
+    maxread, maxwrite = _get_iosize(t, c, c.homedir)
+    maxio = min(maxread, maxwrite)
     fh, stateid = c.create_confirm(t.code)
-    data = "abcdefghijklmnopq" * 0x10000
+    pattern="abcdefghijklmnop"
+    data = pattern * (maxio / len(pattern)) + "q" * (maxio % len(pattern))
     # Write the data
     pos = 0
     while pos < len(data):
@@ -150,6 +155,27 @@ def testLargeData(t, env):
     if data != newdata:
         t.fail("READ did not correspond to WRITE with large dataset")
 
+def testTooLargeData(t, env):
+    """WRITE with more than the maximum size
+
+    FLAGS: write read all
+    DEPEND: MKFILE
+    CODE: WRT5b
+    """
+    c = env.c1
+    c.init_connection()
+    maxread, maxwrite = _get_iosize(t, c, c.homedir)
+    fh, stateid = c.create_confirm(t.code)
+    data = "a" * (maxwrite + 1000000)
+    try:
+        # We don't care much what the server does, this is just a check
+        # to make sure it doesn't crash.
+        res = c.write_file(fh, data, 0, stateid)
+    except IOError:
+        # Linux knfsd closes the socket when the write is too large.
+        # That's OK.
+        pass
+
 def testDir(t, env):
     """WRITE to a dir should return NFS4ERR_ISDIR