[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/7] xz: add fall-through comments to a switch statement


  • To: Ian Jackson <iwj@xxxxxxxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Mon, 6 Dec 2021 14:44:18 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=MJFCJ+Hmy/dkB/YXnNr1iTVUpk9yYd3VJjNm55eawj0=; b=c4KTLORYW97xsuabWyadkMUQPEOJZtjnuHujf+NBfaBxFFqSPbDENsl6mDvSFvHmhrmtA3fAXerudA8Mv/Z86zQJnXH4rmK0pWkZ5TRkUrJRD9DkygcrQesDhdRSwbi9obdyy+2oYEDBUJZtM2p+wBuJCZECxHqr2e5FxU76U4K6Mb9tK8EYX/oP2ZvOFQze7K7fs/P4upBWuhFidCAX/hyXlcqt/Bf7ipj0J9Ig5f2aFcGIFHsZx9plaKv9OboYXc3ihRU8y4d0Q9r2s6jEdxOtudFMaGW3OTb93TqoCHSZmQyeqZ8c8tlTVmQr070idMXM0ik9yFhM0ftL4CXDhQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BteJ8c849aEBVZebx0yqy/FO+s1TvAYTK7xJ6c5khE2xS/87qlHn7xBZRg2tnuhQANvrCvvNumw23/CmMry3ood826waBM7eecBmdKl9DwfLKVsfK/BNOleOh2xzZmu2aU9+4INyxWx+NPg2B0bE6q4UPga2gGu/XVmyGL0eHFd9xqvpHrW7YL2WFVb9R3YSgZt8mrlBNllPGQJvsmBM3E2LD+R1srvPKI+5LUcwWPGCCt2ql5W1NmyVXFKqYmeb67z7mGmnkBMPSluVC8FqdSFmxahv4XL5JUXzWasJf/szYaz5ZufUL/o0PeCpS9aRw2rKlaumdolNSHxK6qohhg==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: Julien Grall <julien@xxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 06 Dec 2021 13:44:35 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 26.11.2021 13:52, Ian Jackson wrote:
> Jan Beulich writes ("Re: [PATCH 1/7] xz: add fall-through comments to a 
> switch statement"):
>> On 26.11.2021 11:04, Julien Grall wrote:
>>> For this case, you provided some sort of an explanation but so far, I am 
>>> still waiting for a link to confirm that the signed-off-by match the one 
>>> on the ML.
>>
>> I haven't been able to easily find a mail archive holding this patch.
> 
> I 100% agree with Julien on all points in this thread.
> 
> Please can we keep the Linux S-o-b.
> 
> Note that S-o-b is not a chain of *approval* (whose relevance to us is
> debateable) but but a chain of *custody and transmission* for
> copyright/licence/gdpr purposes.  That latter chain is hightly
> relevant to us.
> 
> All such S-o-b should be retained.
> 
> Of course if you got the patch somewhere other than the Linux commit,
> then the chain of custody doesn't pass through the Linux commit.  But
> in that case I expect you to be able to say where you got it.

I've submitted v2 with S-o-b restored as far as necessary to meet this
requirement. I did not restore all of them, because I continue to not
see the value of retaining them. You saying "is highly relevant to us"
is a statement, but not an explanation of why tags beyond those in the
original submissions need retaining.

Without me seeing the need / value, I'm afraid I see only two ways
forward: Either things are acceptable as they are now (and will be for
future similar imports), or it needs to be someone else to put time
into spotting and then pulling in such changes.

Jan




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.