Re: [PATCH 2/3] KVM: Use kvm_free_assigned_irq() for free irq

Avi Kivity
2008-12-30T11:27:25Z

Sheng Yang wrote:
> So still, how about deal with this at the beginning of kvm_destory_vm? Discard
> currently reference count. In the function, disable all known interrupt(and
> result some nested disabled but we would free them later), then
> cancel_work_sync(). At this time, I think the state is legal(at least
> workable).
>

Certainly, if it's the first thing we do, it's legal (and the mutex is
valid etc.)

Just be sure to add a nice comment there.

--
error compiling committee.c: too many arguments to function