qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
@ 2015-04-11  3:41 Andreas Färber
  2015-04-11 14:33 ` Andreas Färber
                   ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Andreas Färber @ 2015-04-11  3:41 UTC (permalink / raw)
  To: qemu-devel; +Cc: Kevin Wolf, Peter Maydell, Jeff Cody, Stefan Hajnoczi

Hi,

001 seems to hang for -qcow (or is not reasonably "quick": >5 min).

033 is failing for -vhdx.

(Note that `make check-block` only tests -qcow2, so didn't uncover
either of them.)

Given a failing test, am I seeing correctly that there is no command
line option to skip this one failing test? -x seems to be for groups only.

Regards,
Andreas

$ ./check -v -T -qcow -g quick
[...]
001 6s ...        [05:12:39]

$ ./check -v -T -vhdx -g quick
[...]
033 1s ...        [04:06:09] [04:06:11] - output mismatch (see 033.out.bad)
--- /home/andreas/QEMU/qemu/tests/qemu-iotests/033.out	2015-04-08
20:19:11.947290578 +0200
+++ 033.out.bad	2015-04-11 04:06:11.939300375 +0200
@@ -2,52 +2,46 @@
 Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=134217728

 == preparing image ==
-wrote 1024/1024 bytes at offset 512
-1 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
-wrote 1536/1536 bytes at offset 131072
-1.500 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
-wrote 131072/131072 bytes at offset 1024
-128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
+write failed: Operation not supported
+write failed: Operation not supported
+write failed: Operation not supported

 == verifying patterns (1) ==
+Pattern verification failed at offset 512, 512 bytes
 read 512/512 bytes at offset 512
 512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
 read 131072/131072 bytes at offset 1024
 128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
+Pattern verification failed at offset 132096, 512 bytes
 read 512/512 bytes at offset 132096
 512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)

 == rewriting zeroes ==
-wrote 65536/65536 bytes at offset 65536
-64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
-wrote 65536/65536 bytes at offset 65536
-64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
+write failed: Operation not supported
+write failed: Operation not supported

 == verifying patterns (2) ==
 read 131072/131072 bytes at offset 1024
 128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)

 == preparing image ==
-wrote 1024/1024 bytes at offset 512
-1 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
-wrote 1536/1536 bytes at offset 131072
-1.500 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
-wrote 131072/131072 bytes at offset 1024
-128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
+write failed: Operation not supported
+write failed: Operation not supported
+write failed: Operation not supported

 == verifying patterns (1) ==
+Pattern verification failed at offset 512, 512 bytes
 read 512/512 bytes at offset 512
 512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
 read 131072/131072 bytes at offset 1024
 128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
+Pattern verification failed at offset 132096, 512 bytes
 read 512/512 bytes at offset 132096
 512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)

 == rewriting zeroes ==
-wrote 65536/65536 bytes at offset 65536
-64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
-wrote 65536/65536 bytes at offset 65536
-64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
+write failed: Operation not supported
+write failed: Operation not supported

 == verifying patterns (2) ==
 read 131072/131072 bytes at offset 1024
[...]
Not run: 017 018 019 020 024 025 027 028 029 031 034 035 036 037 038 039
042 045 046 047 048 050 053 054 058 059 060 062 063 065 066 067 068 069
071 073 074 075 077 078 081 082 084 086 087 088 089 090 092 094 095 098
101 102 103 107 108 110 111 113 114 116 123 128 130
Failures: 033
Failed 1 of 19 tests

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu,
Graham Norton; HRB 21284 (AG Nürnberg)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-11  3:41 [Qemu-devel] Failing iotests in v2.3.0-rc2 / master Andreas Färber
@ 2015-04-11 14:33 ` Andreas Färber
  2015-04-13 16:42   ` Paolo Bonzini
  2015-04-13 15:29 ` John Snow
  2015-04-14  9:57 ` Kevin Wolf
  2 siblings, 1 reply; 12+ messages in thread
From: Andreas Färber @ 2015-04-11 14:33 UTC (permalink / raw)
  To: qemu-devel
  Cc: Kevin Wolf, Peter Maydell, Jeff Cody, qemu-ppc, Stefan Hajnoczi

Am 11.04.2015 um 05:41 schrieb Andreas Färber:
> Hi,
> 
> 001 seems to hang for -qcow (or is not reasonably "quick": >5 min).
> 
> 033 is failing for -vhdx.

067 is failing for -qcow2 on ppc64 (no $QEMU_PROG => qemu-system-ppc64).
Seems to hang after the below output.

Andreas

$ tests/qemu-iotests-quick.sh
[...]
[  477s] 067                [14:16:53] [14:16:54] - output mismatch (see
067.out.bad)
[  477s] ---
/usr/src/packages/BUILD/qemu-2.3.0-rc2/tests/qemu-iotests/067.out
2015-04-02 22:10:37.000000000 +0000
[  477s] +++ 067.out.bad	2015-04-11 14:16:54.200000003 +0000
[  477s] @@ -60,7 +60,7 @@
[  477s]          },
[  477s]          {
[  477s]              "io-status": "ok",
[  477s] -            "device": "ide1-cd0",
[  477s] +            "device": "scsi0-cd2",
[  477s]              "locked": false,
[  477s]              "removable": true,
[  477s]              "tray_open": false,
[  477s] @@ -83,7 +83,9 @@
[  477s]      ]
[  477s]  }
[  477s]  {
[  477s] -    "return": {
[  477s] +    "error": {
[  477s] +        "class": "GenericError",
[  477s] +        "desc": "Bus 'pci.0' does not support hotplugging"
[  477s]      }
[  477s]  }
[  477s]  {
[  477s] @@ -95,34 +97,58 @@
[  477s]          "seconds":  TIMESTAMP,
[  477s]          "microseconds":  TIMESTAMP
[  477s]      },
[  477s] -    "event": "DEVICE_DELETED",
[  477s] -    "data": {
[  477s] -        "path": "/machine/peripheral/virtio0/virtio-backend"
[  477s] -    }
[  477s] +    "event": "RESET"
[  477s]  }
[  477s]  {
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s] -    "event": "DEVICE_DELETED",
[  477s] +    "return": [
[  477s] +        {
[  477s] +            "io-status": "ok",
[  477s] +            "device": "disk",
[  477s] +            "locked": false,
[  477s] +            "removable": false,
[  477s] +            "inserted": {
[  477s] +                "iops_rd": 0,
[  477s] +                "detect_zeroes": "off",
[  477s] +                "image": {
[  477s] +                    "virtual-size": 134217728,
[  477s] +                    "filename": "TEST_DIR/t.qcow2",
[  477s] +                    "cluster-size": 65536,
[  477s] +                    "format": "qcow2",
[  477s] +                    "actual-size": SIZE,
[  477s] +                    "format-specific": {
[  477s] +                        "type": "qcow2",
[  477s]      "data": {
[  477s] -        "device": "virtio0",
[  477s] -        "path": "/machine/peripheral/virtio0"
[  477s] -    }
[  477s] +                            "compat": "1.1",
[  477s] +                            "lazy-refcounts": false,
[  477s] +                            "refcount-bits": 16,
[  477s] +                            "corrupt": false
[  477s]  }
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s]      },
[  477s] -    "event": "RESET"
[  477s] -}
[  477s] -{
[  477s] -    "return": [
[  477s] +                    "dirty-flag": false
[  477s] +                },
[  477s] +                "iops_wr": 0,
[  477s] +                "ro": false,
[  477s] +                "backing_file_depth": 0,
[  477s] +                "drv": "qcow2",
[  477s] +                "iops": 0,
[  477s] +                "bps_wr": 0,
[  477s] +                "write_threshold": 0,
[  477s] +                "encrypted": false,
[  477s] +                "bps": 0,
[  477s] +                "bps_rd": 0,
[  477s] +                "cache": {
[  477s] +                    "no-flush": false,
[  477s] +                    "direct": false,
[  477s] +                    "writeback": true
[  477s] +                },
[  477s] +                "file": "TEST_DIR/t.qcow2",
[  477s] +                "encryption_key_missing": false
[  477s] +            },
[  477s] +            "type": "unknown"
[  477s] +        },
[  477s]          {
[  477s]              "io-status": "ok",
[  477s] -            "device": "ide1-cd0",
[  477s] +            "device": "scsi0-cd2",
[  477s]              "locked": false,
[  477s]              "removable": true,
[  477s]              "tray_open": false,
[  477s] @@ -162,18 +188,7 @@
[  477s]      },
[  477s]      "event": "DEVICE_TRAY_MOVED",
[  477s]      "data": {
[  477s] -        "device": "ide1-cd0",
[  477s] -        "tray-open": true
[  477s] -    }
[  477s] -}
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s] -    "event": "DEVICE_TRAY_MOVED",
[  477s] -    "data": {
[  477s] -        "device": "floppy0",
[  477s] +        "device": "scsi0-cd2",
[  477s]          "tray-open": true
[  477s]      }
[  477s]  }
[  477s] @@ -238,7 +253,7 @@
[  477s]          },
[  477s]          {
[  477s]              "io-status": "ok",
[  477s] -            "device": "ide1-cd0",
[  477s] +            "device": "scsi0-cd2",
[  477s]              "locked": false,
[  477s]              "removable": true,
[  477s]              "tray_open": false,
[  477s] @@ -261,11 +276,15 @@
[  477s]      ]
[  477s]  }
[  477s]  {
[  477s] -    "return": {
[  477s] +    "error": {
[  477s] +        "class": "GenericError",
[  477s] +        "desc": "Bus 'pci.0' does not support hotplugging"
[  477s]      }
[  477s]  }
[  477s]  {
[  477s] -    "return": {
[  477s] +    "error": {
[  477s] +        "class": "DeviceNotFound",
[  477s] +        "desc": "Device 'virtio0' not found"
[  477s]      }
[  477s]  }
[  477s]  {
[  477s] @@ -277,34 +296,58 @@
[  477s]          "seconds":  TIMESTAMP,
[  477s]          "microseconds":  TIMESTAMP
[  477s]      },
[  477s] -    "event": "DEVICE_DELETED",
[  477s] -    "data": {
[  477s] -        "path": "/machine/peripheral/virtio0/virtio-backend"
[  477s] -    }
[  477s] +    "event": "RESET"
[  477s]  }
[  477s]  {
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s] -    "event": "DEVICE_DELETED",
[  477s] +    "return": [
[  477s] +        {
[  477s] +            "device": "disk",
[  477s] +            "locked": false,
[  477s] +            "removable": true,
[  477s] +            "inserted": {
[  477s] +                "iops_rd": 0,
[  477s] +                "detect_zeroes": "off",
[  477s] +                "image": {
[  477s] +                    "virtual-size": 134217728,
[  477s] +                    "filename": "TEST_DIR/t.qcow2",
[  477s] +                    "cluster-size": 65536,
[  477s] +                    "format": "qcow2",
[  477s] +                    "actual-size": SIZE,
[  477s] +                    "format-specific": {
[  477s] +                        "type": "qcow2",
[  477s]      "data": {
[  477s] -        "device": "virtio0",
[  477s] -        "path": "/machine/peripheral/virtio0"
[  477s] -    }
[  477s] +                            "compat": "1.1",
[  477s] +                            "lazy-refcounts": false,
[  477s] +                            "refcount-bits": 16,
[  477s] +                            "corrupt": false
[  477s]  }
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s]      },
[  477s] -    "event": "RESET"
[  477s] -}
[  477s] -{
[  477s] -    "return": [
[  477s] +                    "dirty-flag": false
[  477s] +                },
[  477s] +                "iops_wr": 0,
[  477s] +                "ro": false,
[  477s] +                "backing_file_depth": 0,
[  477s] +                "drv": "qcow2",
[  477s] +                "iops": 0,
[  477s] +                "bps_wr": 0,
[  477s] +                "write_threshold": 0,
[  477s] +                "encrypted": false,
[  477s] +                "bps": 0,
[  477s] +                "bps_rd": 0,
[  477s] +                "cache": {
[  477s] +                    "no-flush": false,
[  477s] +                    "direct": false,
[  477s] +                    "writeback": true
[  477s] +                },
[  477s] +                "file": "TEST_DIR/t.qcow2",
[  477s] +                "encryption_key_missing": false
[  477s] +            },
[  477s] +            "tray_open": false,
[  477s] +            "type": "unknown"
[  477s] +        },
[  477s]          {
[  477s]              "io-status": "ok",
[  477s] -            "device": "ide1-cd0",
[  477s] +            "device": "scsi0-cd2",
[  477s]              "locked": false,
[  477s]              "removable": true,
[  477s]              "tray_open": false,
[  477s] @@ -344,18 +387,7 @@
[  477s]      },
[  477s]      "event": "DEVICE_TRAY_MOVED",
[  477s]      "data": {
[  477s] -        "device": "ide1-cd0",
[  477s] -        "tray-open": true
[  477s] -    }
[  477s] -}
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s] -    "event": "DEVICE_TRAY_MOVED",
[  477s] -    "data": {
[  477s] -        "device": "floppy0",
[  477s] +        "device": "scsi0-cd2",
[  477s]          "tray-open": true
[  477s]      }
[  477s]  }
[  477s] @@ -378,7 +410,7 @@
[  477s]      "return": [
[  477s]          {
[  477s]              "io-status": "ok",
[  477s] -            "device": "ide1-cd0",
[  477s] +            "device": "scsi0-cd2",
[  477s]              "locked": false,
[  477s]              "removable": true,
[  477s]              "tray_open": false,
[  477s] @@ -446,11 +478,15 @@
[  477s]      ]
[  477s]  }
[  477s]  {
[  477s] -    "return": {
[  477s] +    "error": {
[  477s] +        "class": "GenericError",
[  477s] +        "desc": "Bus 'pci.0' does not support hotplugging"
[  477s]      }
[  477s]  }
[  477s]  {
[  477s] -    "return": {
[  477s] +    "error": {
[  477s] +        "class": "DeviceNotFound",
[  477s] +        "desc": "Device 'virtio0' not found"
[  477s]      }
[  477s]  }
[  477s]  {
[  477s] @@ -462,34 +498,13 @@
[  477s]          "seconds":  TIMESTAMP,
[  477s]          "microseconds":  TIMESTAMP
[  477s]      },
[  477s] -    "event": "DEVICE_DELETED",
[  477s] -    "data": {
[  477s] -        "path": "/machine/peripheral/virtio0/virtio-backend"
[  477s] -    }
[  477s] -}
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s] -    "event": "DEVICE_DELETED",
[  477s] -    "data": {
[  477s] -        "device": "virtio0",
[  477s] -        "path": "/machine/peripheral/virtio0"
[  477s] -    }
[  477s] -}
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s]      "event": "RESET"
[  477s]  }
[  477s]  {
[  477s]      "return": [
[  477s]          {
[  477s]              "io-status": "ok",
[  477s] -            "device": "ide1-cd0",
[  477s] +            "device": "scsi0-cd2",
[  477s]              "locked": false,
[  477s]              "removable": true,
[  477s]              "tray_open": false,
[  477s] @@ -508,6 +523,51 @@
[  477s]              "removable": true,
[  477s]              "tray_open": false,
[  477s]              "type": "unknown"
[  477s] +        },
[  477s] +        {
[  477s] +            "device": "disk",
[  477s] +            "locked": false,
[  477s] +            "removable": true,
[  477s] +            "inserted": {
[  477s] +                "iops_rd": 0,
[  477s] +                "detect_zeroes": "off",
[  477s] +                "image": {
[  477s] +                    "virtual-size": 134217728,
[  477s] +                    "filename": "TEST_DIR/t.qcow2",
[  477s] +                    "cluster-size": 65536,
[  477s] +                    "format": "qcow2",
[  477s] +                    "actual-size": SIZE,
[  477s] +                    "format-specific": {
[  477s] +                        "type": "qcow2",
[  477s] +                        "data": {
[  477s] +                            "compat": "1.1",
[  477s] +                            "lazy-refcounts": false,
[  477s] +                            "refcount-bits": 16,
[  477s] +                            "corrupt": false
[  477s] +                        }
[  477s] +                    },
[  477s] +                    "dirty-flag": false
[  477s] +                },
[  477s] +                "iops_wr": 0,
[  477s] +                "ro": false,
[  477s] +                "backing_file_depth": 0,
[  477s] +                "drv": "qcow2",
[  477s] +                "iops": 0,
[  477s] +                "bps_wr": 0,
[  477s] +                "write_threshold": 0,
[  477s] +                "encrypted": false,
[  477s] +                "bps": 0,
[  477s] +                "bps_rd": 0,
[  477s] +                "cache": {
[  477s] +                    "no-flush": false,
[  477s] +                    "direct": false,
[  477s] +                    "writeback": true
[  477s] +                },
[  477s] +                "file": "TEST_DIR/t.qcow2",
[  477s] +                "encryption_key_missing": false
[  477s] +            },
[  477s] +            "tray_open": false,
[  477s] +            "type": "unknown"
[  477s]          }
[  477s]      ]
[  477s]  }
[  477s] @@ -529,18 +589,7 @@
[  477s]      },
[  477s]      "event": "DEVICE_TRAY_MOVED",
[  477s]      "data": {
[  477s] -        "device": "ide1-cd0",
[  477s] -        "tray-open": true
[  477s] -    }
[  477s] -}
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s] -    "event": "DEVICE_TRAY_MOVED",
[  477s] -    "data": {
[  477s] -        "device": "floppy0",
[  477s] +        "device": "scsi0-cd2",
[  477s]          "tray-open": true
[  477s]      }
[  477s]  }
[  477s] @@ -564,7 +613,7 @@
[  477s]      "return": [
[  477s]          {
[  477s]              "io-status": "ok",
[  477s] -            "device": "ide1-cd0",
[  477s] +            "device": "scsi0-cd2",
[  477s]              "locked": false,
[  477s]              "removable": true,
[  477s]              "tray_open": false,
[  477s] @@ -632,11 +681,15 @@
[  477s]      ]
[  477s]  }
[  477s]  {
[  477s] -    "return": {
[  477s] +    "error": {
[  477s] +        "class": "GenericError",
[  477s] +        "desc": "Bus 'pci.0' does not support hotplugging"
[  477s]      }
[  477s]  }
[  477s]  {
[  477s] -    "return": {
[  477s] +    "error": {
[  477s] +        "class": "DeviceNotFound",
[  477s] +        "desc": "Device 'virtio0' not found"
[  477s]      }
[  477s]  }
[  477s]  {
[  477s] @@ -648,34 +701,13 @@
[  477s]          "seconds":  TIMESTAMP,
[  477s]          "microseconds":  TIMESTAMP
[  477s]      },
[  477s] -    "event": "DEVICE_DELETED",
[  477s] -    "data": {
[  477s] -        "path": "/machine/peripheral/virtio0/virtio-backend"
[  477s] -    }
[  477s] -}
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s] -    "event": "DEVICE_DELETED",
[  477s] -    "data": {
[  477s] -        "device": "virtio0",
[  477s] -        "path": "/machine/peripheral/virtio0"
[  477s] -    }
[  477s] -}
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s]      "event": "RESET"
[  477s]  }
[  477s]  {
[  477s]      "return": [
[  477s]          {
[  477s]              "io-status": "ok",
[  477s] -            "device": "ide1-cd0",
[  477s] +            "device": "scsi0-cd2",
[  477s]              "locked": false,
[  477s]              "removable": true,
[  477s]              "tray_open": false,
[  477s] @@ -696,7 +728,6 @@
[  477s]              "type": "unknown"
[  477s]          },
[  477s]          {
[  477s] -            "io-status": "ok",
[  477s]              "device": "disk",
[  477s]              "locked": false,
[  477s]              "removable": true,
[  477s] @@ -761,18 +792,7 @@
[  477s]      },
[  477s]      "event": "DEVICE_TRAY_MOVED",
[  477s]      "data": {
[  477s] -        "device": "ide1-cd0",
[  477s] -        "tray-open": true
[  477s] -    }
[  477s] -}
[  477s] -{
[  477s] -    "timestamp": {
[  477s] -        "seconds":  TIMESTAMP,
[  477s] -        "microseconds":  TIMESTAMP
[  477s] -    },
[  477s] -    "event": "DEVICE_TRAY_MOVED",
[  477s] -    "data": {
[  477s] -        "device": "floppy0",
[  477s] +        "device": "scsi0-cd2",
[  477s]          "tray-open": true
[  477s]      }
[  477s]  }

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu,
Graham Norton; HRB 21284 (AG Nürnberg)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-11  3:41 [Qemu-devel] Failing iotests in v2.3.0-rc2 / master Andreas Färber
  2015-04-11 14:33 ` Andreas Färber
@ 2015-04-13 15:29 ` John Snow
  2015-04-13 17:03   ` Andreas Färber
  2015-04-14  9:57 ` Kevin Wolf
  2 siblings, 1 reply; 12+ messages in thread
From: John Snow @ 2015-04-13 15:29 UTC (permalink / raw)
  To: Andreas Färber, qemu-devel
  Cc: Kevin Wolf, Peter Maydell, Jeff Cody, Stefan Hajnoczi



On 04/10/2015 11:41 PM, Andreas Färber wrote:
> Hi,
>
> 001 seems to hang for -qcow (or is not reasonably "quick": >5 min).
>

I assume this was a one-off and you cannot reproduce it reliably.

Do you have any data on what call it might have hung on?

> 033 is failing for -vhdx.
>
> (Note that `make check-block` only tests -qcow2, so didn't uncover
> either of them.)
>
> Given a failing test, am I seeing correctly that there is no command
> line option to skip this one failing test? -x seems to be for groups only.
>
> Regards,
> Andreas
>
> $ ./check -v -T -qcow -g quick
> [...]
> 001 6s ...        [05:12:39]
>
> $ ./check -v -T -vhdx -g quick
> [...]
> 033 1s ...        [04:06:09] [04:06:11] - output mismatch (see 033.out.bad)
> --- /home/andreas/QEMU/qemu/tests/qemu-iotests/033.out	2015-04-08
> 20:19:11.947290578 +0200
> +++ 033.out.bad	2015-04-11 04:06:11.939300375 +0200
> @@ -2,52 +2,46 @@
>   Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=134217728
>
>   == preparing image ==
> -wrote 1024/1024 bytes at offset 512
> -1 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> -wrote 1536/1536 bytes at offset 131072
> -1.500 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> -wrote 131072/131072 bytes at offset 1024
> -128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> +write failed: Operation not supported
> +write failed: Operation not supported
> +write failed: Operation not supported
>
>   == verifying patterns (1) ==
> +Pattern verification failed at offset 512, 512 bytes
>   read 512/512 bytes at offset 512
>   512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
>   read 131072/131072 bytes at offset 1024
>   128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> +Pattern verification failed at offset 132096, 512 bytes
>   read 512/512 bytes at offset 132096
>   512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
>
>   == rewriting zeroes ==
> -wrote 65536/65536 bytes at offset 65536
> -64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> -wrote 65536/65536 bytes at offset 65536
> -64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> +write failed: Operation not supported
> +write failed: Operation not supported
>
>   == verifying patterns (2) ==
>   read 131072/131072 bytes at offset 1024
>   128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
>
>   == preparing image ==
> -wrote 1024/1024 bytes at offset 512
> -1 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> -wrote 1536/1536 bytes at offset 131072
> -1.500 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> -wrote 131072/131072 bytes at offset 1024
> -128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> +write failed: Operation not supported
> +write failed: Operation not supported
> +write failed: Operation not supported
>
>   == verifying patterns (1) ==
> +Pattern verification failed at offset 512, 512 bytes
>   read 512/512 bytes at offset 512
>   512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
>   read 131072/131072 bytes at offset 1024
>   128 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> +Pattern verification failed at offset 132096, 512 bytes
>   read 512/512 bytes at offset 132096
>   512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
>
>   == rewriting zeroes ==
> -wrote 65536/65536 bytes at offset 65536
> -64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> -wrote 65536/65536 bytes at offset 65536
> -64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> +write failed: Operation not supported
> +write failed: Operation not supported
>
>   == verifying patterns (2) ==
>   read 131072/131072 bytes at offset 1024
> [...]
> Not run: 017 018 019 020 024 025 027 028 029 031 034 035 036 037 038 039
> 042 045 046 047 048 050 053 054 058 059 060 062 063 065 066 067 068 069
> 071 073 074 075 077 078 081 082 084 086 087 088 089 090 092 094 095 098
> 101 102 103 107 108 110 111 113 114 116 123 128 130
> Failures: 033
> Failed 1 of 19 tests
>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-11 14:33 ` Andreas Färber
@ 2015-04-13 16:42   ` Paolo Bonzini
  0 siblings, 0 replies; 12+ messages in thread
From: Paolo Bonzini @ 2015-04-13 16:42 UTC (permalink / raw)
  To: Andreas Färber, qemu-devel
  Cc: Kevin Wolf, Peter Maydell, Jeff Cody, qemu-ppc, Stefan Hajnoczi



On 11/04/2015 16:33, Andreas Färber wrote:
> 067 is failing for -qcow2 on ppc64 (no $QEMU_PROG => qemu-system-ppc64).
> Seems to hang after the below output.

It really needs a PC machine, it seems.

Paolo

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-13 15:29 ` John Snow
@ 2015-04-13 17:03   ` Andreas Färber
  0 siblings, 0 replies; 12+ messages in thread
From: Andreas Färber @ 2015-04-13 17:03 UTC (permalink / raw)
  To: John Snow, qemu-devel
  Cc: Kevin Wolf, Peter Maydell, Jeff Cody, Stefan Hajnoczi

Am 13.04.2015 um 17:29 schrieb John Snow:
> On 04/10/2015 11:41 PM, Andreas Färber wrote:
>> Hi,
>>
>> 001 seems to hang for -qcow (or is not reasonably "quick": >5 min).
>>
> 
> I assume this was a one-off and you cannot reproduce it reliably.

No, I could reproduce it both locally and in our build service.
Note that this is about -qcow, not -qcow2.

Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu,
Graham Norton; HRB 21284 (AG Nürnberg)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-11  3:41 [Qemu-devel] Failing iotests in v2.3.0-rc2 / master Andreas Färber
  2015-04-11 14:33 ` Andreas Färber
  2015-04-13 15:29 ` John Snow
@ 2015-04-14  9:57 ` Kevin Wolf
  2015-04-15  4:53   ` Jeff Cody
  2015-04-15 11:27   ` Jeff Cody
  2 siblings, 2 replies; 12+ messages in thread
From: Kevin Wolf @ 2015-04-14  9:57 UTC (permalink / raw)
  To: Andreas Färber; +Cc: Peter Maydell, Jeff Cody, qemu-devel, Stefan Hajnoczi

Am 11.04.2015 um 05:41 hat Andreas Färber geschrieben:
> Hi,
> 
> 001 seems to hang for -qcow (or is not reasonably "quick": >5 min).
> 
> 033 is failing for -vhdx.
> 
> (Note that `make check-block` only tests -qcow2, so didn't uncover
> either of them.)
> 
> Given a failing test, am I seeing correctly that there is no command
> line option to skip this one failing test? -x seems to be for groups only.
> 
> Regards,
> Andreas
> 
> $ ./check -v -T -qcow -g quick
> [...]
> 001 6s ...        [05:12:39]

qcow1 is just really slow. 001 passes for me, after 202 seconds (that's
on my SSD, YMMV).

> $ ./check -v -T -vhdx -g quick
> [...]
> 033 1s ...        [04:06:09] [04:06:11] - output mismatch (see 033.out.bad)

This seems to be because blkdebug doesn't implement .bdrv_truncate.
Currently the test case isn't suitable for VHDX, which uses explicit
bdrv_truncate() calls to grow the image file. I'll send a patch for
blkdebug to allow this.

However, it seems that there is another problem which causes assertion
failures when using VHDX over blkdebug. Jeff, does the following fix
make sense to you? (I think it does, but I don't understand yet why the
assertion failure is only triggered with blkdebug - or in other words:
"how could this ever work?")

Kevin

--- a/block/vhdx.c
+++ b/block/vhdx.c
@@ -1285,7 +1285,7 @@ static coroutine_fn int vhdx_co_writev(BlockDriverState *bs, int64_t sector_num,
                         iov2.iov_base = qemu_blockalign(bs, iov2.iov_len);
                         memset(iov2.iov_base, 0, iov2.iov_len);
                         qemu_iovec_concat_iov(&hd_qiov, &iov2, 1, 0,
-                                              sinfo.block_offset);
+                                              iov2.iov_len);
                         sectors_to_write += iov2.iov_len >> BDRV_SECTOR_BITS;
                     }
                 }

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-14  9:57 ` Kevin Wolf
@ 2015-04-15  4:53   ` Jeff Cody
  2015-04-15  9:26     ` Kevin Wolf
  2015-04-15 11:27   ` Jeff Cody
  1 sibling, 1 reply; 12+ messages in thread
From: Jeff Cody @ 2015-04-15  4:53 UTC (permalink / raw)
  To: Kevin Wolf
  Cc: Peter Maydell, Andreas Färber, Stefan Hajnoczi, qemu-devel

On Tue, Apr 14, 2015 at 11:57:35AM +0200, Kevin Wolf wrote:
> Am 11.04.2015 um 05:41 hat Andreas Färber geschrieben:
> > Hi,
> > 
> > 001 seems to hang for -qcow (or is not reasonably "quick": >5 min).
> > 
> > 033 is failing for -vhdx.
> > 
> > (Note that `make check-block` only tests -qcow2, so didn't uncover
> > either of them.)
> > 
> > Given a failing test, am I seeing correctly that there is no command
> > line option to skip this one failing test? -x seems to be for groups only.
> > 
> > Regards,
> > Andreas
> > 
> > $ ./check -v -T -qcow -g quick
> > [...]
> > 001 6s ...        [05:12:39]
> 
> qcow1 is just really slow. 001 passes for me, after 202 seconds (that's
> on my SSD, YMMV).
> 
> > $ ./check -v -T -vhdx -g quick
> > [...]
> > 033 1s ...        [04:06:09] [04:06:11] - output mismatch (see 033.out.bad)
> 
> This seems to be because blkdebug doesn't implement .bdrv_truncate.
> Currently the test case isn't suitable for VHDX, which uses explicit
> bdrv_truncate() calls to grow the image file. I'll send a patch for
> blkdebug to allow this.
> 
> However, it seems that there is another problem which causes assertion
> failures when using VHDX over blkdebug. Jeff, does the following fix
> make sense to you? (I think it does, but I don't understand yet why the
> assertion failure is only triggered with blkdebug - or in other words:
> "how could this ever work?")
> 
> Kevin

Kevin,

Yes, looking at that fix it makes sense - we are wanting to pad the
back part of the block after the actual data with zeros. That back
length should be (block size - (bytes avail + block offset)), which is
iov2.iov_len.

There are two reasons I think we haven't seen this issue (it has been
hidden):

1.) If bs->file supports zero init, we don't do any of this

2.) This is done for the case when the existing BAT state is
PAYLOAD_BLOCK_ZERO.  Until recently (commit 30af51c), we didn't
create VHDX files with blocks in the PAYLOAD_BLOCK_ZERO state.

So it has been a latent bug in a hitherto rarely (if ever) exercised
path.

Jeff
> 
> --- a/block/vhdx.c
> +++ b/block/vhdx.c
> @@ -1285,7 +1285,7 @@ static coroutine_fn int vhdx_co_writev(BlockDriverState *bs, int64_t sector_num,
>                          iov2.iov_base = qemu_blockalign(bs, iov2.iov_len);
>                          memset(iov2.iov_base, 0, iov2.iov_len);
>                          qemu_iovec_concat_iov(&hd_qiov, &iov2, 1, 0,
> -                                              sinfo.block_offset);
> +                                              iov2.iov_len);
>                          sectors_to_write += iov2.iov_len >> BDRV_SECTOR_BITS;
>                      }
>                  }

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-15  4:53   ` Jeff Cody
@ 2015-04-15  9:26     ` Kevin Wolf
  2015-04-15  9:34       ` Andreas Färber
  0 siblings, 1 reply; 12+ messages in thread
From: Kevin Wolf @ 2015-04-15  9:26 UTC (permalink / raw)
  To: Jeff Cody; +Cc: Peter Maydell, Andreas Färber, Stefan Hajnoczi, qemu-devel

Am 15.04.2015 um 06:53 hat Jeff Cody geschrieben:
> On Tue, Apr 14, 2015 at 11:57:35AM +0200, Kevin Wolf wrote:
> > Am 11.04.2015 um 05:41 hat Andreas Färber geschrieben:
> > > Hi,
> > > 
> > > 001 seems to hang for -qcow (or is not reasonably "quick": >5 min).
> > > 
> > > 033 is failing for -vhdx.
> > > 
> > > (Note that `make check-block` only tests -qcow2, so didn't uncover
> > > either of them.)
> > > 
> > > Given a failing test, am I seeing correctly that there is no command
> > > line option to skip this one failing test? -x seems to be for groups only.
> > > 
> > > Regards,
> > > Andreas
> > > 
> > > $ ./check -v -T -qcow -g quick
> > > [...]
> > > 001 6s ...        [05:12:39]
> > 
> > qcow1 is just really slow. 001 passes for me, after 202 seconds (that's
> > on my SSD, YMMV).
> > 
> > > $ ./check -v -T -vhdx -g quick
> > > [...]
> > > 033 1s ...        [04:06:09] [04:06:11] - output mismatch (see 033.out.bad)
> > 
> > This seems to be because blkdebug doesn't implement .bdrv_truncate.
> > Currently the test case isn't suitable for VHDX, which uses explicit
> > bdrv_truncate() calls to grow the image file. I'll send a patch for
> > blkdebug to allow this.
> > 
> > However, it seems that there is another problem which causes assertion
> > failures when using VHDX over blkdebug. Jeff, does the following fix
> > make sense to you? (I think it does, but I don't understand yet why the
> > assertion failure is only triggered with blkdebug - or in other words:
> > "how could this ever work?")
> > 
> > Kevin
> 
> Kevin,
> 
> Yes, looking at that fix it makes sense - we are wanting to pad the
> back part of the block after the actual data with zeros. That back
> length should be (block size - (bytes avail + block offset)), which is
> iov2.iov_len.
> 
> There are two reasons I think we haven't seen this issue (it has been
> hidden):
> 
> 1.) If bs->file supports zero init, we don't do any of this

I see. file does and blkdebug doesn't, so that's the crucial difference.

> 2.) This is done for the case when the existing BAT state is
> PAYLOAD_BLOCK_ZERO.  Until recently (commit 30af51c), we didn't
> create VHDX files with blocks in the PAYLOAD_BLOCK_ZERO state.

Right, I wasn't aware of this either any more.

> So it has been a latent bug in a hitherto rarely (if ever) exercised
> path.

Thanks for your explanation, it's clear to me now what's going on. I'll
send out the patches (for both blkdebug and vhdx) right away. You can
either pick up the vhdx one, or just give your Acked-by and then I'll
merge it through my block tree.

Kevin

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-15  9:26     ` Kevin Wolf
@ 2015-04-15  9:34       ` Andreas Färber
  2015-04-15  9:47         ` Kevin Wolf
  0 siblings, 1 reply; 12+ messages in thread
From: Andreas Färber @ 2015-04-15  9:34 UTC (permalink / raw)
  To: Kevin Wolf, Jeff Cody; +Cc: Peter Maydell, qemu-devel, Stefan Hajnoczi

Am 15.04.2015 um 11:26 schrieb Kevin Wolf:
> Am 15.04.2015 um 06:53 hat Jeff Cody geschrieben:
>> On Tue, Apr 14, 2015 at 11:57:35AM +0200, Kevin Wolf wrote:
>>> Am 11.04.2015 um 05:41 hat Andreas Färber geschrieben:
>>>> Hi,
>>>>
>>>> 001 seems to hang for -qcow (or is not reasonably "quick": >5 min).
>>>>
>>>> 033 is failing for -vhdx.
>>>>
>>>> (Note that `make check-block` only tests -qcow2, so didn't uncover
>>>> either of them.)
>>>>
>>>> Given a failing test, am I seeing correctly that there is no command
>>>> line option to skip this one failing test? -x seems to be for groups only.
>>>>
>>>> Regards,
>>>> Andreas
>>>>
>>>> $ ./check -v -T -qcow -g quick
>>>> [...]
>>>> 001 6s ...        [05:12:39]
>>>
>>> qcow1 is just really slow. 001 passes for me, after 202 seconds (that's
>>> on my SSD, YMMV).
>>>
>>>> $ ./check -v -T -vhdx -g quick
>>>> [...]
>>>> 033 1s ...        [04:06:09] [04:06:11] - output mismatch (see 033.out.bad)
>>>
>>> This seems to be because blkdebug doesn't implement .bdrv_truncate.
>>> Currently the test case isn't suitable for VHDX, which uses explicit
>>> bdrv_truncate() calls to grow the image file. I'll send a patch for
>>> blkdebug to allow this.
>>>
>>> However, it seems that there is another problem which causes assertion
>>> failures when using VHDX over blkdebug. Jeff, does the following fix
>>> make sense to you? (I think it does, but I don't understand yet why the
>>> assertion failure is only triggered with blkdebug - or in other words:
>>> "how could this ever work?")
>>>
>>> Kevin
>>
>> Kevin,
>>
>> Yes, looking at that fix it makes sense - we are wanting to pad the
>> back part of the block after the actual data with zeros. That back
>> length should be (block size - (bytes avail + block offset)), which is
>> iov2.iov_len.
>>
>> There are two reasons I think we haven't seen this issue (it has been
>> hidden):
>>
>> 1.) If bs->file supports zero init, we don't do any of this
> 
> I see. file does and blkdebug doesn't, so that's the crucial difference.
> 
>> 2.) This is done for the case when the existing BAT state is
>> PAYLOAD_BLOCK_ZERO.  Until recently (commit 30af51c), we didn't
>> create VHDX files with blocks in the PAYLOAD_BLOCK_ZERO state.
> 
> Right, I wasn't aware of this either any more.
> 
>> So it has been a latent bug in a hitherto rarely (if ever) exercised
>> path.
> 
> Thanks for your explanation, it's clear to me now what's going on. I'll
> send out the patches (for both blkdebug and vhdx) right away. You can
> either pick up the vhdx one, or just give your Acked-by and then I'll
> merge it through my block tree.

Might 059 (?) failure for -vmdk be another symptom of the same issue?

Thanks,
Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu,
Graham Norton; HRB 21284 (AG Nürnberg)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-15  9:34       ` Andreas Färber
@ 2015-04-15  9:47         ` Kevin Wolf
  2015-04-15  9:54           ` Andreas Färber
  0 siblings, 1 reply; 12+ messages in thread
From: Kevin Wolf @ 2015-04-15  9:47 UTC (permalink / raw)
  To: Andreas Färber; +Cc: Peter Maydell, Jeff Cody, qemu-devel, Stefan Hajnoczi

Am 15.04.2015 um 11:34 hat Andreas Färber geschrieben:
> Am 15.04.2015 um 11:26 schrieb Kevin Wolf:
> > Am 15.04.2015 um 06:53 hat Jeff Cody geschrieben:
> >> On Tue, Apr 14, 2015 at 11:57:35AM +0200, Kevin Wolf wrote:
> >>> Am 11.04.2015 um 05:41 hat Andreas Färber geschrieben:
> >>>> Hi,
> >>>>
> >>>> 001 seems to hang for -qcow (or is not reasonably "quick": >5 min).
> >>>>
> >>>> 033 is failing for -vhdx.
> >>>>
> >>>> (Note that `make check-block` only tests -qcow2, so didn't uncover
> >>>> either of them.)
> >>>>
> >>>> Given a failing test, am I seeing correctly that there is no command
> >>>> line option to skip this one failing test? -x seems to be for groups only.
> >>>>
> >>>> Regards,
> >>>> Andreas
> >>>>
> >>>> $ ./check -v -T -qcow -g quick
> >>>> [...]
> >>>> 001 6s ...        [05:12:39]
> >>>
> >>> qcow1 is just really slow. 001 passes for me, after 202 seconds (that's
> >>> on my SSD, YMMV).
> >>>
> >>>> $ ./check -v -T -vhdx -g quick
> >>>> [...]
> >>>> 033 1s ...        [04:06:09] [04:06:11] - output mismatch (see 033.out.bad)
> >>>
> >>> This seems to be because blkdebug doesn't implement .bdrv_truncate.
> >>> Currently the test case isn't suitable for VHDX, which uses explicit
> >>> bdrv_truncate() calls to grow the image file. I'll send a patch for
> >>> blkdebug to allow this.
> >>>
> >>> However, it seems that there is another problem which causes assertion
> >>> failures when using VHDX over blkdebug. Jeff, does the following fix
> >>> make sense to you? (I think it does, but I don't understand yet why the
> >>> assertion failure is only triggered with blkdebug - or in other words:
> >>> "how could this ever work?")
> >>>
> >>> Kevin
> >>
> >> Kevin,
> >>
> >> Yes, looking at that fix it makes sense - we are wanting to pad the
> >> back part of the block after the actual data with zeros. That back
> >> length should be (block size - (bytes avail + block offset)), which is
> >> iov2.iov_len.
> >>
> >> There are two reasons I think we haven't seen this issue (it has been
> >> hidden):
> >>
> >> 1.) If bs->file supports zero init, we don't do any of this
> > 
> > I see. file does and blkdebug doesn't, so that's the crucial difference.
> > 
> >> 2.) This is done for the case when the existing BAT state is
> >> PAYLOAD_BLOCK_ZERO.  Until recently (commit 30af51c), we didn't
> >> create VHDX files with blocks in the PAYLOAD_BLOCK_ZERO state.
> > 
> > Right, I wasn't aware of this either any more.
> > 
> >> So it has been a latent bug in a hitherto rarely (if ever) exercised
> >> path.
> > 
> > Thanks for your explanation, it's clear to me now what's going on. I'll
> > send out the patches (for both blkdebug and vhdx) right away. You can
> > either pick up the vhdx one, or just give your Acked-by and then I'll
> > merge it through my block tree.
> 
> Might 059 (?) failure for -vmdk be another symptom of the same issue?

059 passes for me with vmdk. If it fails for you, can you please paste
the output diff?

Kevin

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-15  9:47         ` Kevin Wolf
@ 2015-04-15  9:54           ` Andreas Färber
  0 siblings, 0 replies; 12+ messages in thread
From: Andreas Färber @ 2015-04-15  9:54 UTC (permalink / raw)
  To: Kevin Wolf
  Cc: Peter Maydell, Jeff Cody, qemu-devel, Stefan Hajnoczi,
	Bruce Rogers

Am 15.04.2015 um 11:47 schrieb Kevin Wolf:
> Am 15.04.2015 um 11:34 hat Andreas Färber geschrieben:
>> Am 15.04.2015 um 11:26 schrieb Kevin Wolf:
>>> Am 15.04.2015 um 06:53 hat Jeff Cody geschrieben:
>>>> On Tue, Apr 14, 2015 at 11:57:35AM +0200, Kevin Wolf wrote:
>>>>> Am 11.04.2015 um 05:41 hat Andreas Färber geschrieben:
>>>>>> Hi,
>>>>>>
>>>>>> 001 seems to hang for -qcow (or is not reasonably "quick": >5 min).
>>>>>>
>>>>>> 033 is failing for -vhdx.
>>>>>>
>>>>>> (Note that `make check-block` only tests -qcow2, so didn't uncover
>>>>>> either of them.)
>>>>>>
>>>>>> Given a failing test, am I seeing correctly that there is no command
>>>>>> line option to skip this one failing test? -x seems to be for groups only.
>>>>>>
>>>>>> Regards,
>>>>>> Andreas
>>>>>>
>>>>>> $ ./check -v -T -qcow -g quick
>>>>>> [...]
>>>>>> 001 6s ...        [05:12:39]
>>>>>
>>>>> qcow1 is just really slow. 001 passes for me, after 202 seconds (that's
>>>>> on my SSD, YMMV).
>>>>>
>>>>>> $ ./check -v -T -vhdx -g quick
>>>>>> [...]
>>>>>> 033 1s ...        [04:06:09] [04:06:11] - output mismatch (see 033.out.bad)
>>>>>
>>>>> This seems to be because blkdebug doesn't implement .bdrv_truncate.
>>>>> Currently the test case isn't suitable for VHDX, which uses explicit
>>>>> bdrv_truncate() calls to grow the image file. I'll send a patch for
>>>>> blkdebug to allow this.
>>>>>
>>>>> However, it seems that there is another problem which causes assertion
>>>>> failures when using VHDX over blkdebug. Jeff, does the following fix
>>>>> make sense to you? (I think it does, but I don't understand yet why the
>>>>> assertion failure is only triggered with blkdebug - or in other words:
>>>>> "how could this ever work?")
>>>>>
>>>>> Kevin
>>>>
>>>> Kevin,
>>>>
>>>> Yes, looking at that fix it makes sense - we are wanting to pad the
>>>> back part of the block after the actual data with zeros. That back
>>>> length should be (block size - (bytes avail + block offset)), which is
>>>> iov2.iov_len.
>>>>
>>>> There are two reasons I think we haven't seen this issue (it has been
>>>> hidden):
>>>>
>>>> 1.) If bs->file supports zero init, we don't do any of this
>>>
>>> I see. file does and blkdebug doesn't, so that's the crucial difference.
>>>
>>>> 2.) This is done for the case when the existing BAT state is
>>>> PAYLOAD_BLOCK_ZERO.  Until recently (commit 30af51c), we didn't
>>>> create VHDX files with blocks in the PAYLOAD_BLOCK_ZERO state.
>>>
>>> Right, I wasn't aware of this either any more.
>>>
>>>> So it has been a latent bug in a hitherto rarely (if ever) exercised
>>>> path.
>>>
>>> Thanks for your explanation, it's clear to me now what's going on. I'll
>>> send out the patches (for both blkdebug and vhdx) right away. You can
>>> either pick up the vhdx one, or just give your Acked-by and then I'll
>>> merge it through my block tree.
>>
>> Might 059 (?) failure for -vmdk be another symptom of the same issue?
> 
> 059 passes for me with vmdk. If it fails for you, can you please paste
> the output diff?

With v2.3.0-rc3:

-raw:

[ 1714s] 005                [02:45:31] [02:45:31] - output mismatch (see
005.out.bad)
[ 1714s] ---
/home/abuild/rpmbuild/BUILD/qemu-2.3.0-rc3/tests/qemu-iotests/005.out
2015-04-13 18:34:01.000000000 +0000
[ 1714s] +++ 005.out.bad	2015-04-15 02:45:31.540000000 +0000
[ 1714s] @@ -1,13 +1,12 @@
[ 1714s]  QA output created by 005
[ 1714s]
[ 1714s]  creating large image
[ 1714s] +qemu-img: TEST_DIR/t.IMGFMT: The image size is too large for
file format 'IMGFMT'
[ 1714s]  Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=5368709120000
[ 1714s]
[ 1714s]  small read
[ 1714s] -read 4096/4096 bytes at offset 1024
[ 1714s] -4 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
[ 1714s] +read failed: Input/output error
[ 1714s]
[ 1714s]  small write
[ 1714s] -wrote 4096/4096 bytes at offset 8192
[ 1714s] -4 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
[ 1714s] +write failed: Input/output error
[ 1714s]  *** done

-qcow2:

[ 1867s] 028                [02:47:55] [02:48:04] - output mismatch (see
028.out.bad)
[ 1867s] ---
/home/abuild/rpmbuild/BUILD/qemu-2.3.0-rc3/tests/qemu-iotests/028.out
2015-04-13 18:34:01.000000000 +0000
[ 1867s] +++ 028.out.bad	2015-04-15 02:48:04.908000000 +0000
[ 1867s] @@ -473,6 +473,14 @@
[ 1867s]  (qemu)
i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1867s]  Type backup, device disk: Completed 0 of 4294968832 bytes,
speed limit 0 bytes/s
[ 1867s]
i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1867s] +Type backup, device disk: Completed 0 of 4294968832 bytes,
speed limit 0 bytes/s
[ 1867s]
+i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1867s] +Type backup, device disk: Completed 0 of 4294968832 bytes,
speed limit 0 bytes/s
[ 1867s]
+i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1867s] +Type backup, device disk: Completed 0 of 4294968832 bytes,
speed limit 0 bytes/s
[ 1867s]
+i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1867s] +Type backup, device disk: Completed 131072 of 4294968832
bytes, speed limit 0 bytes/s
[ 1867s]
+i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1867s]  No active jobs
[ 1867s]  === IO: pattern 195
[ 1867s]  read 512/512 bytes at offset 3221194240

-qed:

[ 1946s] 028                [02:49:13] [02:49:23] - output mismatch (see
028.out.bad)
[ 1946s] ---
/home/abuild/rpmbuild/BUILD/qemu-2.3.0-rc3/tests/qemu-iotests/028.out
2015-04-13 18:34:01.000000000 +0000
[ 1946s] +++ 028.out.bad	2015-04-15 02:49:23.444000000 +0000
[ 1946s] @@ -473,6 +473,14 @@
[ 1946s]  (qemu)
i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1946s]  Type backup, device disk: Completed 0 of 4294968832 bytes,
speed limit 0 bytes/s
[ 1946s]
i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1946s] +Type backup, device disk: Completed 0 of 4294968832 bytes,
speed limit 0 bytes/s
[ 1946s]
+i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1946s] +Type backup, device disk: Completed 0 of 4294968832 bytes,
speed limit 0 bytes/s
[ 1946s]
+i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1946s] +Type backup, device disk: Completed 0 of 4294968832 bytes,
speed limit 0 bytes/s
[ 1946s]
+i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1946s] +Type backup, device disk: Completed 131072 of 4294968832
bytes, speed limit 0 bytes/s
[ 1946s]
+i^[[K^[[Din^[[K^[[D^[[Dinf^[[K^[[D^[[D^[[Dinfo^[[K^[[D^[[D^[[D^[[Dinfo
^[[K^[[D^[[D^[[D^[[D^[[Dinfo b^[[K^[[D^[[D^[[D^[[D^[[D^[[Dinfo
bl^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
blo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo bloc^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-j^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jo^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo
block-job^[[K^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[D^[[Dinfo block-jobs^[[K
[ 1946s]  No active jobs
[ 1946s]  === IO: pattern 195
[ 1946s]  read 512/512 bytes at offset 3221194240

-vmdk:

[ 2125s] 059                [02:52:18] [02:52:22] - output mismatch (see
059.out.bad)
[ 2125s] ---
/home/abuild/rpmbuild/BUILD/qemu-2.3.0-rc3/tests/qemu-iotests/059.out
2015-04-13 18:34:01.000000000 +0000
[ 2125s] +++ 059.out.bad	2015-04-15 02:52:22.364000000 +0000
[ 2125s] @@ -2264,76 +2264,14 @@
[ 2125s]  512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
[ 2125s]
[ 2125s]  === Testing 4TB monolithicFlat creation and IO ===
[ 2125s] +qemu-img: TEST_DIR/iotest-version3.IMGFMT: Could not truncate
file: File too large
[ 2125s]  Formatting 'TEST_DIR/iotest-version3.IMGFMT', fmt=IMGFMT
size=4398046511104
[ 2125s]  image: TEST_DIR/iotest-version3.IMGFMT
[ 2125s]  file format: IMGFMT
[ 2125s] -virtual size: 4.0T (4398046511104 bytes)
[ 2125s] -wrote 512/512 bytes at offset 966367641600
[ 2125s] -512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
[ 2125s] -e100000000:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000010:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000020:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000030:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000040:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000050:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000060:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000070:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000080:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000090:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000000a0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000000b0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000000c0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000000d0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000000e0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000000f0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000100:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000110:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000120:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000130:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000140:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000150:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000160:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000170:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000180:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000190:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000001a0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000001b0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000001c0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000001d0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000001e0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e1000001f0:  0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a 0a
................
[ 2125s] -e100000200:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000210:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000220:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000230:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000240:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000250:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000260:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000270:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000280:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000290:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000002a0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000002b0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000002c0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000002d0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000002e0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000002f0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000300:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000310:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000320:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000330:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000340:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000350:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000360:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000370:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000380:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e100000390:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000003a0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000003b0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000003c0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000003d0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000003e0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -e1000003f0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
................
[ 2125s] -read 1024/1024 bytes at offset 966367641600
[ 2125s] -1 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
[ 2125s] +virtual size: 16G (17179869184 bytes)
[ 2125s] +cluster_size: 65536
[ 2125s] +qemu-io: can't open device
/home/abuild/rpmbuild/BUILD/qemu-2.3.0-rc3/tests/qemu-iotests/scratch/iotest-version3.vmdk:
VMDK version 3 must be read only
[ 2125s] +no file open, try 'help open'
[ 2125s] +qemu-io: can't open device
/home/abuild/rpmbuild/BUILD/qemu-2.3.0-rc3/tests/qemu-iotests/scratch/iotest-version3.vmdk:
VMDK version 3 must be read only
[ 2125s] +no file open, try 'help open'
[ 2125s]  *** done

https://build.opensuse.org/build/Virtualization/openSUSE_Factory/x86_64/qemu-testsuite/_log

Regards,
Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu,
Graham Norton; HRB 21284 (AG Nürnberg)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Failing iotests in v2.3.0-rc2 / master
  2015-04-14  9:57 ` Kevin Wolf
  2015-04-15  4:53   ` Jeff Cody
@ 2015-04-15 11:27   ` Jeff Cody
  1 sibling, 0 replies; 12+ messages in thread
From: Jeff Cody @ 2015-04-15 11:27 UTC (permalink / raw)
  To: Kevin Wolf
  Cc: Peter Maydell, Andreas Färber, Stefan Hajnoczi, qemu-devel

On Tue, Apr 14, 2015 at 11:57:35AM +0200, Kevin Wolf wrote:

[snip]

> --- a/block/vhdx.c
> +++ b/block/vhdx.c
> @@ -1285,7 +1285,7 @@ static coroutine_fn int vhdx_co_writev(BlockDriverState *bs, int64_t sector_num,
>                          iov2.iov_base = qemu_blockalign(bs, iov2.iov_len);
>                          memset(iov2.iov_base, 0, iov2.iov_len);
>                          qemu_iovec_concat_iov(&hd_qiov, &iov2, 1, 0,
> -                                              sinfo.block_offset);
> +                                              iov2.iov_len);
>                          sectors_to_write += iov2.iov_len >> BDRV_SECTOR_BITS;
>                      }
>                  }

acked-by: Jeff Cody <jcody@redhat.com>

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2015-04-15 11:27 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-11  3:41 [Qemu-devel] Failing iotests in v2.3.0-rc2 / master Andreas Färber
2015-04-11 14:33 ` Andreas Färber
2015-04-13 16:42   ` Paolo Bonzini
2015-04-13 15:29 ` John Snow
2015-04-13 17:03   ` Andreas Färber
2015-04-14  9:57 ` Kevin Wolf
2015-04-15  4:53   ` Jeff Cody
2015-04-15  9:26     ` Kevin Wolf
2015-04-15  9:34       ` Andreas Färber
2015-04-15  9:47         ` Kevin Wolf
2015-04-15  9:54           ` Andreas Färber
2015-04-15 11:27   ` Jeff Cody

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).