* [PATCH] fstests: Add virtio-fs shared file system support
@ 2019-10-15 8:32 QI Fuli
2019-10-18 9:15 ` [Virtio-fs] " Eryu Guan
0 siblings, 1 reply; 2+ messages in thread
From: QI Fuli @ 2019-10-15 8:32 UTC (permalink / raw)
To: fstests, virtio-fs; +Cc: QI Fuli, Dr . David Alan Gilbert, Misono Tomohiro
This patch adds support for virtio-fs shared file system that lets
virtual machines access a directory tree on the host.
To run xfstests on it, first, start virtiofsd daemon in host:
./virtiofsd -o vhost_user_socket=/tmp/vhostqemu0 -o source=$DIR0 -o cache=always
./virtiofsd -o vhost_user_socket=/tmp/vhostqemu1 -o source=$DIR1 -o cache=always
second, launch QEMU with:
-chardev socket,id=char0,path=/tmp/vhostqemu0
-device vhost-user-fs-pci,queue-size=1024,chardev=char0,tag=myfs0
-chardev socket,id=char1,path=/tmp/vhostqemu1
-device vhost-user-fs-pci,queue-size=1024,chardev=char1,tag=myfs1
-m 8G
-object memory-backend-file,id=mem,size=8G,mem-path=/dev/shm,share=on
-numa node,memdev=mem
then, inside the VM run xfstests with:
export TEST_DEV=myfs0
export TEST_DIR=$TESTDIR
export SCRATCH_DEV=myfs1
export SCRATCH_MNT=$SCRATCHMNT
export MOUNT_OPTIONS=""
export TEST_FS_MOUNT_OPTS=""
Cc: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Misono Tomohiro <misono.tomohiro@fujitsu.com>
Signed-off-by: QI Fuli <qi.fuli@fujitsu.com>
---
check | 2 ++
common/config | 4 ++--
common/rc | 18 +++++++++++++++---
3 files changed, 19 insertions(+), 5 deletions(-)
diff --git a/check b/check
index c7f1dc5e..2e148e57 100755
--- a/check
+++ b/check
@@ -56,6 +56,7 @@ check options
-glusterfs test GlusterFS
-cifs test CIFS
-9p test 9p
+ -virtiofs test virtiofs
-overlay test overlay
-pvfs2 test PVFS2
-tmpfs test TMPFS
@@ -268,6 +269,7 @@ while [ $# -gt 0 ]; do
-glusterfs) FSTYP=glusterfs ;;
-cifs) FSTYP=cifs ;;
-9p) FSTYP=9p ;;
+ -virtiofs) FSTYP=virtiofs ;;
-overlay) FSTYP=overlay; export OVERLAY=true ;;
-pvfs2) FSTYP=pvfs2 ;;
-tmpfs) FSTYP=tmpfs ;;
diff --git a/common/config b/common/config
index a0d58bde..2cb0202a 100644
--- a/common/config
+++ b/common/config
@@ -480,8 +480,8 @@ _check_device()
fi
case "$FSTYP" in
- 9p|tmpfs)
- # 9p mount tags are just plain strings, so anything is allowed
+ 9p|tmpfs|virtiofs)
+ # 9p and virtiofs mount tags are just plain strings, so anything is allowed
# tmpfs doesn't use mount source, ignore
;;
overlay)
diff --git a/common/rc b/common/rc
index cfaabf10..3d5c8b23 100644
--- a/common/rc
+++ b/common/rc
@@ -603,6 +603,9 @@ _test_mkfs()
9p)
# do nothing for 9p
;;
+ virtiofs)
+ # do nothing for virtiofs
+ ;;
ceph)
# do nothing for ceph
;;
@@ -640,6 +643,9 @@ _mkfs_dev()
9p)
# do nothing for 9p
;;
+ virtiofs)
+ # do nothing for virtiofs
+ ;;
overlay)
# do nothing for overlay
;;
@@ -704,7 +710,7 @@ _scratch_mkfs()
local mkfs_status
case $FSTYP in
- nfs*|cifs|ceph|overlay|glusterfs|pvfs2|9p)
+ nfs*|cifs|ceph|overlay|glusterfs|pvfs2|9p|virtiofs)
# unable to re-create this fstyp, just remove all files in
# $SCRATCH_MNT to avoid EEXIST caused by the leftover files
# created in previous runs
@@ -1467,7 +1473,7 @@ _require_scratch_nocheck()
_notrun "this test requires a valid \$SCRATCH_MNT"
fi
;;
- 9p)
+ 9p|virtiofs)
if [ -z "$SCRATCH_DEV" ]; then
_notrun "this test requires a valid \$SCRATCH_DEV"
fi
@@ -1591,7 +1597,7 @@ _require_test()
_notrun "this test requires a valid \$TEST_DIR"
fi
;;
- 9p)
+ 9p|virtiofs)
if [ -z "$TEST_DEV" ]; then
_notrun "this test requires a valid \$TEST_DEV"
fi
@@ -2686,6 +2692,9 @@ _check_test_fs()
9p)
# no way to check consistency for 9p
;;
+ virtiofs)
+ # no way to check consistency for virtiofs
+ ;;
ceph)
# no way to check consistency for CephFS
;;
@@ -2744,6 +2753,9 @@ _check_scratch_fs()
9p)
# no way to check consistency for 9p
;;
+ virtiofs)
+ # no way to check consistency for virtiofs
+ ;;
ceph)
# no way to check consistency for CephFS
;;
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Virtio-fs] [PATCH] fstests: Add virtio-fs shared file system support
2019-10-15 8:32 [PATCH] fstests: Add virtio-fs shared file system support QI Fuli
@ 2019-10-18 9:15 ` Eryu Guan
0 siblings, 0 replies; 2+ messages in thread
From: Eryu Guan @ 2019-10-18 9:15 UTC (permalink / raw)
To: QI Fuli; +Cc: fstests, virtio-fs, Misono Tomohiro
On Tue, Oct 15, 2019 at 05:32:28PM +0900, QI Fuli wrote:
> This patch adds support for virtio-fs shared file system that lets
> virtual machines access a directory tree on the host.
>
> To run xfstests on it, first, start virtiofsd daemon in host:
> ./virtiofsd -o vhost_user_socket=/tmp/vhostqemu0 -o source=$DIR0 -o cache=always
> ./virtiofsd -o vhost_user_socket=/tmp/vhostqemu1 -o source=$DIR1 -o cache=always
>
> second, launch QEMU with:
> -chardev socket,id=char0,path=/tmp/vhostqemu0
> -device vhost-user-fs-pci,queue-size=1024,chardev=char0,tag=myfs0
> -chardev socket,id=char1,path=/tmp/vhostqemu1
> -device vhost-user-fs-pci,queue-size=1024,chardev=char1,tag=myfs1
> -m 8G
> -object memory-backend-file,id=mem,size=8G,mem-path=/dev/shm,share=on
> -numa node,memdev=mem
>
> then, inside the VM run xfstests with:
> export TEST_DEV=myfs0
> export TEST_DIR=$TESTDIR
> export SCRATCH_DEV=myfs1
> export SCRATCH_MNT=$SCRATCHMNT
> export MOUNT_OPTIONS=""
> export TEST_FS_MOUNT_OPTS=""
>
> Cc: Dr. David Alan Gilbert <dgilbert@redhat.com>
> Signed-off-by: Misono Tomohiro <misono.tomohiro@fujitsu.com>
> Signed-off-by: QI Fuli <qi.fuli@fujitsu.com>
Looks good to me, thanks for the patch!
Eryu
> ---
> check | 2 ++
> common/config | 4 ++--
> common/rc | 18 +++++++++++++++---
> 3 files changed, 19 insertions(+), 5 deletions(-)
>
> diff --git a/check b/check
> index c7f1dc5e..2e148e57 100755
> --- a/check
> +++ b/check
> @@ -56,6 +56,7 @@ check options
> -glusterfs test GlusterFS
> -cifs test CIFS
> -9p test 9p
> + -virtiofs test virtiofs
> -overlay test overlay
> -pvfs2 test PVFS2
> -tmpfs test TMPFS
> @@ -268,6 +269,7 @@ while [ $# -gt 0 ]; do
> -glusterfs) FSTYP=glusterfs ;;
> -cifs) FSTYP=cifs ;;
> -9p) FSTYP=9p ;;
> + -virtiofs) FSTYP=virtiofs ;;
> -overlay) FSTYP=overlay; export OVERLAY=true ;;
> -pvfs2) FSTYP=pvfs2 ;;
> -tmpfs) FSTYP=tmpfs ;;
> diff --git a/common/config b/common/config
> index a0d58bde..2cb0202a 100644
> --- a/common/config
> +++ b/common/config
> @@ -480,8 +480,8 @@ _check_device()
> fi
>
> case "$FSTYP" in
> - 9p|tmpfs)
> - # 9p mount tags are just plain strings, so anything is allowed
> + 9p|tmpfs|virtiofs)
> + # 9p and virtiofs mount tags are just plain strings, so anything is allowed
> # tmpfs doesn't use mount source, ignore
> ;;
> overlay)
> diff --git a/common/rc b/common/rc
> index cfaabf10..3d5c8b23 100644
> --- a/common/rc
> +++ b/common/rc
> @@ -603,6 +603,9 @@ _test_mkfs()
> 9p)
> # do nothing for 9p
> ;;
> + virtiofs)
> + # do nothing for virtiofs
> + ;;
> ceph)
> # do nothing for ceph
> ;;
> @@ -640,6 +643,9 @@ _mkfs_dev()
> 9p)
> # do nothing for 9p
> ;;
> + virtiofs)
> + # do nothing for virtiofs
> + ;;
> overlay)
> # do nothing for overlay
> ;;
> @@ -704,7 +710,7 @@ _scratch_mkfs()
> local mkfs_status
>
> case $FSTYP in
> - nfs*|cifs|ceph|overlay|glusterfs|pvfs2|9p)
> + nfs*|cifs|ceph|overlay|glusterfs|pvfs2|9p|virtiofs)
> # unable to re-create this fstyp, just remove all files in
> # $SCRATCH_MNT to avoid EEXIST caused by the leftover files
> # created in previous runs
> @@ -1467,7 +1473,7 @@ _require_scratch_nocheck()
> _notrun "this test requires a valid \$SCRATCH_MNT"
> fi
> ;;
> - 9p)
> + 9p|virtiofs)
> if [ -z "$SCRATCH_DEV" ]; then
> _notrun "this test requires a valid \$SCRATCH_DEV"
> fi
> @@ -1591,7 +1597,7 @@ _require_test()
> _notrun "this test requires a valid \$TEST_DIR"
> fi
> ;;
> - 9p)
> + 9p|virtiofs)
> if [ -z "$TEST_DEV" ]; then
> _notrun "this test requires a valid \$TEST_DEV"
> fi
> @@ -2686,6 +2692,9 @@ _check_test_fs()
> 9p)
> # no way to check consistency for 9p
> ;;
> + virtiofs)
> + # no way to check consistency for virtiofs
> + ;;
> ceph)
> # no way to check consistency for CephFS
> ;;
> @@ -2744,6 +2753,9 @@ _check_scratch_fs()
> 9p)
> # no way to check consistency for 9p
> ;;
> + virtiofs)
> + # no way to check consistency for virtiofs
> + ;;
> ceph)
> # no way to check consistency for CephFS
> ;;
> --
> 2.20.1
>
> _______________________________________________
> Virtio-fs mailing list
> Virtio-fs@redhat.com
> https://www.redhat.com/mailman/listinfo/virtio-fs
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-10-18 9:15 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-15 8:32 [PATCH] fstests: Add virtio-fs shared file system support QI Fuli
2019-10-18 9:15 ` [Virtio-fs] " Eryu Guan
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).