linux-kernel - Re: [PATCH v2] uio_hv_generic: Add a check for HV

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] [day] [month] [year] [list]

Message-ID: <Z34pCVpGzhu_u8Pu@liuwe-devbox-debian-v2>
Date: Wed, 8 Jan 2025 07:28:09 +0000
From: Wei Liu <wei.liu@...nel.org>
To: Michael Kelley <mhklinux@...look.com>
Cc: Naman Jain <namjain@...ux.microsoft.com>,
	"K . Y . Srinivasan" <kys@...rosoft.com>,
	Haiyang Zhang <haiyangz@...rosoft.com>,
	Wei Liu <wei.liu@...nel.org>, Dexuan Cui <decui@...rosoft.com>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	"linux-hyperv@...r.kernel.org" <linux-hyperv@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Saurabh Sengar <ssengar@...ux.microsoft.com>
Subject: Re: [PATCH v2] uio_hv_generic: Add a check for HV_NIC for send,
 receive buffers setup

On Wed, Jan 08, 2025 at 05:26:58AM +0000, Michael Kelley wrote:
> From: Naman Jain <namjain@...ux.microsoft.com> Sent: Thursday, January 2, 2025 6:53 AM
> > 
> > Receive and send buffer allocation was originally introduced to support
> > DPDK's networking use case. These buffer sizes were further increased to
> > meet DPDK performance requirements. However, these large buffers are
> > unnecessary for any other UIO use cases.
> > Restrict the allocation of receive and send buffers only for HV_NIC device
> > type, saving 47 MB of memory per device.
> > 
> > While at it, fix some of the syntax related issues in the touched code
> > which are reported by "--strict" option of checkpatch.
> > 
> > Signed-off-by: Naman Jain <namjain@...ux.microsoft.com>
[...]
> 
> Reviewed-by: Michael Kelley <mhklinux@...look.com>

Applied to hyperv-next. Thanks.