netdev - Re: [RFC PATCH v2 11/21] core/skbuff: add page recycling logic for netgpu pages

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20200728182628.GB328787@kroah.com>
Date:   Tue, 28 Jul 2020 20:26:28 +0200
From:   Greg KH <gregkh@...uxfoundation.org>
To:     Jonathan Lemon <jonathan.lemon@...il.com>
Cc:     netdev@...r.kernel.org, kernel-team@...com
Subject: Re: [RFC PATCH v2 11/21] core/skbuff: add page recycling logic for
 netgpu pages

On Tue, Jul 28, 2020 at 11:00:40AM -0700, Jonathan Lemon wrote:
> On Tue, Jul 28, 2020 at 06:28:25PM +0200, Greg KH wrote:
> > On Mon, Jul 27, 2020 at 03:44:34PM -0700, Jonathan Lemon wrote:
> > > From: Jonathan Lemon <bsd@...com>
> > > 
> > > netgpu pages will always have a refcount of at least one (held by
> > > the netgpu module).  If the skb is marked as containing netgpu ZC
> > > pages, recycle them back to netgpu.
> > 
> > What???
> 
> Yes, this is page refcount elevation.  ZONE_DEVICE pages do this also,
> which is hidden in put_devmap_managed_page().  I would really like to
> find a generic solution for this, as it has come up in other cases as
> well (page recycling for page_pool, for example).  Some way to say "this
> page is different", and a separate routine to release refcounts.
> 
> Can we have a discussion on this possibility?

Then propose a generic solution, not a "solution" like this.

greg k-h