From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43548) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZuKuD-00032V-Ua for qemu-devel@nongnu.org; Thu, 05 Nov 2015 08:48:46 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZuKu9-0004HQ-TV for qemu-devel@nongnu.org; Thu, 05 Nov 2015 08:48:45 -0500 Received: from mx1.redhat.com ([209.132.183.28]:59088) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZuKu9-0004HF-Nm for qemu-devel@nongnu.org; Thu, 05 Nov 2015 08:48:41 -0500 References: <1446725610.30393.23.camel@redhat.com> From: Laszlo Ersek Message-ID: <563B5E37.2040701@redhat.com> Date: Thu, 5 Nov 2015 14:48:39 +0100 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] virtio-gpu doesn't build if you do a linux-headers update from kvm/next List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell , Gerd Hoffmann Cc: Paolo Bonzini , QEMU Developers On 11/05/15 13:32, Peter Maydell wrote: > On 5 November 2015 at 12:13, Gerd Hoffmann wrote: >>> etc, because all the virtio_gpu definitions disappear from >>> include/standard-headers/linux/virtio_gpu.h. >> >> Updates not yet in mainline, they are sitting in drm-next and should >> land during the merge window (i.e. 4.4-rc1 should have them). >> >> I'd suggest to exclude virtio_gpu.h changes when updating linux headers >> for the time being. > > I would strongly prefer it if we could get to a point where > we can say "kernel headers must only be updated from this tree" > and be guaranteed that it always works. This used to be true > with the tree in question being kvm/next, but it doesn't seem > to be so now. If it's going to be common that we have header > changes that don't go via kvm/next, maybe we need to coordinate > a tree that merges together the abi-guaranteed-stable changes > from different places before they hit mainline? I've always frowned upon importing headers from one project to another project. First, they can have different coding styles. (Case in point.) Second, not everything that needs to be defined for the original project is useful to the receiving project, and I find such cruft in the receiving project very annoying. Third, in some cases it might even raise licensing questions. If it is an ABI, it should be specified in text format somewhere, and then the projects can have their independent type definitions, macros etc that implement the spec. Laszlo