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: Sun, 16 Feb 2020 19:59:57 -0800 (PST) From: David Miller <davem@...emloft.net> To: ilias.apalodimas@...aro.org 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 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: 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, ^~~~~~~~~~~~~~~~~~
Powered by blists - more mailing lists