public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] VMCI: fix reference to ioctl-number.rst
@ 2024-11-02 10:30 Alyssa Ross
  0 siblings, 0 replies; 3+ messages in thread
From: Alyssa Ross @ 2024-11-02 10:30 UTC (permalink / raw)
  To: Bryan Tan, Vishnu Dasa
  Cc: Broadcom internal kernel review list, Greg Kroah-Hartman,
	linux-kernel

There has never been an ioctl-number.h — this must have been a typo
for ioctl-number.txt (which later become ioctl-number.rst).

At the time this comment was written, the note didn't actually end up
appearing anywhere, but I fixed the omission from ioctl-number.rst in
0a8e4dc1d353 ("Documentation: ioctl: document 0x07 ioctl code").

Fixes: 20259849bb1a ("VMCI: Some header and config files.")
Signed-off-by: Alyssa Ross <hi@alyssa.is>
---
v2: added patch description.
v1: https://lore.kernel.org/lkml/20240731111302.3072858-2-hi@alyssa.is/

 include/linux/vmw_vmci_defs.h | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/include/linux/vmw_vmci_defs.h b/include/linux/vmw_vmci_defs.h
index 6fb663b36f72..244c5ff412e1 100644
--- a/include/linux/vmw_vmci_defs.h
+++ b/include/linux/vmw_vmci_defs.h
@@ -431,11 +431,11 @@ enum {
 	((((_p)[0] & 0xFF) << 24) | (((_p)[1] & 0xFF) << 16) | ((_p)[2]))
 
 /*
- * The VMCI IOCTLs.  We use identity code 7, as noted in ioctl-number.h, and
- * we start at sequence 9f.  This gives us the same values that our shipping
- * products use, starting at 1951, provided we leave out the direction and
- * structure size.  Note that VMMon occupies the block following us, starting
- * at 2001.
+ * The VMCI IOCTLs.  We use identity code 7, as noted in ioctl-number.rst,
+ * and we start at sequence 9f.  This gives us the same values that our
+ * shipping products use, starting at 1951, provided we leave out the
+ * direction and structure size.  Note that VMMon occupies the block
+ * following us, starting at 2001.
  */
 #define IOCTL_VMCI_VERSION			_IO(7, 0x9f)	/* 1951 */
 #define IOCTL_VMCI_INIT_CONTEXT			_IO(7, 0xa0)

base-commit: 11066801dd4b7c4d75fce65c812723a80c1481ae
-- 
2.46.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH v2] VMCI: fix reference to ioctl-number.rst
@ 2025-01-09 19:38 Alyssa Ross
  2025-01-10 15:12 ` Greg Kroah-Hartman
  0 siblings, 1 reply; 3+ messages in thread
From: Alyssa Ross @ 2025-01-09 19:38 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: Broadcom internal kernel review list, Bryan Tan, Vishnu Dasa,
	linux-kernel

There has never been an ioctl-number.h — this must have been a typo
for ioctl-number.txt (which later become ioctl-number.rst).

At the time this comment was written, the note didn't actually end up
appearing anywhere, but I fixed the omission from ioctl-number.rst in
0a8e4dc1d353 ("Documentation: ioctl: document 0x07 ioctl code").

Fixes: 20259849bb1a ("VMCI: Some header and config files.")
Signed-off-by: Alyssa Ross <hi@alyssa.is>
---
Greg, would you mind picking this one up too, since it seems there's no
active VMCI tree at the moment?

v2: added patch description.
    https://lore.kernel.org/lkml/20241102103052.7425-2-hi@alyssa.is/
v1: https://lore.kernel.org/lkml/20240731111302.3072858-2-hi@alyssa.is/

 include/linux/vmw_vmci_defs.h | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/include/linux/vmw_vmci_defs.h b/include/linux/vmw_vmci_defs.h
index 6fb663b36f72..244c5ff412e1 100644
--- a/include/linux/vmw_vmci_defs.h
+++ b/include/linux/vmw_vmci_defs.h
@@ -431,11 +431,11 @@ enum {
 	((((_p)[0] & 0xFF) << 24) | (((_p)[1] & 0xFF) << 16) | ((_p)[2]))

 /*
- * The VMCI IOCTLs.  We use identity code 7, as noted in ioctl-number.h, and
- * we start at sequence 9f.  This gives us the same values that our shipping
- * products use, starting at 1951, provided we leave out the direction and
- * structure size.  Note that VMMon occupies the block following us, starting
- * at 2001.
+ * The VMCI IOCTLs.  We use identity code 7, as noted in ioctl-number.rst,
+ * and we start at sequence 9f.  This gives us the same values that our
+ * shipping products use, starting at 1951, provided we leave out the
+ * direction and structure size.  Note that VMMon occupies the block
+ * following us, starting at 2001.
  */
 #define IOCTL_VMCI_VERSION			_IO(7, 0x9f)	/* 1951 */
 #define IOCTL_VMCI_INIT_CONTEXT			_IO(7, 0xa0)

base-commit: 11066801dd4b7c4d75fce65c812723a80c1481ae
--
2.46.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] VMCI: fix reference to ioctl-number.rst
  2025-01-09 19:38 [PATCH v2] VMCI: fix reference to ioctl-number.rst Alyssa Ross
@ 2025-01-10 15:12 ` Greg Kroah-Hartman
  0 siblings, 0 replies; 3+ messages in thread
From: Greg Kroah-Hartman @ 2025-01-10 15:12 UTC (permalink / raw)
  To: Alyssa Ross
  Cc: Broadcom internal kernel review list, Bryan Tan, Vishnu Dasa,
	linux-kernel

On Thu, Jan 09, 2025 at 08:38:07PM +0100, Alyssa Ross wrote:
> There has never been an ioctl-number.h — this must have been a typo
> for ioctl-number.txt (which later become ioctl-number.rst).
> 
> At the time this comment was written, the note didn't actually end up
> appearing anywhere, but I fixed the omission from ioctl-number.rst in
> 0a8e4dc1d353 ("Documentation: ioctl: document 0x07 ioctl code").
> 
> Fixes: 20259849bb1a ("VMCI: Some header and config files.")
> Signed-off-by: Alyssa Ross <hi@alyssa.is>
> ---
> Greg, would you mind picking this one up too, since it seems there's no
> active VMCI tree at the moment?

No problem, now taken.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-01-10 15:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-01-09 19:38 [PATCH v2] VMCI: fix reference to ioctl-number.rst Alyssa Ross
2025-01-10 15:12 ` Greg Kroah-Hartman
  -- strict thread matches above, loose matches on Subject: below --
2024-11-02 10:30 Alyssa Ross

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox