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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 17 Feb 2020 07:57:36 +0200 From: Ilias Apalodimas <ilias.apalodimas@...aro.org> To: David Miller <davem@...emloft.net> Cc: netdev@...r.kernel.org, jonathan.lemon@...il.com, lorenzo@...nel.org, thomas.petazzoni@...tlin.com, jaswinder.singh@...aro.org, peppe.cavallaro@...com, alexandre.torgue@...com, joabreu@...opsys.com, mcoquelin.stm32@...il.com, hawk@...nel.org, kuba@...nel.org, ast@...nel.org, daniel@...earbox.net, john.fastabend@...il.com, linux-kernel@...r.kernel.org, linux-stm32@...md-mailman.stormreply.com, linux-arm-kernel@...ts.infradead.org, bpf@...r.kernel.org Subject: Re: [PATCH net-next] net: page_pool: API cleanup and comments On Sun, Feb 16, 2020 at 07:59:57PM -0800, David Miller wrote: > From: David Miller <davem@...emloft.net> > Date: Sun, 16 Feb 2020 19:53:00 -0800 (PST) > > > From: Ilias Apalodimas <ilias.apalodimas@...aro.org> > > Date: Sun, 16 Feb 2020 11:40:55 +0200 > > > >> Functions starting with __ usually indicate those which are exported, > >> but should not be called directly. Update some of those declared in the > >> API and make it more readable. > >> > >> page_pool_unmap_page() and page_pool_release_page() were doing > >> exactly the same thing. Keep the page_pool_release_page() variant > >> and export it in order to show up on perf logs. > >> Finally rename __page_pool_put_page() to page_pool_put_page() since we > >> can now directly call it from drivers and rename the existing > >> page_pool_put_page() to page_pool_put_full_page() since they do the same > >> thing but the latter is trying to sync the full DMA area. > >> > >> Also update netsec, mvneta and stmmac drivers which use those functions. > >> > >> Suggested-by: Jonathan Lemon <jonathan.lemon@...il.com> > >> Signed-off-by: Ilias Apalodimas <ilias.apalodimas@...aro.org> > > > > Applied to net-next, thanks. > > Actually this doesn't compile, please respin: Ooops :( I was compiling for armv7, stmmac and mvneta are included but netsec isn't. Sorry for the noise, i'll send a v2 > > drivers/net/ethernet/socionext/netsec.c: In function ‘netsec_uninit_pkt_dring’: > drivers/net/ethernet/socionext/netsec.c:1201:4: error: too few arguments to function ‘page_pool_put_page’ > page_pool_put_page(dring->page_pool, page, false); > ^~~~~~~~~~~~~~~~~~ > In file included from drivers/net/ethernet/socionext/netsec.c:17: > ./include/net/page_pool.h:172:6: note: declared here > void page_pool_put_page(struct page_pool *pool, struct page *page, > ^~~~~~~~~~~~~~~~~~ Thanks /Ilias
Powered by blists - more mailing lists