From: Suren Baghdasaryan <surenb@google.com>
To: kernel test robot <lkp@intel.com>
Cc: akpm@linux-foundation.org, llvm@lists.linux.dev,
oe-kbuild-all@lists.linux.dev, peterz@infradead.org,
willy@infradead.org, liam.howlett@oracle.com,
lorenzo.stoakes@oracle.com, mhocko@suse.com, vbabka@suse.cz,
hannes@cmpxchg.org, mjguzik@gmail.com, oliver.sang@intel.com,
mgorman@techsingularity.net, david@redhat.com,
peterx@redhat.com, oleg@redhat.com, dave@stgolabs.net,
paulmck@kernel.org, brauner@kernel.org, dhowells@redhat.com,
hdanton@sina.com, hughd@google.com, lokeshgidra@google.com,
minchan@google.com, jannh@google.com, shakeel.butt@linux.dev,
souravpanda@google.com, pasha.tatashin@soleen.com,
klarasmodin@gmail.com, corbet@lwn.net,
linux-doc@vger.kernel.org, linux-mm@kvack.org
Subject: Re: [PATCH v7 13/17] mm/debug: print vm_refcnt state when dumping the vma
Date: Thu, 26 Dec 2024 11:51:28 -0800 [thread overview]
Message-ID: <CAJuCfpHb23mRT6bS8LNd79Zm0XU9eq9hB671G21UnkU992j82Q@mail.gmail.com> (raw)
In-Reply-To: <202412270328.HiJX1mRt-lkp@intel.com>
On Thu, Dec 26, 2024 at 11:41 AM kernel test robot <lkp@intel.com> wrote:
>
> Hi Suren,
>
> kernel test robot noticed the following build errors:
>
> [auto build test ERROR on 431614f1580a03c1a653340c55ea76bd12a9403f]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Suren-Baghdasaryan/mm-introduce-vma_start_read_locked-_nested-helpers/20241227-010925
> base: 431614f1580a03c1a653340c55ea76bd12a9403f
> patch link: https://lore.kernel.org/r/20241226170710.1159679-14-surenb%40google.com
> patch subject: [PATCH v7 13/17] mm/debug: print vm_refcnt state when dumping the vma
> config: hexagon-randconfig-002-20241227 (https://download.01.org/0day-ci/archive/20241227/202412270328.HiJX1mRt-lkp@intel.com/config)
> compiler: clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241227/202412270328.HiJX1mRt-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/202412270328.HiJX1mRt-lkp@intel.com/
>
> All errors (new ones prefixed by >>):
>
> >> mm/debug.c:189:54: error: no member named 'vm_refcnt' in 'struct vm_area_struct'
Ugh, ok. On architectures with ARCH_SUPPORTS_PER_VMA_LOCK=n this will
happen. Preparing a fix.
> 189 | vma->vm_flags, &vma->vm_flags, refcount_read(&vma->vm_refcnt));
> | ~~~ ^
> include/linux/printk.h:514:35: note: expanded from macro 'pr_emerg'
> 514 | printk(KERN_EMERG pr_fmt(fmt), ##__VA_ARGS__)
> | ^~~~~~~~~~~
> include/linux/printk.h:501:60: note: expanded from macro 'printk'
> 501 | #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
> | ^~~~~~~~~~~
> include/linux/printk.h:473:19: note: expanded from macro 'printk_index_wrap'
> 473 | _p_func(_fmt, ##__VA_ARGS__); \
> | ^~~~~~~~~~~
> 1 error generated.
>
>
> vim +189 mm/debug.c
>
> 178
> 179 void dump_vma(const struct vm_area_struct *vma)
> 180 {
> 181 pr_emerg("vma %px start %px end %px mm %px\n"
> 182 "prot %lx anon_vma %px vm_ops %px\n"
> 183 "pgoff %lx file %px private_data %px\n"
> 184 "flags: %#lx(%pGv) refcnt %x\n",
> 185 vma, (void *)vma->vm_start, (void *)vma->vm_end, vma->vm_mm,
> 186 (unsigned long)pgprot_val(vma->vm_page_prot),
> 187 vma->anon_vma, vma->vm_ops, vma->vm_pgoff,
> 188 vma->vm_file, vma->vm_private_data,
> > 189 vma->vm_flags, &vma->vm_flags, refcount_read(&vma->vm_refcnt));
> 190 }
> 191 EXPORT_SYMBOL(dump_vma);
> 192
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2024-12-26 19:51 UTC|newest]
Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-26 17:06 [PATCH v7 00/17] move per-vma lock into vm_area_struct Suren Baghdasaryan
2024-12-26 17:06 ` [PATCH v7 01/17] mm: introduce vma_start_read_locked{_nested} helpers Suren Baghdasaryan
2025-01-08 14:59 ` Liam R. Howlett
2024-12-26 17:06 ` [PATCH v7 02/17] mm: move per-vma lock into vm_area_struct Suren Baghdasaryan
2025-01-08 14:59 ` Liam R. Howlett
2024-12-26 17:06 ` [PATCH v7 03/17] mm: mark vma as detached until it's added into vma tree Suren Baghdasaryan
2025-01-08 15:01 ` Liam R. Howlett
2024-12-26 17:06 ` [PATCH v7 04/17] mm: modify vma_iter_store{_gfp} to indicate if it's storing a new vma Suren Baghdasaryan
2025-01-07 16:48 ` Vlastimil Babka
2025-01-07 16:49 ` Liam R. Howlett
2025-01-07 17:12 ` Suren Baghdasaryan
2024-12-26 17:06 ` [PATCH v7 05/17] mm: mark vmas detached upon exit Suren Baghdasaryan
2025-01-07 17:08 ` Vlastimil Babka
2025-01-07 17:13 ` Suren Baghdasaryan
2024-12-26 17:06 ` [PATCH v7 06/17] mm/nommu: fix the last places where vma is not locked before being attached Suren Baghdasaryan
2025-01-07 17:51 ` Liam R. Howlett
2025-01-07 18:05 ` Suren Baghdasaryan
2024-12-26 17:06 ` [PATCH v7 07/17] types: move struct rcuwait into types.h Suren Baghdasaryan
2024-12-27 18:35 ` Davidlohr Bueso
2025-01-08 15:02 ` Liam R. Howlett
2024-12-26 17:07 ` [PATCH v7 08/17] mm: allow vma_start_read_locked/vma_start_read_locked_nested to fail Suren Baghdasaryan
2025-01-07 17:28 ` Vlastimil Babka
2025-01-07 17:31 ` Suren Baghdasaryan
2024-12-26 17:07 ` [PATCH v7 09/17] mm: move mmap_init_lock() out of the header file Suren Baghdasaryan
2025-01-07 17:30 ` Vlastimil Babka
2024-12-26 17:07 ` [PATCH v7 10/17] mm: uninline the main body of vma_start_write() Suren Baghdasaryan
2025-01-07 17:35 ` Vlastimil Babka
2025-01-07 17:45 ` Suren Baghdasaryan
2025-01-07 18:51 ` Suren Baghdasaryan
2025-04-08 4:39 ` Eric Naim
2025-04-08 6:01 ` Christoph Hellwig
2025-04-08 6:25 ` Lorenzo Stoakes
2025-04-08 7:52 ` Eric Naim
2025-04-08 17:09 ` Suren Baghdasaryan
2024-12-26 17:07 ` [PATCH v7 11/17] refcount: introduce __refcount_{add|inc}_not_zero_limited Suren Baghdasaryan
2025-01-08 9:16 ` Vlastimil Babka
2025-01-08 15:06 ` Matthew Wilcox
2025-01-08 15:45 ` Suren Baghdasaryan
2025-01-10 13:32 ` David Laight
2025-01-10 16:29 ` Suren Baghdasaryan
2024-12-26 17:07 ` [PATCH v7 12/17] mm: replace vm_lock and detached flag with a reference count Suren Baghdasaryan
2025-01-06 0:38 ` Wei Yang
2025-01-06 17:26 ` Suren Baghdasaryan
2025-01-07 18:44 ` Liam R. Howlett
2025-01-07 19:38 ` Suren Baghdasaryan
2025-01-08 11:52 ` Vlastimil Babka
2025-01-08 17:53 ` Suren Baghdasaryan
2024-12-26 17:07 ` [PATCH v7 13/17] mm/debug: print vm_refcnt state when dumping the vma Suren Baghdasaryan
2024-12-26 19:40 ` kernel test robot
2024-12-26 19:51 ` Suren Baghdasaryan [this message]
2024-12-26 19:54 ` Suren Baghdasaryan
2024-12-26 20:04 ` Suren Baghdasaryan
2024-12-26 20:13 ` kernel test robot
2024-12-26 17:07 ` [PATCH v7 14/17] mm: remove extra vma_numab_state_init() call Suren Baghdasaryan
2025-01-08 18:04 ` Vlastimil Babka
2024-12-26 17:07 ` [PATCH v7 15/17] mm: prepare lock_vma_under_rcu() for vma reuse possibility Suren Baghdasaryan
2025-01-08 18:05 ` Vlastimil Babka
2024-12-26 17:07 ` [PATCH v7 16/17] mm: make vma cache SLAB_TYPESAFE_BY_RCU Suren Baghdasaryan
2025-01-08 14:55 ` Liam R. Howlett
2025-01-08 18:21 ` Vlastimil Babka
2025-01-08 18:44 ` Suren Baghdasaryan
2025-01-08 19:00 ` Vlastimil Babka
2025-01-08 19:17 ` Suren Baghdasaryan
2024-12-26 17:07 ` [PATCH v7 17/17] docs/mm: document latest changes to vm_lock Suren Baghdasaryan
2025-01-08 15:46 ` Liam R. Howlett
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=CAJuCfpHb23mRT6bS8LNd79Zm0XU9eq9hB671G21UnkU992j82Q@mail.gmail.com \
--to=surenb@google.com \
--cc=akpm@linux-foundation.org \
--cc=brauner@kernel.org \
--cc=corbet@lwn.net \
--cc=dave@stgolabs.net \
--cc=david@redhat.com \
--cc=dhowells@redhat.com \
--cc=hannes@cmpxchg.org \
--cc=hdanton@sina.com \
--cc=hughd@google.com \
--cc=jannh@google.com \
--cc=klarasmodin@gmail.com \
--cc=liam.howlett@oracle.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=lkp@intel.com \
--cc=llvm@lists.linux.dev \
--cc=lokeshgidra@google.com \
--cc=lorenzo.stoakes@oracle.com \
--cc=mgorman@techsingularity.net \
--cc=mhocko@suse.com \
--cc=minchan@google.com \
--cc=mjguzik@gmail.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=oleg@redhat.com \
--cc=oliver.sang@intel.com \
--cc=pasha.tatashin@soleen.com \
--cc=paulmck@kernel.org \
--cc=peterx@redhat.com \
--cc=peterz@infradead.org \
--cc=shakeel.butt@linux.dev \
--cc=souravpanda@google.com \
--cc=vbabka@suse.cz \
--cc=willy@infradead.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox