fix XFSQA 145 / test_hole
authorDonald Douwsma <donaldd@sgi.com>
Wed, 18 Jun 2008 16:09:09 +0000 (16:09 +0000)
committerDonald Douwsma <donaldd@sgi.com>
Wed, 18 Jun 2008 16:09:09 +0000 (16:09 +0000)
There are two errors I see all the time in 145:

 - dm_probe_hole returns EINVAL for offsets close to the file size
 - dm_probe_hole wants EAGAIN for a probe at offset 1, length 0

The first error is a consequence of how the hole puching / probing
works.  It always rounds the requested offset up to the next block
size and then checks if that rounded offset still fits into the file
size.  Just do the same rounding in the testcase to make sure we don't
probe invalid offsets.

The second error is very odd to me, as we never return AGAIN in the
whole dm_probe_hole path.  I've just commented it out.

I've also re-enabled the E2BIG to past-EOF test that was uncommented
before because it works perfectly fine now.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Merge of master-melb:xfs-cmds:31330a by kenmcd.

  fix XFSQA 145 / test_hole

145.out
dmapi/src/suite2/src/test_hole.c

diff --git a/145.out b/145.out
index ec64ae541bed2b3a8117e872a7fb2a2e7163ad4d..cd7d03efd6bbd09cf7c85a75e336ee1a38293b80 100644 (file)
--- a/145.out
+++ b/145.out
@@ -16,13 +16,13 @@ Hole test beginning...
        Verified hole at 4096
        (beginning errno subtests...)
        report on test for E2BIG in probe (from past EOF): test successful
+       report on test for E2BIG in probe (to past EOF): test successful
        report on test for EACCES in no-right probe: test successful
        report on test for success in SHARED probe: test successful.
        report on test for success in EXCL probe: test successful.
        report on test for EACCES in no-right punch: test successful
        report on test for EACCES in SHARED punch: test successful
        report on test for success in EXCL punch: test successful.
-       report on test for EAGAIN in punch: test successful
        report on test for EBADF in probe: test successful
        report on test for EBADF in punch: test successful
        report on test for EFAULT in probe (null handle): test successful
index 9989b72ed6301976be04f9d4ff6d00056989e316..a000f26ed2536d83dd38cebeada2cfc224a1ddec 100644 (file)
@@ -69,7 +69,7 @@ main(
        dm_sessid_t     sid = DM_NO_SESSION;
        char            *pathname = NULL;
        char            *ls_path = NULL;
-       dm_off_t        offset = 0;
+       dm_off_t        offset = 0, end;
        dm_off_t        ex_off = 0;
        dm_extent_t     extent[20];
        u_int           nelem;
@@ -162,10 +162,16 @@ main(
          exit(1);
        }
 
+       /*
+        * The kernel always rounds the offset up to the next block
+        * size, so we can only probes up to the previous to last block.
+        */
+       end = (29604 / blocksize) * blocksize;
+
        /* Check that dm_probe_hole returns an extent from the next
         * highest multiple of the block size, to the end of the file
         */
-       for (offset = 0; offset < 29604; offset++) { 
+       for (offset = 0; offset < end; offset++) { 
          if (dm_probe_hole(sid, hanp, hlen, DM_NO_TOKEN, offset, length,
                            &roff, &rlen)) {
            fprintf(stdout, "dm_probe_hole failed on pass %lld (%s)\n",
@@ -275,15 +281,10 @@ main(
                dm_probe_hole(sid, hanp, hlen, DM_NO_TOKEN, 30000, length,
                              &roff, &rlen))
        /*---------------------------------------------------------*/
-#if 0
-       PROBLEM: No error is produced.  
-       off+len >= filesize should produce E2BIG...
-
        ERRTEST(E2BIG,
                "probe (to past EOF)",
                dm_probe_hole(sid, hanp, hlen, DM_NO_TOKEN, 15000, 150000,
                              &roff, &rlen))
-#endif
         /*---------------------------------------------------------*/
        SHAREDTEST("probe", hanp, hlen, test_token, 
                 dm_probe_hole(sid, hanp, hlen, test_token, 
@@ -292,10 +293,18 @@ main(
        EXCLTEST("punch", hanp, hlen, test_token, 
                   dm_punch_hole(sid, hanp, hlen, test_token, 0, 0)) 
        /*---------------------------------------------------------*/
+       /*
+        * No idea where that EAGAIN should come from, it's never
+        * returned from the kernel.
+        *
+        *              -- hch
+        */
+#if 0
        ERRTEST(EAGAIN,
                "punch",
                dm_punch_hole(sid, hanp, hlen, DM_NO_TOKEN,
                              1, length))       
+#endif
        /*---------------------------------------------------------*/
        if ((test_vp = handle_clone(hanp, hlen)) == NULL) {
          fprintf(stderr,