lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Thu, 11 Sep 2014 09:11:29 +0200
From:	Paolo Bonzini <pbonzini@...hat.com>
To:	Christoffer Dall <christoffer.dall@...aro.org>
CC:	Eric Auger <eric.auger@...aro.org>, eric.auger@...com,
	marc.zyngier@....com, linux-arm-kernel@...ts.infradead.org,
	kvmarm@...ts.cs.columbia.edu, kvm@...r.kernel.org, gleb@...nel.org,
	borntraeger@...ibm.com, cornelia.huck@...ibm.com,
	xiantao.zhang@...el.com, agraf@...e.de,
	linux-kernel@...r.kernel.org, patches@...aro.org
Subject: Re: [PATCH] KVM: EVENTFD: remove inclusion of irq.h

Il 11/09/2014 05:09, Christoffer Dall ha scritto:
> On Mon, Sep 01, 2014 at 12:11:19PM +0200, Paolo Bonzini wrote:
>> Il 01/09/2014 10:36, Eric Auger ha scritto:
>>> No more needed. irq.h would be void on ARM.
>>>
>>> Signed-off-by: Eric Auger <eric.auger@...aro.org>
>>>
>>> ---
>>>
>>> I don't think irq.h is needed anymore since Paul Mackerras' work. However
>>> I did not compile for all architectures.
>>> ---
>>>  virt/kvm/eventfd.c | 1 -
>>>  1 file changed, 1 deletion(-)
>>>
>>> diff --git a/virt/kvm/eventfd.c b/virt/kvm/eventfd.c
>>> index 3c5981c..0c712a7 100644
>>> --- a/virt/kvm/eventfd.c
>>> +++ b/virt/kvm/eventfd.c
>>> @@ -36,7 +36,6 @@
>>>  #include <linux/seqlock.h>
>>>  #include <trace/events/kvm.h>
>>>  
>>> -#include "irq.h"
>>>  #include "iodev.h"
>>>  
>>>  #ifdef CONFIG_HAVE_KVM_IRQFD
>>>
>>
>> Acked-by: Paolo Bonzini <pbonzini@...hat.com>
>>
>> Christoffer, please include this via the ARM tree, together with ARM
>> irqfd support.  Thanks,
>>
> Marc is dealing with the tree this week and the next so he can apply it
> to kvmarm/queue.  Do you want it to wait and go with the irqfd patch
> (which has dependencies not yet resolved) or should we just queue it?

As you prefer.

Paolo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ