linux-kernel - Re: [PATCH RESEND] VMCI: remove unused ioctl definitions

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAF+opq1=w5C8snwktDkOBxzsFPu+viG7Xj0dh-FqXhzCpKNvYA@mail.gmail.com>
Date: Thu, 19 Dec 2024 22:40:02 -0800
From: Vishnu Dasa <vishnu.dasa@...adcom.com>
To: Alyssa Ross <hi@...ssa.is>
Cc: Arnd Bergmann <arnd@...db.de>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, 
	"open list:VMWARE VMCI DRIVER" <linux-kernel@...r.kernel.org>, Bryan Tan <bryan-bt.tan@...adcom.com>, 
	Broadcom internal kernel review list <bcm-kernel-feedback-list@...adcom.com>
Subject: Re: [PATCH RESEND] VMCI: remove unused ioctl definitions

On Thu, Dec 19, 2024 at 3:30 PM Alyssa Ross <hi@...ssa.is> wrote:
>
> IOCTL_VMCI_SOCKETS_VERSION and IOCTL_VMCI_SOCKETS_GET_AF_VALUE were
> never implemented, because VSOCK ended up being implemented as a
> generic mechanism with a static AF value.  Likewise,
> IOCTL_VMCI_SOCKETS_GET_LOCAL_CID ended up being implemented as
> IOCTL_VM_SOCKETS_GET_LOCAL_CID.
>
> This isn't a UAPI header, so it should be fine to remove the unused
> values.  I've left a comment noting IOCTL_VM_SOCKETS_GET_LOCAL_CID is
> in the VMCI range to avoid unintentional reuse.
>
> Signed-off-by: Alyssa Ross <hi@...ssa.is>

Acked-by: Vishnu Dasa <vishnu.dasa@...adcom.com>

> ---
> Hi misc maintainers!  I'm sending this version of the patch to you
> because the VMWare VMCI maintainers don't seem to be around any more.
> The last message on lore from either of them was in May.  Since then,
> I've submitted this patch twice, and haven't heard anything in response.
>
>  include/linux/vmw_vmci_defs.h | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/include/linux/vmw_vmci_defs.h b/include/linux/vmw_vmci_defs.h
> index 6fb663b36f72..c2df94696593 100644
> --- a/include/linux/vmw_vmci_defs.h
> +++ b/include/linux/vmw_vmci_defs.h
> @@ -453,9 +453,7 @@ enum {
>  #define IOCTL_VMCI_CTX_GET_CPT_STATE           _IO(7, 0xb1)
>  #define IOCTL_VMCI_CTX_SET_CPT_STATE           _IO(7, 0xb2)
>  #define IOCTL_VMCI_GET_CONTEXT_ID              _IO(7, 0xb3)
> -#define IOCTL_VMCI_SOCKETS_VERSION             _IO(7, 0xb4)
> -#define IOCTL_VMCI_SOCKETS_GET_AF_VALUE                _IO(7, 0xb8)
> -#define IOCTL_VMCI_SOCKETS_GET_LOCAL_CID       _IO(7, 0xb9)
> +/*IOCTL_VM_SOCKETS_GET_LOCAL_CID               _IO(7, 0xb9)*/
>  #define IOCTL_VMCI_SET_NOTIFY                  _IO(7, 0xcb)    /* 1995 */
>  /*IOCTL_VMMON_START                            _IO(7, 0xd1)*/  /* 2001 */
>
>
> base-commit: c33ffdb70cc6df4105160f991288e7d2567d7ffa
> --
> 2.45.1
>

Download attachment "smime.p7s" of type "application/pkcs7-signature" (5430 bytes)