On Tue, 15 Sep 2026 08:23:55 +0100 Vincent Donnefort <[email protected]> wrote:
> > For a single CPU, this now happens before the nr_pages == > > cpu_buffer->nr_pages > > early exit, so writing the same size to per_cpu/cpuN/buffer_size_kb on a > > mapped or persistent instance CPU fails with EBUSY instead of succeeding. > > Intended ? > > Actually no, I didn't see that it is also "fixing" this discrepancy between > the > per_cpu buffer_size_kb and the global one. > > It seems to me better to align the behaviour for both interface, but then it > is > touching something that is user interface... > > Steven, WDYS? I don't think we should worry about it. If something is mapped, then we shouldn't be touching that file. Even writing the same value should error out. There's no need to do that. Hopefully it doesn't break anything because if it does, then yeah, we will need to do something different here. Oh, and can you break this up into two patches. One that adds this and and one that does the: Additionally, remove the unnecessary cpumask_test_cpu in ring_buffer_subbuf_order_set(). for_each_buffer_cpu() already iterates over buffer->cpumask. Hey, breaking up patches improves your commit count ;-) -- Steve
