Re: [PATCH 1/1] Docs/mm/allocation-profiling: describe sysctrl limitations in debug mode
From: Suren Baghdasaryan
Date: Fri Jan 16 2026 - 13:45:42 EST
On Fri, Jan 16, 2026 at 10:30 AM Suren Baghdasaryan <surenb@xxxxxxxxxx> wrote:
>
> On Thu, Jan 15, 2026 at 6:18 PM SeongJae Park <sj@xxxxxxxxxx> wrote:
> >
> > On Thu, 15 Jan 2026 09:05:25 -0800 Suren Baghdasaryan <surenb@xxxxxxxxxx> wrote:
> >
> > > On Wed, Jan 14, 2026 at 9:57 PM Matthew Wilcox <willy@xxxxxxxxxxxxx> wrote:
> > > >
> > > > On Wed, Jan 14, 2026 at 09:45:57PM -0800, Suren Baghdasaryan wrote:
> > > > > + warnings produces by allocations made while profiling is disabled and freed
> > > >
> > > > "produced"
> > >
> > > Thanks! I'll wait for a day and if there are no other objections, I
> > > will post a fixed version.
> >
> > Assuming Matthiew's good finding will be fixed,
> >
> > Acked-by: SeongJae Park <sj@xxxxxxxxxx>
>
> Thanks!
>
> >
> > Fwiw, the typo is also on the .../sysctl/vm.rst part.
>
> Correct, I'll fix in both places.
v2 is posted at
https://lore.kernel.org/all/20260116184423.2708363-1-surenb@xxxxxxxxxx/
>
> > And from the finding, I
> > was wondering if it is better to put the description only one of two documents
> > rather than having the duplication, and further if the 'Usage:' part of
> > allocation-profiling.rst is better to be moved to
> > 'Documentation/admin-guide/mm/'. But I ended up thinking those are too trivial
> > and small things.
>
> Yes, I didn't want to complicate reader's life by adding a reference
> for a couple of sentences.
>
> >
> >
> > Thanks,
> > SJ