perf: Fix default aux_watermark calculation
authorAdrian Hunter <adrian.hunter@intel.com>
Mon, 24 Jun 2024 20:11:00 +0000 (23:11 +0300)
committerPeter Zijlstra <peterz@infradead.org>
Thu, 4 Jul 2024 14:00:23 +0000 (16:00 +0200)
commit43deb76b19663a96ec2189d8f4eb9a9dc2d7623f
tree7eeecacf11ab99771efdcb7e15219b909e43bdb4
parentdbc48c8f41c208082cfa95e973560134489e3309
perf: Fix default aux_watermark calculation

The default aux_watermark is half the AUX area buffer size. In general,
on a 64-bit architecture, the AUX area buffer size could be a bigger than
fits in a 32-bit type, but the calculation does not allow for that
possibility.

However the aux_watermark value is recorded in a u32, so should not be
more than U32_MAX either.

Fix by doing the calculation in a correctly sized type, and limiting the
result to U32_MAX.

Fixes: d68e6799a5c8 ("perf: Cap allocation order at aux_watermark")
Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lore.kernel.org/r/20240624201101.60186-7-adrian.hunter@intel.com
kernel/events/ring_buffer.c