From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:48563) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SwyXH-0007n8-Sw for qemu-devel@nongnu.org; Thu, 02 Aug 2012 12:46:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SwyXG-0008Pt-Q5 for qemu-devel@nongnu.org; Thu, 02 Aug 2012 12:46:07 -0400 Received: from mout.web.de ([212.227.17.11]:49254) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SwyXG-0008Oy-G6 for qemu-devel@nongnu.org; Thu, 02 Aug 2012 12:46:06 -0400 Message-ID: <501AAEC4.7000005@web.de> Date: Thu, 02 Aug 2012 18:45:56 +0200 From: Jan Kiszka MIME-Version: 1.0 References: <20120801050241.22163.78549.stgit@bling.home> <20120801051807.22163.96305.stgit@bling.home> <5018D713.1050304@web.de> <1343844589.6698.3.camel@bling.home> <501A4240.8050602@web.de> <1343925460.8003.19.camel@ul30vt> In-Reply-To: <1343925460.8003.19.camel@ul30vt> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enigDD9FC6F83A272AFABD7914EA" Subject: Re: [Qemu-devel] [PATCH 1/3] vfio: Import vfio kernel header List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alex Williamson Cc: aik@ozlabs.ru, aliguori@us.ibm.com, qemu-devel@nongnu.org, kvm@vger.kernel.org This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enigDD9FC6F83A272AFABD7914EA Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable 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 vfi= o.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 i= n >>> 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 s= o >> that the next one running it against a fixed kernel will not >> accidentally drop the vfio header again. >=20 > 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 --------------enigDD9FC6F83A272AFABD7914EA Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAlAarsYACgkQitSsb3rl5xSJWwCg1Py7eYFYaI6AaH5RIHGVsQMq 2BkAn0kiSM58a3xcyv0K4llLyo1GXhjN =maBY -----END PGP SIGNATURE----- --------------enigDD9FC6F83A272AFABD7914EA--