[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] vfio: Import vfio kernel header
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] vfio: Import vfio kernel header |
Date: |
Thu, 02 Aug 2012 18:45:56 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2012-08-02 18:37, Alex Williamson wrote:
> On Thu, 2012-08-02 at 11:02 +0200, Jan Kiszka wrote:
>> On 2012-08-01 20:09, Alex Williamson wrote:
>>> On Wed, 2012-08-01 at 09:13 +0200, Jan Kiszka wrote:
>>>> On 2012-08-01 07:18, Alex Williamson wrote:
>>>
>>>>> +#define VFIO_IOMMU_UNMAP_DMA _IO(VFIO_TYPE, VFIO_BASE + 14)
>>>>> +
>>>>> +#endif /* VFIO_H */
>>>>>
>>>>
>>>> Please patch update-linux-headers.sh and let it do its work instead
>>>> (separate patches).
>>>
>>> Unfortunately I missed updating the Kbuild file in the kernel, so vfio.h
>>> doesn't get installed yet. I'll include that in my next pull request,
>>> but maybe in the meantime I should temporarily put this header back in
>>> hw/ unless you have a better idea. Thanks,
>>
>> Well, if it's going to be fixed very soon, we can merge it like this.
>> But you should already be able provide a patch for the update script so
>> that the next one running it against a fixed kernel will not
>> accidentally drop the vfio header again.
>
> What's the reason for the rm -fr; mkdir -p in the update script? I
> don't see how we'd be unintentionally generating cruft in these
> directories if we were to just overwrite files instead of removing the
> directory and repopulating it. I'll get the kernel fixed soon, but I'm
> tempted to add vfio and drop the expunge of the directory at the same
> time. Thanks,
Well, chances to pile up cruft are low, granted. On the other hand, this
policy enforces proper upstream header exporting as we see...
Jan
signature.asc
Description: OpenPGP digital signature
[Qemu-devel] [PATCH 3/3] vfio: Enable vfio-pci and mark supported, Alex Williamson, 2012/08/01
Re: [Qemu-devel] [PATCH 3/3] vfio: Enable vfio-pci and mark supported, Anthony Liguori, 2012/08/13
Re: [Qemu-devel] [PATCH 3/3] vfio: Enable vfio-pci and mark supported, Alex Williamson, 2012/08/14
Re: [Qemu-devel] [PATCH 3/3] vfio: Enable vfio-pci and mark supported, Avi Kivity, 2012/08/14
[Qemu-devel] [PATCH 2/3] vfio: vfio-pci device assignment driver, Alex Williamson, 2012/08/01