On 27/09/18 11:11, Christian Lindig wrote:
>
>> On 27 Sep 2018, at 09:59, Andrew Cooper <andrew.coop...@citrix.com> wrote:
>>
>> On 27/09/18 08:53, Yang Qian wrote:
>>> 1. Add a common bind virq function
>>> 2. Reduce the stub code of `bind_dom_exc_virq`
>>>
>>> Signed-off-by: Yang Qian <yang.q...@citrix.com>
>> CC'ing the relevant maintainers.
>>
>> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> (forwarding my
>> internal review of this patch)
> There is a discussion about caml_{leave,enter}_blocking_section required for 
> this to work in a thread context that is not yet addressed in the patch.

Right, but that is just a performance issue is it not?  We hold the GC
lock longer than we need to.

i.e. AFACIT, that is a good followup patch, but doesn't invalidate this one?

~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to