Re: [PATCH net] net: gen_estimator: protect seqcount updates from hardirq readers

[Runyu Xiao]

Hi Sebastian,

Thanks for the detailed review.

You are right. I rechecked the in-tree callers and confirmed that
gen_estimator_read() has no hardirq caller. The hardirq call in my test
harness was an artificial caller that is not part of a reachable mainline
execution path, so it does not demonstrate a real bug.

The injected reader also overlooked that a hypothetical caller would first
hit est_fetch_counters() and stats_lock. In addition, replacing
preempt_disable_nested() with local_irq_save() would conflict with the
PREEMPT_RT rationale from 9f74c0ea9b26.

I will withdraw this patch and will not send a v2.

Regards,
Runyu