From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33310) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wn2nb-00057o-Qh for qemu-devel@nongnu.org; Wed, 21 May 2014 05:27:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Wn2nW-0008RR-SG for qemu-devel@nongnu.org; Wed, 21 May 2014 05:26:59 -0400 Received: from mx1.redhat.com ([209.132.183.28]:58988) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wn2nW-0008Qv-IS for qemu-devel@nongnu.org; Wed, 21 May 2014 05:26:54 -0400 Date: Wed, 21 May 2014 12:25:45 +0300 From: "Michael S. Tsirkin" Message-ID: <20140521092545.GA19646@redhat.com> References: <20140520150510.GH1657@ERROL.INI.CMU.EDU> <537C6696.9000509@suse.de> <20140521090456.GC19109@redhat.com> <537C6E0A.4020200@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <537C6E0A.4020200@suse.de> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH] e1000: allow command-line selection of card model List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andreas =?iso-8859-1?Q?F=E4rber?= Cc: romain@dolbeau.org, qemu-devel@nongnu.org, agraf@suse.de, "Gabriel L. Somlo" , stefanha@redhat.com, pbonzini@redhat.com On Wed, May 21, 2014 at 11:12:42AM +0200, Andreas F=E4rber wrote: > Am 21.05.2014 11:04, schrieb Michael S. Tsirkin: > > On Wed, May 21, 2014 at 10:40:54AM +0200, Andreas F=E4rber wrote: > >> Hi, > >> > >> Am 20.05.2014 17:05, schrieb Gabriel L. Somlo: > >>> Allow selection of different card models from the qemu > >>> command line, to better accomodate a wider range of guests. > >>> > >>> Based-on-patch-by: Romain Dolbeau > >> > >> If that patch carried a Signed-off-by line, you should retain it. > >=20 > > Actually I think that would be confusing. Romain didn't sign off > > on *this* patch, he signed off on a previous one. > > A signature by Gabriel indicates Developer's Certificate of Origin 1.= 1 > > which has an option to incorporate other's work - it > > does not seem to require signatures by these others. >=20 > With the same argument you could drop anyone's Sob you get as a > maintainer. I could but it would not be nice to submitters, and it drops useful info (author's Sob). So if someone thinks there's problematic code here and comes complaining, we want to be able to say "this code came from XYZ". > But the purpose of Sob is to track through whose hands a > patch went, not just who last touched it. Went untouched or mostly untouched. Did you bother checking? I looked and Romain's patch isn't very similar to this one. > My point here was that Based-on-patch-by is very unusual. What's the harm? Gabriel's just being nice and crediting other's work. > The alternative would be to leave the original From: Romain Dolbeau, hi= s > Sob, then a [gsomlo: Dropped this, added that] followed by Sob. >=20 > Cheers, > Andreas That's just asking submitter to do a lot of extra work, I don't see why would we place roadblocks in submitter's paths like this. Linux certainly does not and we didn't ask for this in the past. Further, the patch author in git will also be the original author then which is only fair if the patch is changed in very minor ways. In which case keeping the original Sob around *would* be right. > --=20 > SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany > GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrn= berg