Message ID | 20210316010938.525657-1-mpe@ellerman.id.au (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
Series | powerpc/pseries: Only register vio drivers if vio bus exists | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/apply_patch | success | Successfully applied on branch powerpc/merge (91966823812efbd175f904599e5cf2a854b39809) |
snowpatch_ozlabs/build-ppc64le | success | Build succeeded |
snowpatch_ozlabs/build-ppc64be | success | Build succeeded |
snowpatch_ozlabs/build-ppc64e | success | Build succeeded |
snowpatch_ozlabs/build-pmac32 | success | Build succeeded |
snowpatch_ozlabs/checkpatch | warning | total: 0 errors, 1 warnings, 0 checks, 10 lines checked |
snowpatch_ozlabs/needsstable | warning | Please consider tagging this patch for stable! |
Dear Michael, Am 16.03.21 um 02:09 schrieb Michael Ellerman: > The vio bus is a fake bus, which we use on pseries LPARs (guests) to > discover devices provided by the hypervisor. There's no need or sense > in creating the vio bus on bare metal systems. > > Which is why commit 4336b9337824 ("powerpc/pseries: Make vio and > ibmebus initcalls pseries specific") made the initialisation of the > vio bus only happen in LPARs. > > However as a result of that commit we now see errors at boot on bare > metal systems: > > Driver 'hvc_console' was unable to register with bus_type 'vio' because the bus was not initialized. > Driver 'tpm_ibmvtpm' was unable to register with bus_type 'vio' because the bus was not initialized. > > This happens because those drivers are built-in, and are calling > vio_register_driver(). It in turn calls driver_register() with a > reference to vio_bus_type, but we haven't registered vio_bus_type with > the driver core. > > Fix it by also guarding vio_register_driver() with a check to see if > we are on pseries. > > Fixes: 4336b9337824 ("powerpc/pseries: Make vio and ibmebus initcalls pseries specific") > Reported-by: Paul Menzel <pmenzel@molgen.mpg.de> > Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> > --- > arch/powerpc/platforms/pseries/vio.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/arch/powerpc/platforms/pseries/vio.c b/arch/powerpc/platforms/pseries/vio.c > index 9cb4fc839fd5..429053d0402a 100644 > --- a/arch/powerpc/platforms/pseries/vio.c > +++ b/arch/powerpc/platforms/pseries/vio.c > @@ -1285,6 +1285,10 @@ static int vio_bus_remove(struct device *dev) > int __vio_register_driver(struct vio_driver *viodrv, struct module *owner, > const char *mod_name) > { > + // vio_bus_type is only initialised for pseries > + if (!machine_is(pseries)) > + return -ENODEV; > + > pr_debug("%s: driver %s registering\n", __func__, viodrv->name); > > /* fill in 'struct driver' fields */ Thank you. The errors are gone now. Tested-by: Paul Menzel <pmenzel@molgen.mpg.de> # IBM S822L (POWER8) As it fixes a commit from Linux 5.8, should it be tagged for the stable releases, or is it going to be picked up automatically due to the Fixes tag? Kind regards, Paul
On 3/15/21 6:09 PM, Michael Ellerman wrote: > The vio bus is a fake bus, which we use on pseries LPARs (guests) to > discover devices provided by the hypervisor. There's no need or sense > in creating the vio bus on bare metal systems. > > Which is why commit 4336b9337824 ("powerpc/pseries: Make vio and > ibmebus initcalls pseries specific") made the initialisation of the > vio bus only happen in LPARs. > > However as a result of that commit we now see errors at boot on bare > metal systems: > > Driver 'hvc_console' was unable to register with bus_type 'vio' because the bus was not initialized. > Driver 'tpm_ibmvtpm' was unable to register with bus_type 'vio' because the bus was not initialized. > > This happens because those drivers are built-in, and are calling > vio_register_driver(). It in turn calls driver_register() with a > reference to vio_bus_type, but we haven't registered vio_bus_type with > the driver core. > > Fix it by also guarding vio_register_driver() with a check to see if > we are on pseries. > > Fixes: 4336b9337824 ("powerpc/pseries: Make vio and ibmebus initcalls pseries specific") > Reported-by: Paul Menzel <pmenzel@molgen.mpg.de> > Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> > --- Reviewed-by: Tyrel Datwyler <tyreld@linux.ibm.com>
Paul Menzel <pmenzel@molgen.mpg.de> writes: > Am 16.03.21 um 02:09 schrieb Michael Ellerman: >> The vio bus is a fake bus, which we use on pseries LPARs (guests) to >> discover devices provided by the hypervisor. There's no need or sense >> in creating the vio bus on bare metal systems. >> >> Which is why commit 4336b9337824 ("powerpc/pseries: Make vio and >> ibmebus initcalls pseries specific") made the initialisation of the >> vio bus only happen in LPARs. >> >> However as a result of that commit we now see errors at boot on bare >> metal systems: >> >> Driver 'hvc_console' was unable to register with bus_type 'vio' because the bus was not initialized. >> Driver 'tpm_ibmvtpm' was unable to register with bus_type 'vio' because the bus was not initialized. >> >> This happens because those drivers are built-in, and are calling >> vio_register_driver(). It in turn calls driver_register() with a >> reference to vio_bus_type, but we haven't registered vio_bus_type with >> the driver core. >> >> Fix it by also guarding vio_register_driver() with a check to see if >> we are on pseries. >> >> Fixes: 4336b9337824 ("powerpc/pseries: Make vio and ibmebus initcalls pseries specific") >> Reported-by: Paul Menzel <pmenzel@molgen.mpg.de> >> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> >> --- >> arch/powerpc/platforms/pseries/vio.c | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/arch/powerpc/platforms/pseries/vio.c b/arch/powerpc/platforms/pseries/vio.c >> index 9cb4fc839fd5..429053d0402a 100644 >> --- a/arch/powerpc/platforms/pseries/vio.c >> +++ b/arch/powerpc/platforms/pseries/vio.c >> @@ -1285,6 +1285,10 @@ static int vio_bus_remove(struct device *dev) >> int __vio_register_driver(struct vio_driver *viodrv, struct module *owner, >> const char *mod_name) >> { >> + // vio_bus_type is only initialised for pseries >> + if (!machine_is(pseries)) >> + return -ENODEV; >> + >> pr_debug("%s: driver %s registering\n", __func__, viodrv->name); >> >> /* fill in 'struct driver' fields */ > > Thank you. The errors are gone now. > > Tested-by: Paul Menzel <pmenzel@molgen.mpg.de> # IBM S822L (POWER8) > > As it fixes a commit from Linux 5.8, should it be tagged for the stable > releases, or is it going to be picked up automatically due to the Fixes tag? It's not what I'd describe as a bad bug, so I'm not that inclined to tag it for stable. But given it has a Fixes tag, and is quite small, the bots will probably backport it automatically. cheers
On Tue, 16 Mar 2021 12:09:38 +1100, Michael Ellerman wrote: > The vio bus is a fake bus, which we use on pseries LPARs (guests) to > discover devices provided by the hypervisor. There's no need or sense > in creating the vio bus on bare metal systems. > > Which is why commit 4336b9337824 ("powerpc/pseries: Make vio and > ibmebus initcalls pseries specific") made the initialisation of the > vio bus only happen in LPARs. > > [...] Applied to powerpc/next. [1/1] powerpc/pseries: Only register vio drivers if vio bus exists https://git.kernel.org/powerpc/c/11d92156f7a862091009d7655d19c1e7de37fc7a cheers
diff --git a/arch/powerpc/platforms/pseries/vio.c b/arch/powerpc/platforms/pseries/vio.c index 9cb4fc839fd5..429053d0402a 100644 --- a/arch/powerpc/platforms/pseries/vio.c +++ b/arch/powerpc/platforms/pseries/vio.c @@ -1285,6 +1285,10 @@ static int vio_bus_remove(struct device *dev) int __vio_register_driver(struct vio_driver *viodrv, struct module *owner, const char *mod_name) { + // vio_bus_type is only initialised for pseries + if (!machine_is(pseries)) + return -ENODEV; + pr_debug("%s: driver %s registering\n", __func__, viodrv->name); /* fill in 'struct driver' fields */
The vio bus is a fake bus, which we use on pseries LPARs (guests) to discover devices provided by the hypervisor. There's no need or sense in creating the vio bus on bare metal systems. Which is why commit 4336b9337824 ("powerpc/pseries: Make vio and ibmebus initcalls pseries specific") made the initialisation of the vio bus only happen in LPARs. However as a result of that commit we now see errors at boot on bare metal systems: Driver 'hvc_console' was unable to register with bus_type 'vio' because the bus was not initialized. Driver 'tpm_ibmvtpm' was unable to register with bus_type 'vio' because the bus was not initialized. This happens because those drivers are built-in, and are calling vio_register_driver(). It in turn calls driver_register() with a reference to vio_bus_type, but we haven't registered vio_bus_type with the driver core. Fix it by also guarding vio_register_driver() with a check to see if we are on pseries. Fixes: 4336b9337824 ("powerpc/pseries: Make vio and ibmebus initcalls pseries specific") Reported-by: Paul Menzel <pmenzel@molgen.mpg.de> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> --- arch/powerpc/platforms/pseries/vio.c | 4 ++++ 1 file changed, 4 insertions(+)