Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs

2023-12-03 Thread Bruce Ashfield
On Thu, Nov 30, 2023 at 11:38 PM Liu, Yongxin  wrote:
>
> > -Original Message-
> > From: Gortmaker, Paul 
> > Sent: Friday, December 1, 2023 11:14
> > To: Liu, Yongxin 
> > Cc: Bruce Ashfield ; linux-
> > yo...@lists.yoctoproject.org
> > Subject: Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number
> > of CPUs
> >
> > [RE: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs]
> > On 30/11/2023 (Thu 21:43) Liu, Yongxin wrote:
> >
> > > > -Original Message-
> > > > From: Gortmaker, Paul 
> > > > Sent: Friday, December 1, 2023 10:27
> > > > To: Liu, Yongxin 
> > > > Cc: Bruce Ashfield ; linux-
> > > > yo...@lists.yoctoproject.org
> > > > Subject: Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for
> > > > number of CPUs
> > > >
> > > > [RE: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number
> > > > of CPUs] On 30/11/2023 (Thu 20:12) Liu, Yongxin wrote:
> > > >
> > > > > > -Original Message-
> > > > > > From: linux-yocto@lists.yoctoproject.org  > > > > > yo...@lists.yoctoproject.org> On Behalf Of Paul Gortmaker via
> > > > > > lists.yoctoproject.org
> > > > > > Sent: Friday, December 1, 2023 03:08
> > > > > > To: Bruce Ashfield 
> > > > > > Cc: linux-yocto@lists.yoctoproject.org
> > > > > > Subject: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for
> > > > > > number of CPUs
> > > > > >
> > > > > > From: Paul Gortmaker 
> > > > > >
> > > > > > The x86-64 BSP isn't quite the same as the "more specific" BSP
> > > > > > like a Beaglebone Black or the (now deleted) Edgerouter.  Where
> > > > > > we have exact hardware specifics for boards like those, the
> > > > > > x86-64 BSP is more of a "generic" thing used as the baseline
> > > > > > across an endless sea
> > > > of boards.
> > > > > >
> > > > > > To that end, this is somewhat a revert of commit bd77e1f904f6
> > > > > > ("bsp/intel-x86: change the supported maximum number of CPUs to
> > > > > > 512 in 64- bit bsp")
> > > > > >
> > > > > > It is great that a handful of people out there are using Yocto
> > > > > > on these huge server machines, but that doesn't reflect 99% of
> > > > > > the rest of us who continue to lean towards the original
> > > > > > "embedded theme" of
> > > > Yocto.
> > > > > >
> > > > > > That means a whole bunch of extra per-CPU jumping through hoops;
> > > > > > some can be mitigated by booting with "nr_cpus=4" (or whatever
> > > > > > the core count
> > > > > > is) but I guarantee largely nobody out there is doing that.
> > > > > >
> > > > > > Let those users with the crazy CPU count own that config
> > > > > > customization locally.  The default is 64 which still seems way
> > > > > > too large IMHO, but at least we are moving in the right direction.
> > > > >
> > > > >
> > > > > This intel-x86-64 BSP is a generic one used from mobile to server.
> > > > >
> > > > > Customers need to customize not only the CPU number config but
> > > > > also other configs, like, removing unused drivers or adding debug
> > options.
> > > > > From this point of view, there is no difference between 64 or 512.
> > >
> > > I changed 64 to 512. Because we have server machines with more than 64
> > CPU.
> > > I want the BSP support those machines by default.
> >
> > But you still miss the point.  It doesn't matter what you or any company
> > "want" in this case.  Like it or not, it is a shared resource and so the
> > defaults have to be what is good for Yocto project and not for *you*
> >
> > >
> > > >
> > > > So you've basically argued my case for me.  If changes are
> > > > inevitable, then why do we change the default?
> > > >
> > > > > But it changes the "rule" that intel-x86-64 works for all
> > > > > supported
> > > > platforms.
> > > > > We need to do extra work for servers with large CPU number.
> > > >
> > > > No.  There is no "rule" in Yocto like that.  That is nonsense
> > > > because there is no way Yocto can commit to "support" all the crazy
> > > > different
> > > > x86-64 variants out there.
> > >
> > >
> > > I think this "bsp/intel-x86" is used only by Wind River.
> > > So bsp/intel-x86 should work for all supported machines claimed by Wind
> > River.
> >
> > No. That is where you are dead wrong.  Wind River does not own Yocto.
> > Think for a minute.  A new Yocto user comes along and sees "intel-x86"
> > and because that name is so generic -- thinks "I'll build that for my old
> > PC."
>
> I have a question why we need bsp/intel-x86, because Yocto already has 
> bsp/intel-common and bsp/common-pc?
>
>
> >
> > > If we need to do some local change to support some machine. That's not
> > good.
> > > Because people usually build image with default configs and then
> > complain something doesn't work.
> >
> > Again, it is NOT the problem of the Yocto project what isn't good for YOU.
> > If you need EDAC and NUMA and 500+ CPU support, then make a proper BSP
> > with those settings and submit it as "bsp/mega-server-2000" or whatever.
>
> Then I think we should revise 

Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs

2023-11-30 Thread Yongxin Liu via lists.yoctoproject.org
> -Original Message-
> From: Gortmaker, Paul 
> Sent: Friday, December 1, 2023 11:14
> To: Liu, Yongxin 
> Cc: Bruce Ashfield ; linux-
> yo...@lists.yoctoproject.org
> Subject: Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number
> of CPUs
> 
> [RE: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs]
> On 30/11/2023 (Thu 21:43) Liu, Yongxin wrote:
> 
> > > -Original Message-
> > > From: Gortmaker, Paul 
> > > Sent: Friday, December 1, 2023 10:27
> > > To: Liu, Yongxin 
> > > Cc: Bruce Ashfield ; linux-
> > > yo...@lists.yoctoproject.org
> > > Subject: Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for
> > > number of CPUs
> > >
> > > [RE: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number
> > > of CPUs] On 30/11/2023 (Thu 20:12) Liu, Yongxin wrote:
> > >
> > > > > -Original Message-
> > > > > From: linux-yocto@lists.yoctoproject.org  > > > > yo...@lists.yoctoproject.org> On Behalf Of Paul Gortmaker via
> > > > > lists.yoctoproject.org
> > > > > Sent: Friday, December 1, 2023 03:08
> > > > > To: Bruce Ashfield 
> > > > > Cc: linux-yocto@lists.yoctoproject.org
> > > > > Subject: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for
> > > > > number of CPUs
> > > > >
> > > > > From: Paul Gortmaker 
> > > > >
> > > > > The x86-64 BSP isn't quite the same as the "more specific" BSP
> > > > > like a Beaglebone Black or the (now deleted) Edgerouter.  Where
> > > > > we have exact hardware specifics for boards like those, the
> > > > > x86-64 BSP is more of a "generic" thing used as the baseline
> > > > > across an endless sea
> > > of boards.
> > > > >
> > > > > To that end, this is somewhat a revert of commit bd77e1f904f6
> > > > > ("bsp/intel-x86: change the supported maximum number of CPUs to
> > > > > 512 in 64- bit bsp")
> > > > >
> > > > > It is great that a handful of people out there are using Yocto
> > > > > on these huge server machines, but that doesn't reflect 99% of
> > > > > the rest of us who continue to lean towards the original
> > > > > "embedded theme" of
> > > Yocto.
> > > > >
> > > > > That means a whole bunch of extra per-CPU jumping through hoops;
> > > > > some can be mitigated by booting with "nr_cpus=4" (or whatever
> > > > > the core count
> > > > > is) but I guarantee largely nobody out there is doing that.
> > > > >
> > > > > Let those users with the crazy CPU count own that config
> > > > > customization locally.  The default is 64 which still seems way
> > > > > too large IMHO, but at least we are moving in the right direction.
> > > >
> > > >
> > > > This intel-x86-64 BSP is a generic one used from mobile to server.
> > > >
> > > > Customers need to customize not only the CPU number config but
> > > > also other configs, like, removing unused drivers or adding debug
> options.
> > > > From this point of view, there is no difference between 64 or 512.
> >
> > I changed 64 to 512. Because we have server machines with more than 64
> CPU.
> > I want the BSP support those machines by default.
> 
> But you still miss the point.  It doesn't matter what you or any company
> "want" in this case.  Like it or not, it is a shared resource and so the
> defaults have to be what is good for Yocto project and not for *you*
> 
> >
> > >
> > > So you've basically argued my case for me.  If changes are
> > > inevitable, then why do we change the default?
> > >
> > > > But it changes the "rule" that intel-x86-64 works for all
> > > > supported
> > > platforms.
> > > > We need to do extra work for servers with large CPU number.
> > >
> > > No.  There is no "rule" in Yocto like that.  That is nonsense
> > > because there is no way Yocto can commit to "support" all the crazy
> > > different
> > > x86-64 variants out there.
> >
> >
> > I think this "bsp/intel-x86" is used only by Wind River.
> > So bsp/intel-x86 should work for all supported machines claimed by Wind
> River.
> 
> No. That is where you are dead wrong.  Wind River does not own Yocto.
> Think for a minute.  A new Yocto user comes along and sees "intel-x86"
> and because that name is so generic -- thinks "I'll build that for my old
> PC."

I have a question why we need bsp/intel-x86, because Yocto already has 
bsp/intel-common and bsp/common-pc?


> 
> > If we need to do some local change to support some machine. That's not
> good.
> > Because people usually build image with default configs and then
> complain something doesn't work.
> 
> Again, it is NOT the problem of the Yocto project what isn't good for YOU.
> If you need EDAC and NUMA and 500+ CPU support, then make a proper BSP
> with those settings and submit it as "bsp/mega-server-2000" or whatever.

Then I think we should revise bsp/intel-x86, because it has enabled many 
uncommon features by

intel-x86.scc:include features/intel-idxd/intel-idxd.scc
intel-x86.scc:include features/intel-uncore-frequency/intel-uncore-frequency.scc
intel-x86.scc:include features/intel-dptf/intel-dptf.scc
intel-x86.scc:include 

Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs

2023-11-30 Thread Paul Gortmaker via lists.yoctoproject.org
[RE: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs] On 
30/11/2023 (Thu 21:43) Liu, Yongxin wrote:

> > -Original Message-
> > From: Gortmaker, Paul 
> > Sent: Friday, December 1, 2023 10:27
> > To: Liu, Yongxin 
> > Cc: Bruce Ashfield ; linux-
> > yo...@lists.yoctoproject.org
> > Subject: Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number
> > of CPUs
> > 
> > [RE: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs]
> > On 30/11/2023 (Thu 20:12) Liu, Yongxin wrote:
> > 
> > > > -Original Message-
> > > > From: linux-yocto@lists.yoctoproject.org  > > > yo...@lists.yoctoproject.org> On Behalf Of Paul Gortmaker via
> > > > lists.yoctoproject.org
> > > > Sent: Friday, December 1, 2023 03:08
> > > > To: Bruce Ashfield 
> > > > Cc: linux-yocto@lists.yoctoproject.org
> > > > Subject: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for
> > > > number of CPUs
> > > >
> > > > From: Paul Gortmaker 
> > > >
> > > > The x86-64 BSP isn't quite the same as the "more specific" BSP like
> > > > a Beaglebone Black or the (now deleted) Edgerouter.  Where we have
> > > > exact hardware specifics for boards like those, the x86-64 BSP is
> > > > more of a "generic" thing used as the baseline across an endless sea
> > of boards.
> > > >
> > > > To that end, this is somewhat a revert of commit bd77e1f904f6
> > > > ("bsp/intel-x86: change the supported maximum number of CPUs to 512
> > > > in 64- bit bsp")
> > > >
> > > > It is great that a handful of people out there are using Yocto on
> > > > these huge server machines, but that doesn't reflect 99% of the rest
> > > > of us who continue to lean towards the original "embedded theme" of
> > Yocto.
> > > >
> > > > That means a whole bunch of extra per-CPU jumping through hoops;
> > > > some can be mitigated by booting with "nr_cpus=4" (or whatever the
> > > > core count
> > > > is) but I guarantee largely nobody out there is doing that.
> > > >
> > > > Let those users with the crazy CPU count own that config
> > > > customization locally.  The default is 64 which still seems way too
> > > > large IMHO, but at least we are moving in the right direction.
> > >
> > >
> > > This intel-x86-64 BSP is a generic one used from mobile to server.
> > >
> > > Customers need to customize not only the CPU number config but also
> > > other configs, like, removing unused drivers or adding debug options.
> > > From this point of view, there is no difference between 64 or 512.
> 
> I changed 64 to 512. Because we have server machines with more than 64 CPU.
> I want the BSP support those machines by default.

But you still miss the point.  It doesn't matter what you or any company
"want" in this case.  Like it or not, it is a shared resource and so the
defaults have to be what is good for Yocto project and not for *you*

> 
> > 
> > So you've basically argued my case for me.  If changes are inevitable,
> > then why do we change the default?
> > 
> > > But it changes the "rule" that intel-x86-64 works for all supported
> > platforms.
> > > We need to do extra work for servers with large CPU number.
> > 
> > No.  There is no "rule" in Yocto like that.  That is nonsense because
> > there is no way Yocto can commit to "support" all the crazy different
> > x86-64 variants out there.
> 
> 
> I think this "bsp/intel-x86" is used only by Wind River.
> So bsp/intel-x86 should work for all supported machines claimed by Wind River.

No. That is where you are dead wrong.  Wind River does not own Yocto.
Think for a minute.  A new Yocto user comes along and sees "intel-x86"
and because that name is so generic -- thinks "I'll build that for my old PC."

> If we need to do some local change to support some machine. That's not good.
> Because people usually build image with default configs and then complain 
> something doesn't work.

Again, it is NOT the problem of the Yocto project what isn't good for YOU.
If you need EDAC and NUMA and 500+ CPU support, then make a proper BSP
with those settings and submit it as "bsp/mega-server-2000" or whatever.

Don't just be using intel-x86 as a dumping ground for whatever random
setting you need today.  That isn't fair to all the other Yocto users
out there who might not even know who Wind River is.

Paul.
--

> 
> 
> Thanks,
> Yongxin
> 
> 
> > If a re-seller/integrator wants to take Yocto and tune it for platform XYZ
> > because there is customer demand and claim it is then "supported" by them,
> > then fine.  But then to expect the Yocto project to own that?  No.
> 
> 
> > 
> > Paul.
> > --
> > 
> > >
> > > Thanks,
> > > Yongxin
> > >
> > > >
> > > > Signed-off-by: Paul Gortmaker 
> > > > ---
> > > >  bsp/intel-x86/intel-x86-64.cfg | 3 ---
> > > >  1 file changed, 3 deletions(-)
> > > >
> > > > diff --git a/bsp/intel-x86/intel-x86-64.cfg
> > > > b/bsp/intel-x86/intel-x86- 64.cfg index 58b0fed637e8..da9bc7b57eca
> > > > 100644
> > > > --- a/bsp/intel-x86/intel-x86-64.cfg
> > > > +++ 

Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs

2023-11-30 Thread Yongxin Liu via lists.yoctoproject.org
> -Original Message-
> From: Gortmaker, Paul 
> Sent: Friday, December 1, 2023 10:27
> To: Liu, Yongxin 
> Cc: Bruce Ashfield ; linux-
> yo...@lists.yoctoproject.org
> Subject: Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number
> of CPUs
> 
> [RE: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs]
> On 30/11/2023 (Thu 20:12) Liu, Yongxin wrote:
> 
> > > -Original Message-
> > > From: linux-yocto@lists.yoctoproject.org  > > yo...@lists.yoctoproject.org> On Behalf Of Paul Gortmaker via
> > > lists.yoctoproject.org
> > > Sent: Friday, December 1, 2023 03:08
> > > To: Bruce Ashfield 
> > > Cc: linux-yocto@lists.yoctoproject.org
> > > Subject: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for
> > > number of CPUs
> > >
> > > From: Paul Gortmaker 
> > >
> > > The x86-64 BSP isn't quite the same as the "more specific" BSP like
> > > a Beaglebone Black or the (now deleted) Edgerouter.  Where we have
> > > exact hardware specifics for boards like those, the x86-64 BSP is
> > > more of a "generic" thing used as the baseline across an endless sea
> of boards.
> > >
> > > To that end, this is somewhat a revert of commit bd77e1f904f6
> > > ("bsp/intel-x86: change the supported maximum number of CPUs to 512
> > > in 64- bit bsp")
> > >
> > > It is great that a handful of people out there are using Yocto on
> > > these huge server machines, but that doesn't reflect 99% of the rest
> > > of us who continue to lean towards the original "embedded theme" of
> Yocto.
> > >
> > > That means a whole bunch of extra per-CPU jumping through hoops;
> > > some can be mitigated by booting with "nr_cpus=4" (or whatever the
> > > core count
> > > is) but I guarantee largely nobody out there is doing that.
> > >
> > > Let those users with the crazy CPU count own that config
> > > customization locally.  The default is 64 which still seems way too
> > > large IMHO, but at least we are moving in the right direction.
> >
> >
> > This intel-x86-64 BSP is a generic one used from mobile to server.
> >
> > Customers need to customize not only the CPU number config but also
> > other configs, like, removing unused drivers or adding debug options.
> > From this point of view, there is no difference between 64 or 512.

I changed 64 to 512. Because we have server machines with more than 64 CPU.
I want the BSP support those machines by default.

> 
> So you've basically argued my case for me.  If changes are inevitable,
> then why do we change the default?
> 
> > But it changes the "rule" that intel-x86-64 works for all supported
> platforms.
> > We need to do extra work for servers with large CPU number.
> 
> No.  There is no "rule" in Yocto like that.  That is nonsense because
> there is no way Yocto can commit to "support" all the crazy different
> x86-64 variants out there.


I think this "bsp/intel-x86" is used only by Wind River.
So bsp/intel-x86 should work for all supported machines claimed by Wind River.
If we need to do some local change to support some machine. That's not good.
Because people usually build image with default configs and then complain 
something doesn't work.


Thanks,
Yongxin


> If a re-seller/integrator wants to take Yocto and tune it for platform XYZ
> because there is customer demand and claim it is then "supported" by them,
> then fine.  But then to expect the Yocto project to own that?  No.


> 
> Paul.
> --
> 
> >
> > Thanks,
> > Yongxin
> >
> > >
> > > Signed-off-by: Paul Gortmaker 
> > > ---
> > >  bsp/intel-x86/intel-x86-64.cfg | 3 ---
> > >  1 file changed, 3 deletions(-)
> > >
> > > diff --git a/bsp/intel-x86/intel-x86-64.cfg
> > > b/bsp/intel-x86/intel-x86- 64.cfg index 58b0fed637e8..da9bc7b57eca
> > > 100644
> > > --- a/bsp/intel-x86/intel-x86-64.cfg
> > > +++ b/bsp/intel-x86/intel-x86-64.cfg
> > > @@ -31,6 +31,3 @@ CONFIG_CRYPTO_DEV_QAT_DH895xCCVF=m
> > >
> > >  # x86 CPU resource control support
> > >  CONFIG_X86_CPU_RESCTRL=y
> > > -
> > > -# Processor type and features
> > > -CONFIG_NR_CPUS=512
> > > --
> > > 2.40.0
> >

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#13345): 
https://lists.yoctoproject.org/g/linux-yocto/message/13345
Mute This Topic: https://lists.yoctoproject.org/mt/102900654/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs

2023-11-30 Thread Paul Gortmaker via lists.yoctoproject.org
[RE: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs] On 
30/11/2023 (Thu 20:12) Liu, Yongxin wrote:

> > -Original Message-
> > From: linux-yocto@lists.yoctoproject.org  > yo...@lists.yoctoproject.org> On Behalf Of Paul Gortmaker via
> > lists.yoctoproject.org
> > Sent: Friday, December 1, 2023 03:08
> > To: Bruce Ashfield 
> > Cc: linux-yocto@lists.yoctoproject.org
> > Subject: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of
> > CPUs
> > 
> > From: Paul Gortmaker 
> > 
> > The x86-64 BSP isn't quite the same as the "more specific" BSP like a
> > Beaglebone Black or the (now deleted) Edgerouter.  Where we have exact
> > hardware specifics for boards like those, the x86-64 BSP is more of a
> > "generic" thing used as the baseline across an endless sea of boards.
> > 
> > To that end, this is somewhat a revert of commit bd77e1f904f6
> > ("bsp/intel-x86: change the supported maximum number of CPUs to 512 in 64-
> > bit bsp")
> > 
> > It is great that a handful of people out there are using Yocto on these
> > huge server machines, but that doesn't reflect 99% of the rest of us who
> > continue to lean towards the original "embedded theme" of Yocto.
> > 
> > That means a whole bunch of extra per-CPU jumping through hoops; some can
> > be mitigated by booting with "nr_cpus=4" (or whatever the core count
> > is) but I guarantee largely nobody out there is doing that.
> > 
> > Let those users with the crazy CPU count own that config customization
> > locally.  The default is 64 which still seems way too large IMHO, but at
> > least we are moving in the right direction.
> 
> 
> This intel-x86-64 BSP is a generic one used from mobile to server.
> 
> Customers need to customize not only the CPU number config but also other 
> configs,
> like, removing unused drivers or adding debug options.
> From this point of view, there is no difference between 64 or 512.

So you've basically argued my case for me.  If changes are inevitable,
then why do we change the default?

> But it changes the "rule" that intel-x86-64 works for all supported platforms.
> We need to do extra work for servers with large CPU number.

No.  There is no "rule" in Yocto like that.  That is nonsense because
there is no way Yocto can commit to "support" all the crazy different
x86-64 variants out there.

If a re-seller/integrator wants to take Yocto and tune it for platform
XYZ because there is customer demand and claim it is then "supported" by
them, then fine.  But then to expect the Yocto project to own that?  No.

Paul.
--

> 
> Thanks,
> Yongxin
> 
> > 
> > Signed-off-by: Paul Gortmaker 
> > ---
> >  bsp/intel-x86/intel-x86-64.cfg | 3 ---
> >  1 file changed, 3 deletions(-)
> > 
> > diff --git a/bsp/intel-x86/intel-x86-64.cfg b/bsp/intel-x86/intel-x86-
> > 64.cfg index 58b0fed637e8..da9bc7b57eca 100644
> > --- a/bsp/intel-x86/intel-x86-64.cfg
> > +++ b/bsp/intel-x86/intel-x86-64.cfg
> > @@ -31,6 +31,3 @@ CONFIG_CRYPTO_DEV_QAT_DH895xCCVF=m
> > 
> >  # x86 CPU resource control support
> >  CONFIG_X86_CPU_RESCTRL=y
> > -
> > -# Processor type and features
> > -CONFIG_NR_CPUS=512
> > --
> > 2.40.0
> 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#13343): 
https://lists.yoctoproject.org/g/linux-yocto/message/13343
Mute This Topic: https://lists.yoctoproject.org/mt/102900654/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs

2023-11-30 Thread Yongxin Liu via lists.yoctoproject.org
> -Original Message-
> From: linux-yocto@lists.yoctoproject.org  yo...@lists.yoctoproject.org> On Behalf Of Paul Gortmaker via
> lists.yoctoproject.org
> Sent: Friday, December 1, 2023 03:08
> To: Bruce Ashfield 
> Cc: linux-yocto@lists.yoctoproject.org
> Subject: [linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of
> CPUs
> 
> From: Paul Gortmaker 
> 
> The x86-64 BSP isn't quite the same as the "more specific" BSP like a
> Beaglebone Black or the (now deleted) Edgerouter.  Where we have exact
> hardware specifics for boards like those, the x86-64 BSP is more of a
> "generic" thing used as the baseline across an endless sea of boards.
> 
> To that end, this is somewhat a revert of commit bd77e1f904f6
> ("bsp/intel-x86: change the supported maximum number of CPUs to 512 in 64-
> bit bsp")
> 
> It is great that a handful of people out there are using Yocto on these
> huge server machines, but that doesn't reflect 99% of the rest of us who
> continue to lean towards the original "embedded theme" of Yocto.
> 
> That means a whole bunch of extra per-CPU jumping through hoops; some can
> be mitigated by booting with "nr_cpus=4" (or whatever the core count
> is) but I guarantee largely nobody out there is doing that.
> 
> Let those users with the crazy CPU count own that config customization
> locally.  The default is 64 which still seems way too large IMHO, but at
> least we are moving in the right direction.


This intel-x86-64 BSP is a generic one used from mobile to server.

Customers need to customize not only the CPU number config but also other 
configs,
like, removing unused drivers or adding debug options.
>From this point of view, there is no difference between 64 or 512.

But it changes the "rule" that intel-x86-64 works for all supported platforms.
We need to do extra work for servers with large CPU number.

Thanks,
Yongxin

> 
> Signed-off-by: Paul Gortmaker 
> ---
>  bsp/intel-x86/intel-x86-64.cfg | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/bsp/intel-x86/intel-x86-64.cfg b/bsp/intel-x86/intel-x86-
> 64.cfg index 58b0fed637e8..da9bc7b57eca 100644
> --- a/bsp/intel-x86/intel-x86-64.cfg
> +++ b/bsp/intel-x86/intel-x86-64.cfg
> @@ -31,6 +31,3 @@ CONFIG_CRYPTO_DEV_QAT_DH895xCCVF=m
> 
>  # x86 CPU resource control support
>  CONFIG_X86_CPU_RESCTRL=y
> -
> -# Processor type and features
> -CONFIG_NR_CPUS=512
> --
> 2.40.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#13342): 
https://lists.yoctoproject.org/g/linux-yocto/message/13342
Mute This Topic: https://lists.yoctoproject.org/mt/102900654/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[linux-yocto] [PATCH 4/5] x86-64: use the defaults for number of CPUs

2023-11-30 Thread Paul Gortmaker via lists.yoctoproject.org
From: Paul Gortmaker 

The x86-64 BSP isn't quite the same as the "more specific" BSP like a
Beaglebone Black or the (now deleted) Edgerouter.  Where we have exact
hardware specifics for boards like those, the x86-64 BSP is more of a
"generic" thing used as the baseline across an endless sea of boards.

To that end, this is somewhat a revert of commit bd77e1f904f6
("bsp/intel-x86: change the supported maximum number of CPUs to 512 in 64-bit 
bsp")

It is great that a handful of people out there are using Yocto on these
huge server machines, but that doesn't reflect 99% of the rest of us who
continue to lean towards the original "embedded theme" of Yocto.

That means a whole bunch of extra per-CPU jumping through hoops; some
can be mitigated by booting with "nr_cpus=4" (or whatever the core count
is) but I guarantee largely nobody out there is doing that.

Let those users with the crazy CPU count own that config customization
locally.  The default is 64 which still seems way too large IMHO, but
at least we are moving in the right direction.

Signed-off-by: Paul Gortmaker 
---
 bsp/intel-x86/intel-x86-64.cfg | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/bsp/intel-x86/intel-x86-64.cfg b/bsp/intel-x86/intel-x86-64.cfg
index 58b0fed637e8..da9bc7b57eca 100644
--- a/bsp/intel-x86/intel-x86-64.cfg
+++ b/bsp/intel-x86/intel-x86-64.cfg
@@ -31,6 +31,3 @@ CONFIG_CRYPTO_DEV_QAT_DH895xCCVF=m
 
 # x86 CPU resource control support
 CONFIG_X86_CPU_RESCTRL=y
-
-# Processor type and features
-CONFIG_NR_CPUS=512
-- 
2.40.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#13339): 
https://lists.yoctoproject.org/g/linux-yocto/message/13339
Mute This Topic: https://lists.yoctoproject.org/mt/102900654/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-