* [PATCH 10/12] tpm: fix up the tpm_class shutdown_pre pointer when created [not found] <20230313181843.1207845-1-gregkh@linuxfoundation.org> @ 2023-03-13 18:18 ` Greg Kroah-Hartman 2023-03-14 11:09 ` Jarkko Sakkinen 0 siblings, 1 reply; 4+ messages in thread From: Greg Kroah-Hartman @ 2023-03-13 18:18 UTC (permalink / raw) To: linux-kernel Cc: rafael, Greg Kroah-Hartman, Peter Huewe, Jarkko Sakkinen, Jason Gunthorpe, linux-integrity Do not wait until long after the struct class has been created to set the shutdown_pre pointer for the tpm_class, assign it right away. This is the only in-kernel offender that tries to modify the device->class pointer contents after it has been assigned to a device, so fix that up by doing the function pointer assignment before it is matched with the device. Because of this, the patch should go through the driver core tree to allow later changes to struct device to be possible. Cc: Peter Huewe <peterhuewe@gmx.de> Cc: Jarkko Sakkinen <jarkko@kernel.org> Cc: Jason Gunthorpe <jgg@ziepe.ca> Cc: linux-integrity@vger.kernel.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> --- drivers/char/tpm/tpm-chip.c | 3 +-- drivers/char/tpm/tpm-interface.c | 1 + drivers/char/tpm/tpm.h | 1 + 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/char/tpm/tpm-chip.c b/drivers/char/tpm/tpm-chip.c index b99f55f2d4fd..7c444209a256 100644 --- a/drivers/char/tpm/tpm-chip.c +++ b/drivers/char/tpm/tpm-chip.c @@ -282,7 +282,7 @@ static void tpm_dev_release(struct device *dev) * * Return: always 0 (i.e. success) */ -static int tpm_class_shutdown(struct device *dev) +int tpm_class_shutdown(struct device *dev) { struct tpm_chip *chip = container_of(dev, struct tpm_chip, dev); @@ -337,7 +337,6 @@ struct tpm_chip *tpm_chip_alloc(struct device *pdev, device_initialize(&chip->dev); chip->dev.class = tpm_class; - chip->dev.class->shutdown_pre = tpm_class_shutdown; chip->dev.release = tpm_dev_release; chip->dev.parent = pdev; chip->dev.groups = chip->groups; diff --git a/drivers/char/tpm/tpm-interface.c b/drivers/char/tpm/tpm-interface.c index 8763c820d1f8..43e23a04433a 100644 --- a/drivers/char/tpm/tpm-interface.c +++ b/drivers/char/tpm/tpm-interface.c @@ -467,6 +467,7 @@ static int __init tpm_init(void) int rc; tpm_class = class_create("tpm"); + tpm_class->shutdown_pre = tpm_class_shutdown; if (IS_ERR(tpm_class)) { pr_err("couldn't create tpm class\n"); return PTR_ERR(tpm_class); diff --git a/drivers/char/tpm/tpm.h b/drivers/char/tpm/tpm.h index 24ee4e1cc452..a45eb39db0c4 100644 --- a/drivers/char/tpm/tpm.h +++ b/drivers/char/tpm/tpm.h @@ -183,6 +183,7 @@ int tpm1_get_pcr_allocation(struct tpm_chip *chip); unsigned long tpm_calc_ordinal_duration(struct tpm_chip *chip, u32 ordinal); int tpm_pm_suspend(struct device *dev); int tpm_pm_resume(struct device *dev); +int tpm_class_shutdown(struct device *dev); static inline void tpm_msleep(unsigned int delay_msec) { -- 2.39.2 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 10/12] tpm: fix up the tpm_class shutdown_pre pointer when created 2023-03-13 18:18 ` [PATCH 10/12] tpm: fix up the tpm_class shutdown_pre pointer when created Greg Kroah-Hartman @ 2023-03-14 11:09 ` Jarkko Sakkinen 2023-03-14 12:53 ` Greg Kroah-Hartman 0 siblings, 1 reply; 4+ messages in thread From: Jarkko Sakkinen @ 2023-03-14 11:09 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: linux-kernel, rafael, Peter Huewe, Jason Gunthorpe, linux-integrity On Mon, Mar 13, 2023 at 07:18:41PM +0100, Greg Kroah-Hartman wrote: > Do not wait until long after the struct class has been created to set > the shutdown_pre pointer for the tpm_class, assign it right away. > > This is the only in-kernel offender that tries to modify the > device->class pointer contents after it has been assigned to a device, > so fix that up by doing the function pointer assignment before it is > matched with the device. Because of this, the patch should go through > the driver core tree to allow later changes to struct device to be > possible. > > Cc: Peter Huewe <peterhuewe@gmx.de> > Cc: Jarkko Sakkinen <jarkko@kernel.org> > Cc: Jason Gunthorpe <jgg@ziepe.ca> > Cc: linux-integrity@vger.kernel.org > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > --- > drivers/char/tpm/tpm-chip.c | 3 +-- > drivers/char/tpm/tpm-interface.c | 1 + > drivers/char/tpm/tpm.h | 1 + > 3 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/char/tpm/tpm-chip.c b/drivers/char/tpm/tpm-chip.c > index b99f55f2d4fd..7c444209a256 100644 > --- a/drivers/char/tpm/tpm-chip.c > +++ b/drivers/char/tpm/tpm-chip.c > @@ -282,7 +282,7 @@ static void tpm_dev_release(struct device *dev) > * > * Return: always 0 (i.e. success) > */ > -static int tpm_class_shutdown(struct device *dev) > +int tpm_class_shutdown(struct device *dev) > { > struct tpm_chip *chip = container_of(dev, struct tpm_chip, dev); > > @@ -337,7 +337,6 @@ struct tpm_chip *tpm_chip_alloc(struct device *pdev, > device_initialize(&chip->dev); > > chip->dev.class = tpm_class; > - chip->dev.class->shutdown_pre = tpm_class_shutdown; > chip->dev.release = tpm_dev_release; > chip->dev.parent = pdev; > chip->dev.groups = chip->groups; > diff --git a/drivers/char/tpm/tpm-interface.c b/drivers/char/tpm/tpm-interface.c > index 8763c820d1f8..43e23a04433a 100644 > --- a/drivers/char/tpm/tpm-interface.c > +++ b/drivers/char/tpm/tpm-interface.c > @@ -467,6 +467,7 @@ static int __init tpm_init(void) > int rc; > > tpm_class = class_create("tpm"); > + tpm_class->shutdown_pre = tpm_class_shutdown; > if (IS_ERR(tpm_class)) { > pr_err("couldn't create tpm class\n"); > return PTR_ERR(tpm_class); > diff --git a/drivers/char/tpm/tpm.h b/drivers/char/tpm/tpm.h > index 24ee4e1cc452..a45eb39db0c4 100644 > --- a/drivers/char/tpm/tpm.h > +++ b/drivers/char/tpm/tpm.h > @@ -183,6 +183,7 @@ int tpm1_get_pcr_allocation(struct tpm_chip *chip); > unsigned long tpm_calc_ordinal_duration(struct tpm_chip *chip, u32 ordinal); > int tpm_pm_suspend(struct device *dev); > int tpm_pm_resume(struct device *dev); > +int tpm_class_shutdown(struct device *dev); > > static inline void tpm_msleep(unsigned int delay_msec) > { > -- > 2.39.2 > Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> Are you going to pick this? BR, Jarkko ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 10/12] tpm: fix up the tpm_class shutdown_pre pointer when created 2023-03-14 11:09 ` Jarkko Sakkinen @ 2023-03-14 12:53 ` Greg Kroah-Hartman 2023-03-14 12:57 ` Jarkko Sakkinen 0 siblings, 1 reply; 4+ messages in thread From: Greg Kroah-Hartman @ 2023-03-14 12:53 UTC (permalink / raw) To: Jarkko Sakkinen Cc: linux-kernel, rafael, Peter Huewe, Jason Gunthorpe, linux-integrity On Tue, Mar 14, 2023 at 01:09:04PM +0200, Jarkko Sakkinen wrote: > On Mon, Mar 13, 2023 at 07:18:41PM +0100, Greg Kroah-Hartman wrote: > > Do not wait until long after the struct class has been created to set > > the shutdown_pre pointer for the tpm_class, assign it right away. > > > > This is the only in-kernel offender that tries to modify the > > device->class pointer contents after it has been assigned to a device, > > so fix that up by doing the function pointer assignment before it is > > matched with the device. Because of this, the patch should go through > > the driver core tree to allow later changes to struct device to be > > possible. > > > > Cc: Peter Huewe <peterhuewe@gmx.de> > > Cc: Jarkko Sakkinen <jarkko@kernel.org> > > Cc: Jason Gunthorpe <jgg@ziepe.ca> > > Cc: linux-integrity@vger.kernel.org > > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > --- > > drivers/char/tpm/tpm-chip.c | 3 +-- > > drivers/char/tpm/tpm-interface.c | 1 + > > drivers/char/tpm/tpm.h | 1 + > > 3 files changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/char/tpm/tpm-chip.c b/drivers/char/tpm/tpm-chip.c > > index b99f55f2d4fd..7c444209a256 100644 > > --- a/drivers/char/tpm/tpm-chip.c > > +++ b/drivers/char/tpm/tpm-chip.c > > @@ -282,7 +282,7 @@ static void tpm_dev_release(struct device *dev) > > * > > * Return: always 0 (i.e. success) > > */ > > -static int tpm_class_shutdown(struct device *dev) > > +int tpm_class_shutdown(struct device *dev) > > { > > struct tpm_chip *chip = container_of(dev, struct tpm_chip, dev); > > > > @@ -337,7 +337,6 @@ struct tpm_chip *tpm_chip_alloc(struct device *pdev, > > device_initialize(&chip->dev); > > > > chip->dev.class = tpm_class; > > - chip->dev.class->shutdown_pre = tpm_class_shutdown; > > chip->dev.release = tpm_dev_release; > > chip->dev.parent = pdev; > > chip->dev.groups = chip->groups; > > diff --git a/drivers/char/tpm/tpm-interface.c b/drivers/char/tpm/tpm-interface.c > > index 8763c820d1f8..43e23a04433a 100644 > > --- a/drivers/char/tpm/tpm-interface.c > > +++ b/drivers/char/tpm/tpm-interface.c > > @@ -467,6 +467,7 @@ static int __init tpm_init(void) > > int rc; > > > > tpm_class = class_create("tpm"); > > + tpm_class->shutdown_pre = tpm_class_shutdown; > > if (IS_ERR(tpm_class)) { > > pr_err("couldn't create tpm class\n"); > > return PTR_ERR(tpm_class); > > diff --git a/drivers/char/tpm/tpm.h b/drivers/char/tpm/tpm.h > > index 24ee4e1cc452..a45eb39db0c4 100644 > > --- a/drivers/char/tpm/tpm.h > > +++ b/drivers/char/tpm/tpm.h > > @@ -183,6 +183,7 @@ int tpm1_get_pcr_allocation(struct tpm_chip *chip); > > unsigned long tpm_calc_ordinal_duration(struct tpm_chip *chip, u32 ordinal); > > int tpm_pm_suspend(struct device *dev); > > int tpm_pm_resume(struct device *dev); > > +int tpm_class_shutdown(struct device *dev); > > > > static inline void tpm_msleep(unsigned int delay_msec) > > { > > -- > > 2.39.2 > > > > Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> > > Are you going to pick this? Yes, as the changelog text said, I'd like to take it through my tree, thanks! greg k-h ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 10/12] tpm: fix up the tpm_class shutdown_pre pointer when created 2023-03-14 12:53 ` Greg Kroah-Hartman @ 2023-03-14 12:57 ` Jarkko Sakkinen 0 siblings, 0 replies; 4+ messages in thread From: Jarkko Sakkinen @ 2023-03-14 12:57 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: linux-kernel, rafael, Peter Huewe, Jason Gunthorpe, linux-integrity On Tue, Mar 14, 2023 at 01:53:59PM +0100, Greg Kroah-Hartman wrote: > On Tue, Mar 14, 2023 at 01:09:04PM +0200, Jarkko Sakkinen wrote: > > On Mon, Mar 13, 2023 at 07:18:41PM +0100, Greg Kroah-Hartman wrote: > > > Do not wait until long after the struct class has been created to set > > > the shutdown_pre pointer for the tpm_class, assign it right away. > > > > > > This is the only in-kernel offender that tries to modify the > > > device->class pointer contents after it has been assigned to a device, > > > so fix that up by doing the function pointer assignment before it is > > > matched with the device. Because of this, the patch should go through > > > the driver core tree to allow later changes to struct device to be > > > possible. > > > > > > Cc: Peter Huewe <peterhuewe@gmx.de> > > > Cc: Jarkko Sakkinen <jarkko@kernel.org> > > > Cc: Jason Gunthorpe <jgg@ziepe.ca> > > > Cc: linux-integrity@vger.kernel.org > > > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > --- > > > drivers/char/tpm/tpm-chip.c | 3 +-- > > > drivers/char/tpm/tpm-interface.c | 1 + > > > drivers/char/tpm/tpm.h | 1 + > > > 3 files changed, 3 insertions(+), 2 deletions(-) > > > > > > diff --git a/drivers/char/tpm/tpm-chip.c b/drivers/char/tpm/tpm-chip.c > > > index b99f55f2d4fd..7c444209a256 100644 > > > --- a/drivers/char/tpm/tpm-chip.c > > > +++ b/drivers/char/tpm/tpm-chip.c > > > @@ -282,7 +282,7 @@ static void tpm_dev_release(struct device *dev) > > > * > > > * Return: always 0 (i.e. success) > > > */ > > > -static int tpm_class_shutdown(struct device *dev) > > > +int tpm_class_shutdown(struct device *dev) > > > { > > > struct tpm_chip *chip = container_of(dev, struct tpm_chip, dev); > > > > > > @@ -337,7 +337,6 @@ struct tpm_chip *tpm_chip_alloc(struct device *pdev, > > > device_initialize(&chip->dev); > > > > > > chip->dev.class = tpm_class; > > > - chip->dev.class->shutdown_pre = tpm_class_shutdown; > > > chip->dev.release = tpm_dev_release; > > > chip->dev.parent = pdev; > > > chip->dev.groups = chip->groups; > > > diff --git a/drivers/char/tpm/tpm-interface.c b/drivers/char/tpm/tpm-interface.c > > > index 8763c820d1f8..43e23a04433a 100644 > > > --- a/drivers/char/tpm/tpm-interface.c > > > +++ b/drivers/char/tpm/tpm-interface.c > > > @@ -467,6 +467,7 @@ static int __init tpm_init(void) > > > int rc; > > > > > > tpm_class = class_create("tpm"); > > > + tpm_class->shutdown_pre = tpm_class_shutdown; > > > if (IS_ERR(tpm_class)) { > > > pr_err("couldn't create tpm class\n"); > > > return PTR_ERR(tpm_class); > > > diff --git a/drivers/char/tpm/tpm.h b/drivers/char/tpm/tpm.h > > > index 24ee4e1cc452..a45eb39db0c4 100644 > > > --- a/drivers/char/tpm/tpm.h > > > +++ b/drivers/char/tpm/tpm.h > > > @@ -183,6 +183,7 @@ int tpm1_get_pcr_allocation(struct tpm_chip *chip); > > > unsigned long tpm_calc_ordinal_duration(struct tpm_chip *chip, u32 ordinal); > > > int tpm_pm_suspend(struct device *dev); > > > int tpm_pm_resume(struct device *dev); > > > +int tpm_class_shutdown(struct device *dev); > > > > > > static inline void tpm_msleep(unsigned int delay_msec) > > > { > > > -- > > > 2.39.2 > > > > > > > Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> > > > > Are you going to pick this? > > Yes, as the changelog text said, I'd like to take it through my tree, > thanks! OK, right. Just wanted to prevent any races, thanks. BR, Jarkko ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-03-14 13:03 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20230313181843.1207845-1-gregkh@linuxfoundation.org>
2023-03-13 18:18 ` [PATCH 10/12] tpm: fix up the tpm_class shutdown_pre pointer when created Greg Kroah-Hartman
2023-03-14 11:09 ` Jarkko Sakkinen
2023-03-14 12:53 ` Greg Kroah-Hartman
2023-03-14 12:57 ` Jarkko Sakkinen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox