[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20240607115032.GA31351@sony.com>
Date: Fri, 7 Jun 2024 11:50:32 +0000
From: Khasnis Soumya <soumya.khasnis@...y.com>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: gregkh@...uxfoundation.org, rafael@...nel.org,
linux-kernel@...r.kernel.org, daniel.lezcano@...aro.org,
festevam@...x.de, lee@...nel.org, benjamin.bara@...data.com,
dmitry.osipenko@...labora.com, ldmldm05@...il.com,
soumya.khasnis@...y.com, srinavasa.nagaraju@...y.com,
Madhusudan.Bobbili@...y.com, shingo.takeuchi@...y.com,
keita.aihara@...y.com, masaya.takahashi@...y.com
Subject: Re: [PATCH v3] driver core: Add timeout for device shutdown
On Thu, Jun 06, 2024 at 10:52:14PM +0200, Greg KH wrote:
> On Thu, Jun 06, 2024 at 08:50:03AM +0000, Soumya Khasnis wrote:
> > The device shutdown callbacks invoked during shutdown/reboot
> > are prone to errors depending on the device state or mishandling
> > by one or more driver. In order to prevent a device hang in such
> > scenarios, we bail out after a timeout while dumping a meaningful
> > call trace of the shutdown callback to kernel logs, which blocks
> > the shutdown or reboot process.
> >
> > Signed-off-by: Soumya Khasnis <soumya.khasnis@...y.com>
> > Signed-off-by: Srinavasa Nagaraju <Srinavasa.Nagaraju@...y.com>
> > ---
> > Changes in v3:
> > -fix review comments
Sorry for inconvenience caused Greg, i will update details properly.
here are changes i did in patchset-3(v3)
1. added help text
2. set configuration by default "y"
3. added range for timeout value(DEVICE_SHUTDOWN_TIMEOUT_SEC)
4. moved #define's to base.h file
5. moved timeout functionality to device_shutdown() driver/base/core.c from reboot.c
>
> What ones? In what way? This really doesn't help, sorry. Would you
> like to see this type of response if you were to review other people's
> code?
>
> > -updated commit message
>
1. added information of where call trace is logged.
2. changed patch subject from "reboot:" to "driver core:"
> In what way?
>
> confused,
>
Sorry again for inconvenience caused.
> greg k-h
Thanks and Regards,
soumya.
Powered by blists - more mailing lists