diff options
| author | Victor Nogueira <victor@mojatatu.com> | 2026-09-19 20:04:27 -0300 |
|---|---|---|
| committer | Paolo Abeni <pabeni@redhat.com> | 2026-09-22 12:46:46 +0200 |
| commit | f09db8b83e836f9554ec8e0f985c70699b3cb83f (patch) | |
| tree | 69f0afa5db7c6de5a930b95cf56d78e136cbc3fd /include/linux/debugobjects.h | |
| parent | 10cfa109c880092df32e396647b4afdca9be8350 (diff) | |
net/sched: act_mirred: account for TCA_MIRRED_BLOCKID in get_fill_size
tcf_mirred_get_fill_size budgets only TCA_MIRRED_PARMS, but
tcf_mirred_dump also emits TCA_MIRRED_BLOCKID whenever the action was
created with a block instead of a device, so the budget falls 8 bytes
short of what the dump may produce for such actions.
Account for TCA_MIRRED_BLOCKID unconditionally: reading tcfm_blockid to
size it conditionally would race a concurrent replace, which can flip the
action between device and block between sizing and dumping.
Reported-by: Sashiko <sashiko-bot@kernel.org>
Link: https://netdev-ai.bots.linux.dev/sashiko/#/patchset/20260824153903.4143642-1-victor@mojatatu.com
Co-developed-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: Victor Nogueira <victor@mojatatu.com>
Link: https://patch.msgid.link/QDISC-RYTB.v1.20260918093205@mojatatu.com.1
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Diffstat (limited to 'include/linux/debugobjects.h')
0 files changed, 0 insertions, 0 deletions