On 7/14/26, 11:44 AM, "Vladimir Sementsov-Ogievskiy"
<[email protected] <mailto:[email protected]>> wrote:
> +static void tap_vm_state_change(void *opaque, bool running, RunState state)
> +{
> + TAPState *s = opaque;
> +
> + if (running) {
> + if (s->read_poll_detached) {
> + tap_read_poll(s, true);
> + s->read_poll_detached = false;
> + }
> + } else if (state == RUN_STATE_FINISH_MIGRATE) {
> + if (s->read_poll) {
> + s->read_poll_detached = true;
> + tap_read_poll(s, false);
> + }
> + }
> +}
> +
tap_read_poll is called in a few other places. Is it necessary to update
read_poll_detached there as well?
I'm not sure that it is, I'm trying to wrap my head around any possible
ordering issues and it isn't obvious to me one way or the other.
Thanks,
Ben
