Re: [PATCH 0/5] x86/platform/uv: Move UV procfs leaves to sysfs

From: Mike Travis <mike.travis@hpe.com>
To: Hans de Goede <hdegoede@redhat.com>,
	Justin Ernst <justin.ernst@hpe.com>,
	Mark Gross <mgross@linux.intel.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
	x86@kernel.org, Steve Wahl <steve.wahl@hpe.com>
Cc: Dimitri Sivanich <dimitri.sivanich@hpe.com>,
	Russ Anderson <russ.anderson@hpe.com>,
	"H. Peter Anvin" <hpa@zytor.com>,
	Mauro Carvalho Chehab <mchehab+huawei@kernel.org>,
	Suzuki K Poulose <suzuki.poulose@arm.com>,
	Ilya Dryomov <idryomov@gmail.com>,
	linux-kernel@vger.kernel.org,
	platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH 0/5] x86/platform/uv: Move UV procfs leaves to sysfs
Date: Fri, 27 Nov 2020 06:41:32 -0800	[thread overview]
Message-ID: <2acf6156-7bae-7817-4cc6-902798c7af4e@hpe.com> (raw)
In-Reply-To: <815f9d40-6c17-9bd3-f3c8-626d007b8597@redhat.com>

On 11/26/2020 2:44 AM, Hans de Goede wrote:
> Hi,
> 
> On 11/25/20 6:29 PM, Mike Travis wrote:
>>
>> Duplicate the current UV procfs leaves to the uv_sysfs driver so they show
>> up under /sys/firmware/sgi_uv.  Show a 'deprecated' warning message if
>> any of the old /proc/sgi_uv leaves are used.
>>
>> These patches depend on the prior set sent by Justin Ernst <justin.ernst@hpe.com>
>> 	x86/platform/uv: Remove existing /sys/firmware/sgi_uv/ interface
>> 	x86/platform/uv: Add and export uv_bios_* functions
>> 	x86/platform/uv: Add new uv_sysfs platform driver
>> 	x86/platform/uv: Update ABI documentation of /sys/firmware/sgi_uv/
>> 	x86/platform/uv: Update MAINTAINERS for uv_sysfs driver
>>
>> Mike Travis (5):
>>    x86/platform/uv: Add kernel interfaces for obtaining system info.
>>    x86/platform/uv: Add sysfs leaves to replace those in procfs
>>    x86/platform/uv: Add sysfs hubless leaves
>>    x86/platform/uv: Add deprecated messages to /proc info leaves
>>    x86/platform/uv: Update sysfs document file
>>
>>   .../ABI/testing/sysfs-firmware-sgi_uv         | 16 +++++
>>   arch/x86/include/asm/uv/bios.h                |  2 +
>>   arch/x86/kernel/apic/x2apic_uv_x.c            | 38 +++++++++-
>>   drivers/platform/x86/uv_sysfs.c               | 70 ++++++++++++++++++-
>>   4 files changed, 123 insertions(+), 3 deletions(-)
> 
> This series depends on the:
> 
> [PATCH v3 0/5] x86/platform/uv: Add uv_sysfs platform driver
> 
> Series, next time when such a thing is the case, please add a note
> about this to the cover letter.
> 
> Like with the "[PATCH v3 0/5] x86/platform/uv: Add uv_sysfs platform driver" series,
> this series too should be merged in its entirety through the x86/tip tree (once the
> other series is merged). Please also add this info to the cover letter.
> 
> I've one remark to patch 4 (which I send in a reply to that). So a v2 is going
> to be necessary. Please include the above bits in the v2 cover-letter.

I was just about to send a notice that Justin changed the underlying 
patch that these depend on.  (I was gone late Wednesday and all day 
yesterday).  That will be done later today, thanks!

> 
> Regards,
> 
> Hans
> 