RE: [PATCH v2 0/3] topology/sysfs: only export used sysfs attributes

["Song Bao Hua (Barry Song)" <song.bao.hua@hisilicon.com>]

> -----Original Message-----
> From: Brice Goglin [mailto:Brice.Goglin@inria.fr]
> Sent: Saturday, December 4, 2021 10:48 PM
> To: Song Bao Hua (Barry Song) <song.bao.hua@hisilicon.com>; Greg Kroah-Hartman
> <gregkh@linuxfoundation.org>; Heiko Carstens <hca@linux.ibm.com>
> Cc: Peter Zijlstra <peterz@infradead.org>; Rafael J . Wysocki
> <rafael@kernel.org>; Andrew Morton <akpm@linux-foundation.org>;
> linux-kernel@vger.kernel.org; Jonathan Cameron <jonathan.cameron@huawei.com>;
> Len Brown <len.brown@intel.com>; Thomas Richter <tmricht@linux.ibm.com>; Ian
> Rogers <irogers@google.com>
> Subject: Re: [PATCH v2 0/3] topology/sysfs: only export used sysfs attributes
> 
> Le 04/12/2021 à 10:07, Song Bao Hua (Barry Song) a écrit :
> >
> > Could you give me one minute?
> >
> > +Brice
> >
> > I'd like to hear some feedbacks from hwloc if there
> > is a chance to break userspace and if userspace depends
> > on the existence of sysfs even though the topology
> > doesn't exist.
> >
> > If no, I feel it is safe to take.
> >
> 
> Hello
> 
> If the question is whether hwloc *requires* cluster/die/book/drawer
> sysfs files to exist, then the answer is no. We have to support old
> kernels without those files anyway.

Thanks for clarification, Brice. I saw hwloc is reading sysfs
drawer, book and cluster. But since it doesn't assume their
existence in sysfs, I have more confidence on this patchset
now.

Greg, please ignore my comment then :-)

> 
> Brice
> 
> 

Thanks
Barry