lkml.org 
[lkml]   [2009]   [Jul]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [KVM PATCH v9 2/2] KVM: add iosignalfd support
On 07/07/2009 03:48 PM, Michael S. Tsirkin wrote:
>>>> +
>>>> + __kvm_io_bus_unregister_dev(bus,&p->dev);
>>>> + iosignalfd_release(p);
>>>>
>>>>
>>> a single deassign removed multiple irqfds? Looks ugly.
>>>
>>>
>> Avi requested this general concept.
>>
>
> Really? Avi, could you explain? I would think each
> assign needs to be matched with 1 deassign. No?
>

Doesn't it follow naturally? How can a single deassign remove multiple
fds (unless all were registered with exactly the same arguments)?

Maybe we should instead detect that the duplicate iosignaldfds are
registered.

Note that we do need to allow the same fd to be multiple times, but not
with exactly the same address/match etc.

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



\
 
 \ /
  Last update: 2009-07-07 14:57    [W:0.058 / U:0.296 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site