09.02.2024 20:11, Hanna Czenczek :
The mentioned comit is v8.2.0-812-gd3f6f294ae, - ie, past 8.2.
Is this new change still relevant for stable?
Sorry again. :/
There's nothing to be sorry about here - it's regular work, and is quite
good at it, - I just asked to be sure, maybe I misunderstood something.
This patch is a clean-up patch that won’t apply to 8.2. Now, 8.2 does have basically the same logic as described in the patch
message (d3f6f294aea restored it after it was broken), so a similar patch could be made for it (removing the event_notifier_set() from
virtio_blk_data_plane_start()), but whether we kick the virtqueues once or twice on start-up probably won’t make a difference, certainly not in terms
of correctness.
Ok, excellent, this makes good sense now.
I'm not including this one in stable-8.2 :)
Thank you very much for the excellent work and
the clarification!
/mjt
