[Qemu-devel] [PATCH v5 1/1] 9pfs: local: Add support for custom fmode/dmode in 9ps mapped security modes

2017-06-19 Thread Tobias Schramm
ff-by: Tobias Schramm --- v5: Eliminate expandable variables, check mandatory commandline options first v4: Use OPT_NUMBER for file mode arguments, fix back to front naming, fix resource leak and add sanity checking for fmode/dmode arguments v3: Use unsigned types for umask v2: Adjust

[Qemu-devel] [PATCH v5 0/1] 9pfs: local: Add support for custom fmode/dmode in 9ps mapped security modes

2017-06-19 Thread Tobias Schramm
good idea. Considering that it is used in quite a few places might it be a good idea to write a macro for that check? Also I have added a much more descriptive commit message. Again, thank you for the great help (and patience :)). Regards, Tobias Schramm Tobias Schramm (1): 9pfs: local: Add s

[Qemu-devel] [PATCH v4 1/1] 9pfs: local: Add support for custom fmode/dmode in 9ps mapped security modes

2017-06-19 Thread Tobias Schramm
Signed-off-by: Tobias Schramm --- v4: Use OPT_NUMBER for file mode arguments, fix back to front naming, fix resource leak and add sanity checking for fmode/dmode arguments v3: Use unsigned types for umask v2: Adjust patch to QEMU code style fsdev/file-op-9p.h | 4 fsdev/qemu

[Qemu-devel] [PATCH v4 0/1] 9pfs: local: Add support for custom fmode/dmode in 9ps mapped security modes

2017-06-19 Thread Tobias Schramm
dup if an error during parsing of fmode/dmode occurred. Thanks to Greg Kurz for pointing out most of the above issues to me. Tobias Schramm Tobias Schramm (1): Add support for custom fmode/dmode in 9ps mapped security modes fsdev/file-op-9p.h | 4 fsdev/qemu-fsdev-opts.c | 12 ++

[Qemu-devel] [PATCH v3 0/1] Add fmask/dmask option for 9p mapped mode

2017-06-18 Thread Tobias Schramm
Hi, as per Manos suggestion I'm now parsing the fmask/dmask as unsigned rather than signed longs. Tobias Schramm Tobias Schramm (1): Add support for custom fmasks/dmasks in 9ps mapped mode fsdev/file-op-9p.h | 4 fsdev/qemu-fsdev-opts.c | 12 hw/9pfs/9p-lo

[Qemu-devel] [PATCH v3 1/1] Add support for custom fmasks/dmasks in 9ps mapped mode

2017-06-18 Thread Tobias Schramm
Signed-off-by: Tobias Schramm --- v3: Use unsigned types for umask v2: Adjust patch to QEMU code style fsdev/file-op-9p.h | 4 fsdev/qemu-fsdev-opts.c | 12 hw/9pfs/9p-local.c | 29 + hw/9pfs/9p.c| 3 +++ 4 files changed

[Qemu-devel] [PATCH v2 0/1] Add fmask/dmask option for 9p mapped mode

2017-06-18 Thread Tobias Schramm
Hi, as suggested by the automated tests I adjusted the code style to be compatible with QEMUs existing code base. The previous failed test seems to be irrelevant since it passes fine on my dev machine and the failure is in a totally unrelated component. Tobias Schramm Tobias Schramm (1): Add

[Qemu-devel] [PATCH v2 1/1] Add support for custom fmasks/dmasks in 9ps mapped mode

2017-06-18 Thread Tobias Schramm
Signed-off-by: Tobias Schramm --- v2: Adjust patch to QEMU code style fsdev/file-op-9p.h | 4 fsdev/qemu-fsdev-opts.c | 12 hw/9pfs/9p-local.c | 29 + hw/9pfs/9p.c| 3 +++ 4 files changed, 44 insertions(+), 4 deletions

[Qemu-devel] [PATCH 1/1] Add support for custom fmasks/dmasks in 9ps mapped mode

2017-06-18 Thread Tobias Schramm
Signed-off-by: Tobias Schramm --- fsdev/file-op-9p.h | 4 fsdev/qemu-fsdev-opts.c | 12 hw/9pfs/9p-local.c | 33 + hw/9pfs/9p.c| 3 +++ 4 files changed, 48 insertions(+), 4 deletions(-) diff --git a/fsdev/file-op-9p.h b

[Qemu-devel] [PATCH 0/1] Add fmask/dmask option for 9p mapped mode

2017-06-18 Thread Tobias Schramm
ctual permissions of newly created files and folders on the host. This is my first patch for QEMU. Please let me know if there are any issues with it. Tobias Schramm Tobias Schramm (1): Add support for custom fmasks/dmasks in 9ps mapped mode fsdev/file-op-9p.h | 4 fsdev/qemu-