RE: [PATCH v2 0/5] Add uv_sysfs platform driver

[Ernst, Justin]

Hans,
Thank you for your Ack of my patch set.

I've found a couple bugs that need fixing:

1. In my re-introduction of /sys/firmware/sgi_uv/coherence_id, I failed to export the associated sn_coherency_id variable, causing the build to fail when UV_SYSFS=m

2. A null pointer dereference in drivers/platform/x86/uv_sysfs.c:uv_ports_exit() caused by calling kobject_put() on an out of range index value.

I will be resubmitting the patch series shortly as v3.

I apologize for the hiccup.

Thanks,
Justin

> -----Original Message-----
> From: Hans de Goede [mailto:hdegoede@xxxxxxxxxx]
> Sent: Tuesday, November 24, 2020 5:30 AM
> To: Ernst, Justin <justin.ernst@xxxxxxx>; Borislav Petkov <bp@xxxxxxxxx>; Ingo Molnar
> <mingo@xxxxxxxxxx>; Mark Gross <mgross@xxxxxxxxxxxxxxx>; Thomas Gleixner <tglx@xxxxxxxxxxxxx>; Wahl,
> Steve <steve.wahl@xxxxxxx>; x86@xxxxxxxxxx
> Cc: Andy Shevchenko <andy@xxxxxxxxxxxxx>; Darren Hart <dvhart@xxxxxxxxxxxxx>; Sivanich, Dimitri
> <dimitri.sivanich@xxxxxxx>; H . Peter Anvin <hpa@xxxxxxxxx>; Anderson, Russ <russ.anderson@xxxxxxx>;
> linux-kernel@xxxxxxxxxxxxxxx; platform-driver-x86@xxxxxxxxxxxxxxx; Cezary Rojewski
> <cezary.rojewski@xxxxxxxxx>; Ilya Dryomov <idryomov@xxxxxxxxx>; Jonathan Cameron
> <Jonathan.Cameron@xxxxxxxxxx>; Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx>; Vaibhav Jain
> <vaibhav@xxxxxxxxxxxxx>
> Subject: Re: [PATCH v2 0/5] Add uv_sysfs platform driver
> 
> Hi,
> 
> Quick self intro for the x86/tip maintainers: I have take over
> drivers/platform/x86 maintainership from Andy.
> 
> On 11/18/20 5:47 PM, Justin Ernst wrote:
> > Introduce a new platform driver to gather topology information from UV systems
> > and expose that information via a sysfs interface at /sys/firmware/sgi_uv/.
> >
> > This is version 2 with these changes since version 1:
> >
> >  * Re-introduced /sys/firmware/sgi_uv/coherence_id file in the new driver after
> > 	removing it in Patch 1/5. This keeps the userspace API unbroken.
> >
> > Justin Ernst (5):
> >   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>
> >  .../ABI/testing/sysfs-firmware-sgi_uv         | 141 ++-
> >  MAINTAINERS                                   |   6 +
> >  arch/x86/include/asm/uv/bios.h                |  49 +
> >  arch/x86/include/asm/uv/uv_geo.h              | 103 +++
> >  arch/x86/platform/uv/Makefile                 |   2 +-
> >  arch/x86/platform/uv/bios_uv.c                |  54 ++
> >  arch/x86/platform/uv/uv_sysfs.c               |  63 --
> >  drivers/platform/x86/Kconfig                  |  11 +
> >  drivers/platform/x86/Makefile                 |   3 +
> >  drivers/platform/x86/uv_sysfs.c               | 862 ++++++++++++++++++
> >  10 files changed, 1216 insertions(+), 78 deletions(-)
> >  create mode 100644 arch/x86/include/asm/uv/uv_geo.h
> >  delete mode 100644 arch/x86/platform/uv/uv_sysfs.c
> >  create mode 100644 drivers/platform/x86/uv_sysfs.c
> 
> So this touches files under both arch/x86/ and drivers/platform/x86/ ,
> I believe this is best merged through the x86/tip tree and I don't
> expect conflicts for the drivers/platform/x86/{Kconfig,Makefile} changes.
> 
> So here is my ack for merging this series through the x86/tip tree:
> 
> Acked-by: Hans de Goede <hdegoede@xxxxxxxxxx>
> 
> Regards,
> 
> Hans
>