[PATCH 1/7] tools include UAPI: Sync linux/vhost.h with the kernel sources

Arnaldo Carvalho de Melo posted 7 patches 7 months ago
[PATCH 1/7] tools include UAPI: Sync linux/vhost.h with the kernel sources
Posted by Arnaldo Carvalho de Melo 7 months ago
From: Arnaldo Carvalho de Melo <acme@redhat.com>

To get the changes in:

  a940e0a685575424 ("vhost: fix VHOST_*_OWNER documentation")

That just changed lines in comments

This addresses this perf build warning:

  Warning: Kernel ABI header differences:
    diff -u tools/perf/trace/beauty/include/uapi/linux/vhost.h include/uapi/linux/vhost.h

Please see tools/include/uapi/README for further details.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Ian Rogers <irogers@google.com>
Cc: James Clark <james.clark@linaro.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Michael S. Tsirkin <mst@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Stefano Garzarella <sgarzare@redhat.com>
Link: https://lore.kernel.org/r/
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/trace/beauty/include/uapi/linux/vhost.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/perf/trace/beauty/include/uapi/linux/vhost.h b/tools/perf/trace/beauty/include/uapi/linux/vhost.h
index b95dd84eef2db231..d4b3e2ae1314d1fc 100644
--- a/tools/perf/trace/beauty/include/uapi/linux/vhost.h
+++ b/tools/perf/trace/beauty/include/uapi/linux/vhost.h
@@ -28,10 +28,10 @@
 
 /* Set current process as the (exclusive) owner of this file descriptor.  This
  * must be called before any other vhost command.  Further calls to
- * VHOST_OWNER_SET fail until VHOST_OWNER_RESET is called. */
+ * VHOST_SET_OWNER fail until VHOST_RESET_OWNER is called. */
 #define VHOST_SET_OWNER _IO(VHOST_VIRTIO, 0x01)
 /* Give up ownership, and reset the device to default values.
- * Allows subsequent call to VHOST_OWNER_SET to succeed. */
+ * Allows subsequent call to VHOST_SET_OWNER to succeed. */
 #define VHOST_RESET_OWNER _IO(VHOST_VIRTIO, 0x02)
 
 /* Set up/modify memory layout */
-- 
2.49.0
Re: [PATCH 1/7] tools include UAPI: Sync linux/vhost.h with the kernel sources
Posted by Michael S. Tsirkin 6 months, 4 weeks ago
On Mon, May 19, 2025 at 06:41:20PM -0300, Arnaldo Carvalho de Melo wrote:
> From: Arnaldo Carvalho de Melo <acme@redhat.com>
> 
> To get the changes in:
> 
>   a940e0a685575424 ("vhost: fix VHOST_*_OWNER documentation")
> 
> That just changed lines in comments
> 
> This addresses this perf build warning:
> 
>   Warning: Kernel ABI header differences:
>     diff -u tools/perf/trace/beauty/include/uapi/linux/vhost.h include/uapi/linux/vhost.h
> 
> Please see tools/include/uapi/README for further details.
> 
> Cc: Adrian Hunter <adrian.hunter@intel.com>
> Cc: Ian Rogers <irogers@google.com>
> Cc: James Clark <james.clark@linaro.org>
> Cc: Jiri Olsa <jolsa@kernel.org>
> Cc: Kan Liang <kan.liang@linux.intel.com>
> Cc: Michael S. Tsirkin <mst@redhat.com>
> Cc: Namhyung Kim <namhyung@kernel.org>
> Cc: Stefano Garzarella <sgarzare@redhat.com>
> Link: https://lore.kernel.org/r/
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

Acked-by: Michael S. Tsirkin <mst@redhat.com>

> ---
>  tools/perf/trace/beauty/include/uapi/linux/vhost.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/perf/trace/beauty/include/uapi/linux/vhost.h b/tools/perf/trace/beauty/include/uapi/linux/vhost.h
> index b95dd84eef2db231..d4b3e2ae1314d1fc 100644
> --- a/tools/perf/trace/beauty/include/uapi/linux/vhost.h
> +++ b/tools/perf/trace/beauty/include/uapi/linux/vhost.h
> @@ -28,10 +28,10 @@
>  
>  /* Set current process as the (exclusive) owner of this file descriptor.  This
>   * must be called before any other vhost command.  Further calls to
> - * VHOST_OWNER_SET fail until VHOST_OWNER_RESET is called. */
> + * VHOST_SET_OWNER fail until VHOST_RESET_OWNER is called. */
>  #define VHOST_SET_OWNER _IO(VHOST_VIRTIO, 0x01)
>  /* Give up ownership, and reset the device to default values.
> - * Allows subsequent call to VHOST_OWNER_SET to succeed. */
> + * Allows subsequent call to VHOST_SET_OWNER to succeed. */
>  #define VHOST_RESET_OWNER _IO(VHOST_VIRTIO, 0x02)
>  
>  /* Set up/modify memory layout */
> -- 
> 2.49.0
Re: [PATCH 1/7] tools include UAPI: Sync linux/vhost.h with the kernel sources
Posted by Stefano Garzarella 7 months ago
On Mon, May 19, 2025 at 06:41:20PM -0300, Arnaldo Carvalho de Melo wrote:
>From: Arnaldo Carvalho de Melo <acme@redhat.com>
>
>To get the changes in:
>
>  a940e0a685575424 ("vhost: fix VHOST_*_OWNER documentation")
>
>That just changed lines in comments
>
>This addresses this perf build warning:
>
>  Warning: Kernel ABI header differences:
>    diff -u tools/perf/trace/beauty/include/uapi/linux/vhost.h include/uapi/linux/vhost.h
>
>Please see tools/include/uapi/README for further details.
>
>Cc: Adrian Hunter <adrian.hunter@intel.com>
>Cc: Ian Rogers <irogers@google.com>
>Cc: James Clark <james.clark@linaro.org>
>Cc: Jiri Olsa <jolsa@kernel.org>
>Cc: Kan Liang <kan.liang@linux.intel.com>
>Cc: Michael S. Tsirkin <mst@redhat.com>
>Cc: Namhyung Kim <namhyung@kernel.org>
>Cc: Stefano Garzarella <sgarzare@redhat.com>
>Link: https://lore.kernel.org/r/
>Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
>---
> tools/perf/trace/beauty/include/uapi/linux/vhost.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)

Acked-by: Stefano Garzarella <sgarzare@redhat.com>

>
>diff --git a/tools/perf/trace/beauty/include/uapi/linux/vhost.h b/tools/perf/trace/beauty/include/uapi/linux/vhost.h
>index b95dd84eef2db231..d4b3e2ae1314d1fc 100644
>--- a/tools/perf/trace/beauty/include/uapi/linux/vhost.h
>+++ b/tools/perf/trace/beauty/include/uapi/linux/vhost.h
>@@ -28,10 +28,10 @@
>
> /* Set current process as the (exclusive) owner of this file descriptor.  This
>  * must be called before any other vhost command.  Further calls to
>- * VHOST_OWNER_SET fail until VHOST_OWNER_RESET is called. */
>+ * VHOST_SET_OWNER fail until VHOST_RESET_OWNER is called. */
> #define VHOST_SET_OWNER _IO(VHOST_VIRTIO, 0x01)
> /* Give up ownership, and reset the device to default values.
>- * Allows subsequent call to VHOST_OWNER_SET to succeed. */
>+ * Allows subsequent call to VHOST_SET_OWNER to succeed. */
> #define VHOST_RESET_OWNER _IO(VHOST_VIRTIO, 0x02)
>
> /* Set up/modify memory layout */
>-- 
>2.49.0
>
Re: [PATCH 1/7] tools include UAPI: Sync linux/vhost.h with the kernel sources
Posted by Arnaldo Carvalho de Melo 7 months ago
On Tue, May 20, 2025 at 09:20:04AM +0200, Stefano Garzarella wrote:
> On Mon, May 19, 2025 at 06:41:20PM -0300, Arnaldo Carvalho de Melo wrote:
> > From: Arnaldo Carvalho de Melo <acme@redhat.com

> > To get the changes in:
 
> >  a940e0a685575424 ("vhost: fix VHOST_*_OWNER documentation")
 
> Acked-by: Stefano Garzarella <sgarzare@redhat.com>

Thanks, added to the patch,

- Arnaldo