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

RE: [PATCH 16/37] xen/x86: export srat_bad to external


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Wei Chen <Wei.Chen@xxxxxxx>
  • Date: Thu, 27 Jan 2022 08:35:02 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; 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=mab40k7/0J8oTR/+k8Vujec6oE3MdZUkH3iyI2oUna8=; b=ki+H2Y2P0+U7aMNUeMwubKB6UXClupVgy1wrJxYPGyQOiV4bf28uo6mKLIjyhULpE19ZLpea+3uref+B10aHP1TNhyHE6+izKLFuGiRpEdLDhF0rsPXutgS6TAdntL8xgdnp5Mq10ItRYGj9MTbf2rbVGTnyu2mOdKk9cpEykxYHc43YbCOmTJOOr/raEsgZGTIdPJpyjCSWA+bveL86/2zGZvQ1IC7duwTA7PlYu+23ygi326LGzFN2lP8DHTqkzmnWV7UHK3yp3tc+jvBP1d5twHpGCOWXa1IeLjm8LSFOita4iZ69mguNjpsSElDYdakjgVmBxgzPp5I8Ez8cnw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=LIZlkqYkohGaieNnrm9xcmQz/u+XeNshbKMKln+NGei5C8Xrjvhh4BE6t+Xx818T5/kgd6NnUGYLcc80Tm9Fx1ALBF5AXHfrHNOOQ0qJXXpJfLY3EzA79wJfLDDnQ6xx87zxxkc9nLXtB0QB/hlJZelLMm9Gp7Mz0ryw939UMuwUtM84EAY3I1/u+CQeGkIhgMvI6YVpyps4ZYj1VTkc5i0Jz1SClaCJdvIsDAS8t87NYlSs9hIX5kEGjZ6yTlnFuAzAZppkOVw8UfVzkebxZ/L6ceQh9w75zOgh7ISQSxct+9G881h9gPJU1FGNdHVUYA3tsQP9KRZRgsymv2oMzQ==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "sstabellini@xxxxxxxxxx" <sstabellini@xxxxxxxxxx>, "julien@xxxxxxx" <julien@xxxxxxx>
  • Delivery-date: Thu, 27 Jan 2022 08:35:27 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHXsHMnFy02K9wv1EmfXGO37i6/5qx0SfuAgAMEmKA=
  • Thread-topic: [PATCH 16/37] xen/x86: export srat_bad to external

Hi Jan,

> -----Original Message-----
> From: Jan Beulich <jbeulich@xxxxxxxx>
> Sent: 2022年1月25日 18:22
> To: Wei Chen <Wei.Chen@xxxxxxx>
> Cc: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>; xen-
> devel@xxxxxxxxxxxxxxxxxxxx; sstabellini@xxxxxxxxxx; julien@xxxxxxx
> Subject: Re: [PATCH 16/37] xen/x86: export srat_bad to external
> 
> On 23.09.2021 14:02, Wei Chen wrote:
> > srat_bad is used when NUMA initialization code scan SRAT failed.
> > It will turn fw_numa to disabled status. Its implementation depends
> > on NUMA implementation. We want every NUMA implementation to provide
> > this function for common initialization code.
> >
> > In this patch, we export srat_bad to external. This will allow to
> > have the code mostly common.
> 
> Here as well as in the subject it would help if the function name
> wasn't the wrong way round. I also don't see how you mean to use a
> function containing "srat" in its name from non-ACPI code.
> 
> Perhaps alongside numa_mode() (see the reply to the earlier patch)
> you want to have a set_numa_off() helper (name subject to
> improvement)?
> 

Yes, actually, I want a function to disable the numa when we encounter
any error in numa initialization. And we also so want to indicate the
numa off is caused by configuration error. So we had thought to use
numa_fw_fault to replace srat_bad.

> > --- a/xen/arch/x86/srat.c
> > +++ b/xen/arch/x86/srat.c
> > @@ -163,7 +163,7 @@ static __init void cutoff_node(int i, paddr_t start,
> paddr_t end)
> >     }
> >  }
> >
> > -static __init void bad_srat(void)
> > +__init void bad_srat(void)
> 
> Nit: Once again, when touching code, please take the opportunity and
> adjust style issues (here: __init and void want to change places).
> 

Thanks again!

> Jan


 


Rackspace

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