[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] x86/MSI: fix guest unmasking when handling IRQ via event channel



>>> On 08.07.15 at 13:14, <david.vrabel@xxxxxxxxxx> wrote:
> On 08/07/15 11:58, Jan Beulich wrote:
>>>>> On 08.07.15 at 11:39, <david.vrabel@xxxxxxxxxx> wrote:
>>> On 08/07/15 09:56, Jan Beulich wrote:
>>>> +
>>>>  /* Set IRQ type for an SPI */
>>>>  int irq_set_spi_type(unsigned int spi, unsigned int type);
>>>>  
>>>> --- a/xen/include/xen/irq.h
>>>> +++ b/xen/include/xen/irq.h
>>>> @@ -172,4 +172,8 @@ unsigned int set_desc_affinity(struct ir
>>>>  unsigned int arch_hwdom_irqs(domid_t);
>>>>  #endif
>>>>  
>>>> +#ifndef arch_evtchn_bind_pirq
>>>> +void arch_evtchn_bind_pirq(struct domain *, int pirq);
>>>
>>> ... moving this into xen/include/asm-x86/irq.h
>> 
>> Oh, right, (also to Julien) - this is exactly the reason I do not want it
>> to be an inline function for ARM: I want the declaration here, not
>> replicated in every interested arch's header.
> 
> Ok.
> 
> FWIW, with this requirement I would (instead of the macros) add a weak
> arch_evtchn_bind_pirq() that's a no-op.

Yeah, that's how Linux likes to do it. But we learned the hard way
that weak conflicts with our making symbols hidden by default, so
no, weak is not an option either I'm afraid.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.