All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Byungchul Park <byungchul@sk.com>
Cc: oe-kbuild-all@lists.linux.dev
Subject: Re: [RFC 19/19] mm, netmem: remove the page pool members in struct page
Date: Sat, 10 May 2025 09:56:05 +0800	[thread overview]
Message-ID: <202505100932.uzAMBW1y-lkp@intel.com> (raw)
In-Reply-To: <20250509115126.63190-20-byungchul@sk.com>

Hi Byungchul,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build errors:

[auto build test ERROR on fd93b3350b4314eebd8fbf0fea3ca7fe48d777e3]

url:    https://github.com/intel-lab-lkp/linux/commits/Byungchul-Park/netmem-rename-struct-net_iov-to-struct-netmem_desc/20250509-195458
base:   fd93b3350b4314eebd8fbf0fea3ca7fe48d777e3
patch link:    https://lore.kernel.org/r/20250509115126.63190-20-byungchul%40sk.com
patch subject: [RFC 19/19] mm, netmem: remove the page pool members in struct page
config: x86_64-buildonly-randconfig-003-20250510 (https://download.01.org/0day-ci/archive/20250510/202505100932.uzAMBW1y-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250510/202505100932.uzAMBW1y-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202505100932.uzAMBW1y-lkp@intel.com/

All errors (new ones prefixed by >>):

   drivers/net/netdevsim/netdev.c: In function 'nsim_pp_hold_write':
>> drivers/net/netdevsim/netdev.c:852:49: error: 'struct page' has no member named 'pp'
     852 |                 page_pool_put_full_page(ns->page->pp, ns->page, false);
         |                                                 ^~
   drivers/net/netdevsim/netdev.c: In function 'nsim_destroy':
   drivers/net/netdevsim/netdev.c:1077:49: error: 'struct page' has no member named 'pp'
    1077 |                 page_pool_put_full_page(ns->page->pp, ns->page, false);
         |                                                 ^~


vim +852 drivers/net/netdevsim/netdev.c

1580cbcbfe770b Jakub Kicinski 2024-04-12  827  
1580cbcbfe770b Jakub Kicinski 2024-04-12  828  static ssize_t
1580cbcbfe770b Jakub Kicinski 2024-04-12  829  nsim_pp_hold_write(struct file *file, const char __user *data,
1580cbcbfe770b Jakub Kicinski 2024-04-12  830  		   size_t count, loff_t *ppos)
1580cbcbfe770b Jakub Kicinski 2024-04-12  831  {
1580cbcbfe770b Jakub Kicinski 2024-04-12  832  	struct netdevsim *ns = file->private_data;
1580cbcbfe770b Jakub Kicinski 2024-04-12  833  	ssize_t ret;
1580cbcbfe770b Jakub Kicinski 2024-04-12  834  	bool val;
1580cbcbfe770b Jakub Kicinski 2024-04-12  835  
1580cbcbfe770b Jakub Kicinski 2024-04-12  836  	ret = kstrtobool_from_user(data, count, &val);
1580cbcbfe770b Jakub Kicinski 2024-04-12  837  	if (ret)
1580cbcbfe770b Jakub Kicinski 2024-04-12  838  		return ret;
1580cbcbfe770b Jakub Kicinski 2024-04-12  839  
1580cbcbfe770b Jakub Kicinski 2024-04-12  840  	rtnl_lock();
1580cbcbfe770b Jakub Kicinski 2024-04-12  841  	ret = count;
1580cbcbfe770b Jakub Kicinski 2024-04-12  842  	if (val == !!ns->page)
1580cbcbfe770b Jakub Kicinski 2024-04-12  843  		goto exit;
1580cbcbfe770b Jakub Kicinski 2024-04-12  844  
1580cbcbfe770b Jakub Kicinski 2024-04-12  845  	if (!netif_running(ns->netdev) && val) {
1580cbcbfe770b Jakub Kicinski 2024-04-12  846  		ret = -ENETDOWN;
1580cbcbfe770b Jakub Kicinski 2024-04-12  847  	} else if (val) {
915c82f842f955 Jakub Kicinski 2025-01-07  848  		ns->page = page_pool_dev_alloc_pages(ns->rq[0]->page_pool);
1580cbcbfe770b Jakub Kicinski 2024-04-12  849  		if (!ns->page)
1580cbcbfe770b Jakub Kicinski 2024-04-12  850  			ret = -ENOMEM;
1580cbcbfe770b Jakub Kicinski 2024-04-12  851  	} else {
1580cbcbfe770b Jakub Kicinski 2024-04-12 @852  		page_pool_put_full_page(ns->page->pp, ns->page, false);
1580cbcbfe770b Jakub Kicinski 2024-04-12  853  		ns->page = NULL;
1580cbcbfe770b Jakub Kicinski 2024-04-12  854  	}
1580cbcbfe770b Jakub Kicinski 2024-04-12  855  
1580cbcbfe770b Jakub Kicinski 2024-04-12  856  exit:
b9b8301d369b4c Eric Dumazet   2024-12-16  857  	rtnl_unlock();
b9b8301d369b4c Eric Dumazet   2024-12-16  858  	return ret;
1580cbcbfe770b Jakub Kicinski 2024-04-12  859  }
1580cbcbfe770b Jakub Kicinski 2024-04-12  860  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

  parent reply	other threads:[~2025-05-10  1:56 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-09 11:51 [RFC 00/19] Split netmem from struct page Byungchul Park
2025-05-09 11:51 ` [RFC 01/19] netmem: rename struct net_iov to struct netmem_desc Byungchul Park
2025-05-12 13:11   ` Pavel Begunkov
2025-05-12 13:29     ` Byungchul Park
2025-05-12 19:14       ` Mina Almasry
2025-05-13  2:00         ` Byungchul Park
2025-05-13 12:58           ` Pavel Begunkov
2025-05-13 12:49       ` Pavel Begunkov
2025-05-14  0:07         ` Byungchul Park
2025-05-09 11:51 ` [RFC 02/19] netmem: introduce netmem alloc/put API to wrap page alloc/put API Byungchul Park
2025-05-09 13:39   ` Mina Almasry
2025-05-09 14:08     ` Mina Almasry
2025-05-12 12:30       ` Byungchul Park
2025-05-09 11:51 ` [RFC 03/19] page_pool: use netmem alloc/put API in __page_pool_alloc_page_order() Byungchul Park
2025-05-09 11:51 ` [RFC 04/19] page_pool: rename __page_pool_alloc_page_order() to __page_pool_alloc_large_netmem() Byungchul Park
2025-05-09 11:51 ` [RFC 05/19] page_pool: use netmem alloc/put API in __page_pool_alloc_pages_slow() Byungchul Park
2025-05-09 11:51 ` [RFC 06/19] page_pool: rename page_pool_return_page() to page_pool_return_netmem() Byungchul Park
2025-05-09 11:51 ` [RFC 07/19] page_pool: use netmem alloc/put API in page_pool_return_netmem() Byungchul Park
2025-05-09 11:51 ` [RFC 08/19] page_pool: rename __page_pool_release_page_dma() to __page_pool_release_netmem_dma() Byungchul Park
2025-05-09 11:51 ` [RFC 09/19] page_pool: rename __page_pool_put_page() to __page_pool_put_netmem() Byungchul Park
2025-05-09 11:51 ` [RFC 10/19] page_pool: rename __page_pool_alloc_pages_slow() to __page_pool_alloc_netmems_slow() Byungchul Park
2025-05-09 11:51 ` [RFC 11/19] mlx4: use netmem descriptor and API for page pool Byungchul Park
2025-05-09 11:51 ` [RFC 12/19] netmem: introduce page_pool_recycle_direct_netmem() Byungchul Park
2025-05-09 11:51 ` [RFC 13/19] page_pool: expand scope of is_pp_{netmem,page}() to global Byungchul Park
2025-05-10  3:29   ` kernel test robot
2025-05-12 12:46   ` Toke Høiland-Jørgensen
2025-05-12 12:55     ` Byungchul Park
2025-05-14  3:00     ` Byungchul Park
2025-05-14 11:17       ` Toke Høiland-Jørgensen
2025-05-09 11:51 ` [RFC 14/19] mm: page_alloc: do not directly access page->pp_magic but use is_pp_page() Byungchul Park
2025-05-09 11:51 ` [RFC 15/19] mlx5: use netmem descriptor and API for page pool Byungchul Park
2025-05-09 11:51 ` [RFC 16/19] netmem: use _Generic to cover const casting for page_to_netmem() Byungchul Park
2025-05-09 11:51 ` [RFC 17/19] netmem: remove __netmem_get_pp() Byungchul Park
2025-05-09 13:47   ` Mina Almasry
2025-05-09 11:51 ` [RFC 18/19] page_pool: make page_pool_get_dma_addr() just wrap page_pool_get_dma_addr_netmem() Byungchul Park
2025-05-09 13:49   ` Mina Almasry
2025-05-10  7:28   ` Ilias Apalodimas
2025-05-09 11:51 ` [RFC 19/19] mm, netmem: remove the page pool members in struct page Byungchul Park
2025-05-09 17:32   ` Mina Almasry
2025-05-09 18:11     ` Matthew Wilcox
2025-05-09 19:04       ` Mina Almasry
2025-05-09 19:48         ` Matthew Wilcox
2025-05-12 19:10           ` Mina Almasry
2025-05-09 18:02   ` Matthew Wilcox
2025-05-12 12:51     ` Byungchul Park
2025-05-12 14:42       ` Matthew Wilcox
2025-05-13  1:42         ` Byungchul Park
2025-05-13  3:19           ` Matthew Wilcox
2025-05-13 10:24             ` Byungchul Park
2025-05-10  1:56   ` kernel test robot [this message]
2025-05-10  7:26   ` Ilias Apalodimas
2025-05-12 12:58     ` Byungchul Park
2025-05-09 14:09 ` [RFC 00/19] Split netmem from " Mina Almasry
2025-05-12 12:36   ` Byungchul Park
2025-05-12 12:59     ` Pavel Begunkov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202505100932.uzAMBW1y-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=byungchul@sk.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.