[PATCH] xen/include/public: fix 9pfs xenstore path description

Juergen Gross posted 1 patch 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20230530114815.18362-1-jgross@suse.com
xen/include/public/io/9pfs.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] xen/include/public: fix 9pfs xenstore path description
Posted by Juergen Gross 10 months ago
In xen/include/public/io/9pfs.h the name of the Xenstore backend node
"security-model" should be "security_model", as this is how the Xen
tools are creating it and qemu is reading it.

Fixes: ad58142e73a9 ("xen/public: move xenstore related doc into 9pfs.h")
Fixes: cf1d2d22fdfd ("docs/misc: Xen transport for 9pfs")
Signed-off-by: Juergen Gross <jgross@suse.com>
---
 xen/include/public/io/9pfs.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/include/public/io/9pfs.h b/xen/include/public/io/9pfs.h
index a0ce82d0a8..9ad2773082 100644
--- a/xen/include/public/io/9pfs.h
+++ b/xen/include/public/io/9pfs.h
@@ -64,7 +64,7 @@
  *
  *         Host filesystem path to share.
  *
- *    security-model
+ *    security_model
  *         Values:         "none"
  *
  *         *none*: files are stored using the same credentials as they are
-- 
2.35.3
Re: [PATCH] xen/include/public: fix 9pfs xenstore path description
Posted by Stefano Stabellini 10 months ago
On Tue, 30 May 2023, Juergen Gross wrote:
> In xen/include/public/io/9pfs.h the name of the Xenstore backend node
> "security-model" should be "security_model", as this is how the Xen
> tools are creating it and qemu is reading it.
> 
> Fixes: ad58142e73a9 ("xen/public: move xenstore related doc into 9pfs.h")
> Fixes: cf1d2d22fdfd ("docs/misc: Xen transport for 9pfs")
> Signed-off-by: Juergen Gross <jgross@suse.com>

Acked-by: Stefano Stabellini <sstabellini@kernel.org>


> ---
>  xen/include/public/io/9pfs.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/xen/include/public/io/9pfs.h b/xen/include/public/io/9pfs.h
> index a0ce82d0a8..9ad2773082 100644
> --- a/xen/include/public/io/9pfs.h
> +++ b/xen/include/public/io/9pfs.h
> @@ -64,7 +64,7 @@
>   *
>   *         Host filesystem path to share.
>   *
> - *    security-model
> + *    security_model
>   *         Values:         "none"
>   *
>   *         *none*: files are stored using the same credentials as they are
> -- 
> 2.35.3
>
Re: [PATCH] xen/include/public: fix 9pfs xenstore path description
Posted by Jason Andryuk 10 months ago
On Tue, May 30, 2023 at 7:48 AM Juergen Gross <jgross@suse.com> wrote:
>
> In xen/include/public/io/9pfs.h the name of the Xenstore backend node
> "security-model" should be "security_model", as this is how the Xen
> tools are creating it and qemu is reading it.
>
> Fixes: ad58142e73a9 ("xen/public: move xenstore related doc into 9pfs.h")
> Fixes: cf1d2d22fdfd ("docs/misc: Xen transport for 9pfs")
> Signed-off-by: Juergen Gross <jgross@suse.com>

Reviewed-by: Jason Andryuk <jandryuk@gmail.com>

Regards,
Jason