Re: [PATCH v3 2/4] iotests: Switch nbd tests to use Unix rather than TCP

2019-11-18 Thread Eric Blake
On 11/18/19 4:18 PM, Eric Blake wrote: If anything, I'm inclined to use $SOCK_DIR/nbd.raw to indicate that the NBD client sees raw format, regardless of the format in use by the server, to leave the door open for $SOCK_DIR/nbd.qcow2 when we finally are happy to test qcow2 format over NBD.

Re: [PATCH v3 2/4] iotests: Switch nbd tests to use Unix rather than TCP

2019-11-18 Thread Eric Blake
On 11/18/19 11:42 AM, Eric Blake wrote: - TEST_IMG="$DRIVER,file.driver=nbd,file.host=127.0.0.1,file.port=10810" + TEST_IMG="$DRIVER,file.driver=nbd,file.type=unix,file.path=$SOCKDIR/$IMGFMT" Maybe nbd.$IMGFMT? At first glance, it seems reasonable.  But reading further,

Re: [PATCH v3 2/4] iotests: Switch nbd tests to use Unix rather than TCP

2019-11-18 Thread Max Reitz
On 18.11.19 18:42, Eric Blake wrote: > On 11/18/19 11:29 AM, Max Reitz wrote: >> On 14.11.19 22:34, Eric Blake wrote: >>> Up to now, all it took to cause a lot of iotest failures was to have a >>> background process such as 'nbdkit -p 10810 null' running, because we >>> hard-coded the TCP port.  Sw

Re: [PATCH v3 2/4] iotests: Switch nbd tests to use Unix rather than TCP

2019-11-18 Thread Eric Blake
On 11/18/19 11:42 AM, Eric Blake wrote: +++ b/tests/qemu-iotests/common.filter @@ -127,7 +127,8 @@ _filter_img_create()   -e "s#$TEST_DIR#TEST_DIR#g" \   -e "s#$SOCK_DIR#SOCK_DIR#g" \   -e "s#$IMGFMT#IMGFMT#g" \ -    -e 's#nbd:127.0.0.1:10810#TEST_DIR/t.IMGFMT#g' \ + 

Re: [PATCH v3 2/4] iotests: Switch nbd tests to use Unix rather than TCP

2019-11-18 Thread Eric Blake
On 11/18/19 11:29 AM, Max Reitz wrote: On 14.11.19 22:34, Eric Blake wrote: Up to now, all it took to cause a lot of iotest failures was to have a background process such as 'nbdkit -p 10810 null' running, because we hard-coded the TCP port. Switching to a Unix socket eliminates this contention

Re: [PATCH v3 2/4] iotests: Switch nbd tests to use Unix rather than TCP

2019-11-18 Thread Max Reitz
On 14.11.19 22:34, Eric Blake wrote: > Up to now, all it took to cause a lot of iotest failures was to have a > background process such as 'nbdkit -p 10810 null' running, because we > hard-coded the TCP port. Switching to a Unix socket eliminates this > contention. We still have TCP coverage in t

[PATCH v3 2/4] iotests: Switch nbd tests to use Unix rather than TCP

2019-11-14 Thread Eric Blake
Up to now, all it took to cause a lot of iotest failures was to have a background process such as 'nbdkit -p 10810 null' running, because we hard-coded the TCP port. Switching to a Unix socket eliminates this contention. We still have TCP coverage in test 233, and that test is more careful to not