Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-12-15 Thread Kai-Heng Feng
On Tue, Dec 15, 2020 at 2:22 AM Matthew Garrett wrote: > > On Sun, Nov 29, 2020 at 9:36 PM Kai-Heng Feng > wrote: > > > > We are seeing thermal shutdown on Intel based mobile workstations, the > > shutdown happens during the first trip handle in > > thermal_zone_device_register(): > > kernel: the

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-12-14 Thread Matthew Garrett
On Sun, Nov 29, 2020 at 9:36 PM Kai-Heng Feng wrote: > > We are seeing thermal shutdown on Intel based mobile workstations, the > shutdown happens during the first trip handle in > thermal_zone_device_register(): > kernel: thermal thermal_zone15: critical temperature reached (101 C), > shutting d

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-12-09 Thread Srinivas Pandruvada
On Wed, 2020-12-09 at 10:30 +0100, Daniel Lezcano wrote: > On 07/12/2020 06:36, Kai-Heng Feng wrote: > > > > > On Dec 1, 2020, at 02:39, Srinivas Pandruvada < > > > srinivas.pandruv...@linux.intel.com> wrote: > > > > > > On Tue, 2020-12-01 at 02:22 +0800, Kai-Heng Feng wrote: > > > > > On Dec 1,

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-12-09 Thread Daniel Lezcano
On 07/12/2020 06:36, Kai-Heng Feng wrote: > > >> On Dec 1, 2020, at 02:39, Srinivas Pandruvada >> wrote: >> >> On Tue, 2020-12-01 at 02:22 +0800, Kai-Heng Feng wrote: On Dec 1, 2020, at 02:13, Srinivas Pandruvada < srinivas.pandruv...@linux.intel.com> wrote: >>> >>> [snipped] >>> >>>

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-12-06 Thread Kai-Heng Feng
> On Dec 1, 2020, at 02:39, Srinivas Pandruvada > wrote: > > On Tue, 2020-12-01 at 02:22 +0800, Kai-Heng Feng wrote: >>> On Dec 1, 2020, at 02:13, Srinivas Pandruvada < >>> srinivas.pandruv...@linux.intel.com> wrote: >> >> [snipped] >> > What about creating an new callback > >

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-11-30 Thread Srinivas Pandruvada
On Tue, 2020-12-01 at 02:22 +0800, Kai-Heng Feng wrote: > > On Dec 1, 2020, at 02:13, Srinivas Pandruvada < > > srinivas.pandruv...@linux.intel.com> wrote: > > [snipped] > > > > > What about creating an new callback > > > > > > > > enum thermal_trip_status { > > > > THERMAL_TRIP_DISABLE

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-11-30 Thread Kai-Heng Feng
> On Dec 1, 2020, at 02:13, Srinivas Pandruvada > wrote: [snipped] >>> What about creating an new callback >>> >>> enum thermal_trip_status { >>> THERMAL_TRIP_DISABLED = 0, >>> THERMAL_TRIP_ENABLED, >>> }; >>> >>> int get_trip_status(struct thermal_zone_device *, int trip, enum >>>

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-11-30 Thread Srinivas Pandruvada
On Tue, 2020-12-01 at 02:04 +0800, Kai-Heng Feng wrote: > > On Dec 1, 2020, at 00:19, Srinivas Pandruvada < > > srinivas.pandruv...@linux.intel.com> wrote: > > > > On Mon, 2020-11-30 at 16:23 +0800, Kai-Heng Feng wrote: > > > > On Nov 30, 2020, at 15:57, Daniel Lezcano < > > > > daniel.lezc...@lin

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-11-30 Thread Kai-Heng Feng
> On Dec 1, 2020, at 00:19, Srinivas Pandruvada > wrote: > > On Mon, 2020-11-30 at 16:23 +0800, Kai-Heng Feng wrote: >>> On Nov 30, 2020, at 15:57, Daniel Lezcano < >>> daniel.lezc...@linaro.org> wrote: >>> >>> >>> [Added Srinivas] >>> >>> On 28/11/2020 18:54, Kai-Heng Feng wrote: We

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-11-30 Thread Srinivas Pandruvada
On Mon, 2020-11-30 at 16:23 +0800, Kai-Heng Feng wrote: > > On Nov 30, 2020, at 15:57, Daniel Lezcano < > > daniel.lezc...@linaro.org> wrote: > > > > > > [Added Srinivas] > > > > On 28/11/2020 18:54, Kai-Heng Feng wrote: > > > We are seeing thermal shutdown on Intel based mobile > > > workstatio

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-11-30 Thread Kai-Heng Feng
> On Nov 30, 2020, at 15:57, Daniel Lezcano wrote: > > > [Added Srinivas] > > On 28/11/2020 18:54, Kai-Heng Feng wrote: >> We are seeing thermal shutdown on Intel based mobile workstations, the >> shutdown happens during the first trip handle in >> thermal_zone_device_register(): >> kernel:

Re: [PATCH 1/3] thermal: core: Add indication for userspace usage

2020-11-29 Thread Daniel Lezcano
[Added Srinivas] On 28/11/2020 18:54, Kai-Heng Feng wrote: > We are seeing thermal shutdown on Intel based mobile workstations, the > shutdown happens during the first trip handle in > thermal_zone_device_register(): > kernel: thermal thermal_zone15: critical temperature reached (101 C), > shut

[PATCH 1/3] thermal: core: Add indication for userspace usage

2020-11-29 Thread Kai-Heng Feng
We are seeing thermal shutdown on Intel based mobile workstations, the shutdown happens during the first trip handle in thermal_zone_device_register(): kernel: thermal thermal_zone15: critical temperature reached (101 C), shutting down However, we shouldn't do a thermal shutdown here, since 1) We

[PATCH 1/3] thermal: core: Add indication for userspace usage

2020-11-28 Thread Kai-Heng Feng
We are seeing thermal shutdown on Intel based mobile workstations, the shutdown happens during the first trip handle in thermal_zone_device_register(): kernel: thermal thermal_zone15: critical temperature reached (101 C), shutting down However, we shouldn't do a thermal shutdown here, since 1) We