From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:49349) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gjgQp-0001kM-LI for qemu-devel@nongnu.org; Wed, 16 Jan 2019 03:20:16 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gjgQd-0000HZ-ET for qemu-devel@nongnu.org; Wed, 16 Jan 2019 03:20:13 -0500 Received: from mail-wm1-f65.google.com ([209.85.128.65]:36533) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gjgQd-0008Rv-0a for qemu-devel@nongnu.org; Wed, 16 Jan 2019 03:20:03 -0500 Received: by mail-wm1-f65.google.com with SMTP id p6so980574wmc.1 for ; Wed, 16 Jan 2019 00:19:58 -0800 (PST) Date: Wed, 16 Jan 2019 09:19:53 +0100 From: Stefano Garzarella Message-ID: <20190116081953.vvni4sp4q5jw26ms@steredhat> References: <20190115100058.44712-1-sgarzare@redhat.com> <20190115100058.44712-5-sgarzare@redhat.com> <20190115135548-mutt-send-email-mst@kernel.org> <3db58f6f-44fc-da64-2cfb-41404a9b2319@redhat.com> <20190115150238-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190115150238-mutt-send-email-mst@kernel.org> Subject: Re: [Qemu-devel] [PATCH v2 4/4] hw/i386/pc: use PVH option rom List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: Eric Blake , qemu-devel@nongnu.org, Boris Ostrovsky , Eduardo Habkost , Gerd Hoffmann , George Kennedy , Marcel Apfelbaum , Paolo Bonzini , Marc-Andre Lureau , Richard Henderson , Stefan Hajnoczi , Liam Merwick , Maran Wilson Hi Michael, On Tue, Jan 15, 2019 at 03:05:27PM -0500, Michael S. Tsirkin wrote: > On Tue, Jan 15, 2019 at 01:12:21PM -0600, Eric Blake wrote: > > On 1/15/19 12:57 PM, Michael S. Tsirkin wrote: > > > On Tue, Jan 15, 2019 at 11:00:58AM +0100, Stefano Garzarella wrote: > > >> Use pvh.bin option rom when we are booting an uncompressed > > >> kernel using the x86/HVM direct boot ABI. > > >> > > >> Signed-off-by: Stefano Garzarella > > >> Based-on: <1545422632-24444-5-git-send-email-liam.merwick@oracle.com> > > > > > > I don't think this is a great way to give attribution. > > > Can you pls include the author name and the S.O.B from there as well? > > > > > > > > >> --- > > > > Patchew understands Based-on: tags as meaning a prerequisite patch on > > the list that must be applied first before this patch can apply (and NOT > > in the sense that this patch is an updated revision to replace an > > earlier revision posted by someone else) - but when using the annotation > > to keep Patchew happy, it should either be in a 0/N cover letter, or for > > a single patch, after the --- separator, as the condition of a > > prerequisite patch not being applied is a transient condition relevant > > to the current build but not something that needs to be recorded in > > long-term git history. > > > > -- > > Eric Blake, Principal Software Engineer > > Red Hat, Inc. +1-919-301-3226 > > Virtualization: qemu.org | libvirt.org > > > > Good to know. > But I still think listing the original author (e.g. Based on patches by > Foo Bar) or such is the nice thing to do. very sorry for the misunderstanding, as Eric and Paolo said, I used Based-on tag only to inform Patchew and maintainers that this patch must be applied after Liam's patch that is under review. Thanks, Stefano > > -- > MST -- Stefano Garzarella Red Hat