[PATCH 12/15] Documentation: Reserve ioctl number for mshv driver

Nuno Das Neves posted 15 patches 2 years, 1 month ago
There is a newer version of this series
[PATCH 12/15] Documentation: Reserve ioctl number for mshv driver
Posted by Nuno Das Neves 2 years, 1 month ago
Signed-off-by: Nuno Das Neves <nunodasneves@linux.microsoft.com>
---
 Documentation/userspace-api/ioctl/ioctl-number.rst | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/userspace-api/ioctl/ioctl-number.rst b/Documentation/userspace-api/ioctl/ioctl-number.rst
index 0a1882e296ae..ca6b82419118 100644
--- a/Documentation/userspace-api/ioctl/ioctl-number.rst
+++ b/Documentation/userspace-api/ioctl/ioctl-number.rst
@@ -355,6 +355,8 @@ Code  Seq#    Include File                                           Comments
 0xB6  all    linux/fpga-dfl.h
 0xB7  all    uapi/linux/remoteproc_cdev.h                            <mailto:linux-remoteproc@vger.kernel.org>
 0xB7  all    uapi/linux/nsfs.h                                       <mailto:Andrei Vagin <avagin@openvz.org>>
+0xB8  all    uapi/linux/mshv.h                                       Microsoft Hypervisor VM management APIs
+                                                                     <mailto:linux-hyperv@vger.kernel.org>
 0xC0  00-0F  linux/usb/iowarrior.h
 0xCA  00-0F  uapi/misc/cxl.h
 0xCA  10-2F  uapi/misc/ocxl.h
-- 
2.25.1
Re: [PATCH 12/15] Documentation: Reserve ioctl number for mshv driver
Posted by Wei Liu 2 years, 1 month ago
This needs an ack from Jonathan.

On Thu, Jul 27, 2023 at 12:54:47PM -0700, Nuno Das Neves wrote:
> Signed-off-by: Nuno Das Neves <nunodasneves@linux.microsoft.com>
> ---
>  Documentation/userspace-api/ioctl/ioctl-number.rst | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/userspace-api/ioctl/ioctl-number.rst b/Documentation/userspace-api/ioctl/ioctl-number.rst
> index 0a1882e296ae..ca6b82419118 100644
> --- a/Documentation/userspace-api/ioctl/ioctl-number.rst
> +++ b/Documentation/userspace-api/ioctl/ioctl-number.rst
> @@ -355,6 +355,8 @@ Code  Seq#    Include File                                           Comments
>  0xB6  all    linux/fpga-dfl.h
>  0xB7  all    uapi/linux/remoteproc_cdev.h                            <mailto:linux-remoteproc@vger.kernel.org>
>  0xB7  all    uapi/linux/nsfs.h                                       <mailto:Andrei Vagin <avagin@openvz.org>>
> +0xB8  all    uapi/linux/mshv.h                                       Microsoft Hypervisor VM management APIs
> +                                                                     <mailto:linux-hyperv@vger.kernel.org>
>  0xC0  00-0F  linux/usb/iowarrior.h
>  0xCA  00-0F  uapi/misc/cxl.h
>  0xCA  10-2F  uapi/misc/ocxl.h
> -- 
> 2.25.1
>
Re: [PATCH 12/15] Documentation: Reserve ioctl number for mshv driver
Posted by Jonathan Corbet 2 years, 1 month ago
Wei Liu <wei.liu@kernel.org> writes:

> This needs an ack from Jonathan.

From me?  If every docs change needed an ack from me we'd be in rather
worse shape than we are now.  You can certainly have one:

Acked-by: Jonathan Corbet <corbet@lwn.net>

...but I don't control ioctl() numbers and don't need to gate-keep a
change like this.

Thanks,

jon

> On Thu, Jul 27, 2023 at 12:54:47PM -0700, Nuno Das Neves wrote:
>> Signed-off-by: Nuno Das Neves <nunodasneves@linux.microsoft.com>
>> ---
>>  Documentation/userspace-api/ioctl/ioctl-number.rst | 2 ++
>>  1 file changed, 2 insertions(+)
>> 
>> diff --git a/Documentation/userspace-api/ioctl/ioctl-number.rst b/Documentation/userspace-api/ioctl/ioctl-number.rst
>> index 0a1882e296ae..ca6b82419118 100644
>> --- a/Documentation/userspace-api/ioctl/ioctl-number.rst
>> +++ b/Documentation/userspace-api/ioctl/ioctl-number.rst
>> @@ -355,6 +355,8 @@ Code  Seq#    Include File                                           Comments
>>  0xB6  all    linux/fpga-dfl.h
>>  0xB7  all    uapi/linux/remoteproc_cdev.h                            <mailto:linux-remoteproc@vger.kernel.org>
>>  0xB7  all    uapi/linux/nsfs.h                                       <mailto:Andrei Vagin <avagin@openvz.org>>
>> +0xB8  all    uapi/linux/mshv.h                                       Microsoft Hypervisor VM management APIs
>> +                                                                     <mailto:linux-hyperv@vger.kernel.org>
>>  0xC0  00-0F  linux/usb/iowarrior.h
>>  0xCA  00-0F  uapi/misc/cxl.h
>>  0xCA  10-2F  uapi/misc/ocxl.h
>> -- 
>> 2.25.1
>>
Re: [PATCH 12/15] Documentation: Reserve ioctl number for mshv driver
Posted by Wei Liu 2 years, 1 month ago
On Thu, Aug 03, 2023 at 07:23:31AM -0600, Jonathan Corbet wrote:
> Wei Liu <wei.liu@kernel.org> writes:
> 
> > This needs an ack from Jonathan.
> 
> From me?  If every docs change needed an ack from me we'd be in rather
> worse shape than we are now.  You can certainly have one:
> 
> Acked-by: Jonathan Corbet <corbet@lwn.net>
> 
> ...but I don't control ioctl() numbers and don't need to gate-keep a
> change like this.

Thanks Jonathan.

MAINTAINERS says you're the maintainer for Documentation, and I couldn't
find who owns userspace-api or who controls the allocation of ioctl
numbers.

Thanks,
Wei.

> 
> Thanks,
> 
> jon
> 
> > On Thu, Jul 27, 2023 at 12:54:47PM -0700, Nuno Das Neves wrote:
> >> Signed-off-by: Nuno Das Neves <nunodasneves@linux.microsoft.com>
> >> ---
> >>  Documentation/userspace-api/ioctl/ioctl-number.rst | 2 ++
> >>  1 file changed, 2 insertions(+)
> >> 
> >> diff --git a/Documentation/userspace-api/ioctl/ioctl-number.rst b/Documentation/userspace-api/ioctl/ioctl-number.rst
> >> index 0a1882e296ae..ca6b82419118 100644
> >> --- a/Documentation/userspace-api/ioctl/ioctl-number.rst
> >> +++ b/Documentation/userspace-api/ioctl/ioctl-number.rst
> >> @@ -355,6 +355,8 @@ Code  Seq#    Include File                                           Comments
> >>  0xB6  all    linux/fpga-dfl.h
> >>  0xB7  all    uapi/linux/remoteproc_cdev.h                            <mailto:linux-remoteproc@vger.kernel.org>
> >>  0xB7  all    uapi/linux/nsfs.h                                       <mailto:Andrei Vagin <avagin@openvz.org>>
> >> +0xB8  all    uapi/linux/mshv.h                                       Microsoft Hypervisor VM management APIs
> >> +                                                                     <mailto:linux-hyperv@vger.kernel.org>
> >>  0xC0  00-0F  linux/usb/iowarrior.h
> >>  0xCA  00-0F  uapi/misc/cxl.h
> >>  0xCA  10-2F  uapi/misc/ocxl.h
> >> -- 
> >> 2.25.1
> >>