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

Re: [XEN PATCH v3] xen/spinlock: mechanically rename parameter name 'debug'


  • To: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Wed, 26 Jul 2023 08:47:42 +0200
  • 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=cv/S8H7VBVCXw65UxxrbNttfmXrKo98kPc4TNsom7Ns=; b=JKnFP6sVwefppiMuVGrp5bGprP5/q+gkI9rz0YfIDdCOTxt1dLk6WHFxazR0OUMgsoUdG+n9RkcZJU7D1MJtpblOtNpjrtWt0ezErwhjypYG6XrCdZ6erpEeVbWL3E56Q3PqR3CQ+ZcA/mzEp5QB8qFcBr7DsRGbCvWla7ChIk/CDiPJhQdaKWsu9vECXRRmy6mBw8XRWwKdbxTPK1g4dqPad7l7/7LmVWXKRfPpIpoDCeAlYxQcRKANXEkZftVVvWuEy3HtD8jINjZR5ADscrIHjyvy1BMeNOkcZQ7fG+UUZrFIlOgYPTtClqHLcs23htK1Q2zhGDUn0YHNx6hf2Q==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hGwU1C5EO+btJtuGOQd4koBobG9bWW+iPIcxeTPJjYCR05SrOv1Tx7pB2d8p/xXEw8EXFmopo+mYg4OT3RvfHqV+pZcSY8oLJa68tpx2jDTwCMQlEq16QOp9a2S93bh2ICMaaelz6a2S7QWygKLr99+kzeM4DLaVhxsLjvlGb6SVy5SHxwFaez4GA/L3cmdh/6QMa5pu9JX0GTWJumLQbBxSMPfcJNjiuWGLKckDjGXkUWgM5HAIv+pMP/08kgoW3WOsy6f2mknRi420Pw5LvTIBnK+xK6dU7d8lc/R+XGRfXH6IZurfPZ31xLxJ7yzUHIf+X459bzYEyUia9RhwXw==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: sstabellini@xxxxxxxxxx, michal.orzel@xxxxxxx, xenia.ragiadakou@xxxxxxx, ayan.kumar.halder@xxxxxxx, consulting@xxxxxxxxxxx, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Wei Liu <wl@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Wed, 26 Jul 2023 06:48:20 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 26.07.2023 08:42, Nicola Vetrini wrote:
> On 26/07/23 08:34, Jan Beulich wrote:
>> On 25.07.2023 22:45, Nicola Vetrini wrote:
>>> Rule 5.3 has the following headline:
>>> "An identifier declared in an inner scope shall not hide an
>>> identifier declared in an outer scope"
>>>
>>> To avoid any confusion resulting from the parameter 'debug'
>>> hiding the homonymous function declared at
>>> 'xen/arch/x86/include/asm/processor.h:428'
>>> the rename of parameters s/debug/lkdbg/ is performed.
>>>
>>> Signed-off-by: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
>>> ---
>>> Changes in v2:
>>> - s/dbg/lkdbg/
>>> Changes in v3:
>>> - Added missing renames for consistency
>>
>> Hmm, you asked whether to send v3, but then you didn't wait for an
>> answer. So to repeat what I said there: I'd prefer if we could first
>> settle whether to rename the conflicting x86 symbol.
>>
> 
> Stefano replied asking for a v3 [1] before I had a chance to read your 
> message this morning.

Right, sorry, I spotted his reply only after seeing the v3.

Jan



 


Rackspace

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