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

Re: [PATCH] xen: Remove trigraphs from comments


  • To: Michal Orzel <michal.orzel@xxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Tue, 6 Dec 2022 15:08:56 +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=5hxERaxNNSTSSoFJdRk1K9m7CRtM30+s/CnjYtwso9c=; b=YOcvLl1vvkLKQWIEpF4QvIfkn+KXeNwt4MGFNYTyR7vsMv5Tcys3BBJpZqGTmLgvvfdf3gbX9Lq4V8LC4hlnhlMcI75cVASV0+vBA7c9f0EtDIp02bXHuh+8uoPiY6xeP+9f4Pb9Cb09ispunFHw/NYzoQA7txfK7C5O3lqMggaobjMkY17U8UO+y20e6nIxRLKCXDunFF+09sKx/Z+qg7sR0zINYEpOXH5AkZOxJHKIZ6ZPI6W55DSn+ipEg96noCZL56Vh8WgH0R1fDm9VZAmaLNciol9UTwD5SM8NRf+0vgB87FUfrZYS6G6Pm6kB/I3zwRcFrDTTlPXl/fCWTA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Xv6/uRAbary8Kd+aR1bT/bMe/6jpBVbP/cifv9tBmSFgJh/bBGEFUEV+AjAHwHNiCDZk7/5M9cXxGpSFwu2P4KZFDg8ALfP8jWbuLbPy75GzRZe8qvPl8hRQRnoBqxK5Xjx8JXZ4aZJj33gAQwZzJ4ilpJ3syGUZL6//khtAtiOevzJ2S5xxJrEkVOhAbaBxuTbiYO3IqoFPGnkBv+qSDUi+BBFZywnxiRoSgIIel3zdllq7d9OFnv72LSNH7HY/twMSINm6JKxrQHUqGhjWlKqg4YVN8NvwuKz4J+Lj3uK7YAvenGGUmsH5zUPdb58JDsVIGX3GJIfsf99DSYA+pQ==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Tue, 06 Dec 2022 14:09:06 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 06.12.2022 15:05, Michal Orzel wrote:
> On 06/12/2022 14:46, Jan Beulich wrote:
>> On 06.12.2022 14:05, Michal Orzel wrote:
>>> Also there is no functional change being made by this patch so it is ok
>>> to change Xen and not Linux in this case (which has quite a lot of 
>>> trigraphs all over the place).
>>
>> Based on what criteria are you saying this is ok (unconditionally)?
> 
> I said that it is ok to change Xen and not Linux because this file already 
> diverged,
> so we cannot assume that future backports will apply cleanly. If we change a 
> file
> that did not diverge, then we are required to modify the origin first and then
> do the backport.

A file might have diverged, yet commits to be pulled in from the original
tree may still apply cleanly. That why, in the general case, we aim at
limiting the amount of divergence, and we prefer to pull in changes bringing
us back closer to the (meanwhile evolved) original file.

Jan



 


Rackspace

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