Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751106AbdDLEFy (ORCPT <rfc822;w@1wt.eu>);
        Wed, 12 Apr 2017 00:05:54 -0400
Received: from mail-pf0-f196.google.com ([209.85.192.196]:33046 "EHLO
        mail-pf0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1750731AbdDLEFv (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Wed, 12 Apr 2017 00:05:51 -0400
Date: Tue, 11 Apr 2017 21:05:46 -0700
From: Eduardo Valentin <edubezval@gmail.com>
To: Keerthy <j-keerthy@ti.com>
Cc: Zhang Rui <rui.zhang@intel.com>, linux-pm@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, nm@ti.com,
        t-kristo@ti.com
Subject: Re: [PATCH] thermal: core: Add a back up thermal shutdown mechanism
Message-ID: <20170412040542.GA11305@localhost.localdomain>
References: <1490941820-13511-1-git-send-email-j-keerthy@ti.com>
 <20170411172918.GA5193@localhost.localdomain>
 <f64632d5-c780-e5fe-cff7-8ed1459348a0@ti.com>
 <1491967248.2357.25.camel@intel.com>
 <492e72af-ff33-d193-071e-5bc00df9a8b0@ti.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="envbJBWh7q8WU6mo"
Content-Disposition: inline
In-Reply-To: <492e72af-ff33-d193-071e-5bc00df9a8b0@ti.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 4040
Lines: 117


--envbJBWh7q8WU6mo
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Keerthy,

On Wed, Apr 12, 2017 at 09:09:36AM +0530, Keerthy wrote:
>=20
>=20
> On Wednesday 12 April 2017 08:50 AM, Zhang Rui wrote:
> > On Wed, 2017-04-12 at 08:19 +0530, Keerthy wrote:
> >>
> >> On Tuesday 11 April 2017 10:59 PM, Eduardo Valentin wrote:
> >>>
> >>> Hey,
> >>>
> >>> On Fri, Mar 31, 2017 at 12:00:20PM +0530, Keerthy wrote:
> >>>>
> >>>> off).

<cut>

> >>> OK... This seams to me, still a corner case supposed to be fixed at
> >>> orderly_power_off, not at thermal. But..
> >>>

^^^ Then again, this must be fixed not at thermal core. And re-reading
the history of this thread, this seams to be really something broken at
OMAP/DRA7, as mentioned in previous messages. That is probably why you
are pushing for pm_power_off(), which seams to be the one that works for
you, pulling the plug correctly (DRA7).

> >>>>
> >>>>
> >>>> However, there is no clean way of detecting such failure of
> >>>> userspace
> >>>> powering off the system. In such scenarios, it is necessary for a
> >>>> backup
> >>>> workqueue to be able to force a shutdown of the system when
> >>>> orderly
> >>>> shutdown is not successful after a configurable time period.
> >>>>
> >>> Given that system running hot is a thermal issue, I guess we care
> >>> more
> >>> on this matter then..
> >> Yes!
> >>
> > I just read this thread again https://patchwork.kernel.org/patch/802458
> > 1/ to recall the previous discussion.
> >=20
> > https://patchwork.kernel.org/patch/8149891/
> > https://patchwork.kernel.org/patch/8149861/
> > should be the solution made based on Ingo' suggestion, right?
> >=20
> > And to me, this sounds like the right direction to go, thermal does not
> > need a back up shutdown solution, it just needs a kernel function call
> > which guarantees the system can be shutdown/reboot immediately.
> >=20
> > is there any reason that patch 1/2 is not accepted?
>=20
> Zhang,
>=20
> http://www.serverphorums.com/read.php?12,1400964
>=20
> I got a NAK from Alan and was given this direction on a thermal_poweroff
> which is more or less what is done in this patch.
>=20


Actually, Alan's suggestion is more for you to define a
thermal_poweroff() that can be defined per architecture.

Also, please, keep track of your patch versions and also do copy
everybody who has stated their opinion on previous discussions. These
patches must have Ingo, Alan, and RMK copied too. In this way we avoid
loosing track of what has been suggested and we also converge faster to
something everybody (or most of us) agree. Next version, please, fix
that.


To me, thermal core needs a function that simply powers off the system.
No timeouts, delayed works, backups, etc. Simple and straight.

The idea of having a per architecture implementation, as per Alan's
suggestion, makes sense to me too. Having something different from
pm_power_off(), specific to thermal, might also give the opportunity to
save the power off reason.

BR,

Eduardo Valentin

--envbJBWh7q8WU6mo
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBCAAGBQJY7aePAAoJEA6VkvSQfF5T46YQAJj74GgdpogbFhVgMF9qkoMD
AS8bq7AMaDDkmtLCekb2Q8b9iQ9HVHDDl0jM76yrA0HKPRgbs+LjhF4XlfKWJ7he
8SRGw2O/RelAPj1i4kh+/MMOgLUPU4/a87srdMpVg0kn1enKEtCgtOmg2cHuIVTI
5oZZzVLd0pDkEElC1HbYwdw0iIceHv0oF7Dzt5qvsIqENnGgAAHpY/hPazER5CTe
JFOJNHamhIQEO5BWpxtZHDPTSpqQXKxO8P7trUjkdi1mKImb2WCQQotZuX6ydLuI
XE1M/Q5W8tSc8utrn4ZjYVigF6+pgbQilXDhSOyZeT6bn21UL7PRU6WtKQMn34rI
J1L3gp2tfa8jlnJmUgOjSmNIGaS/YZZgmO12Oxe7m91U7ZQt2zT+dBPPTw8zQgHT
SL9NzfyFexwvrAuowNkigK3YTnvKJeKIHNXiKXDXZu25xXOGSTnzPP96LpkhvvLM
GysdcKogvzmLT9KonE7JUJ46Z3cUpjhlah7v7/IgykT9Q1oj7SnTw8SdzYBbh1L1
dn1KQ9ZgCX9K/X3YZFkfP9ZXm8lStReO8UeVRtMcmmqF09ruBFF/3ks3Wuse/y1o
ZkOE7i2u7LYbzzux1lLLKRUSB7FDZEBcn3CueKMXf0jWCmtwnTG4EoR5DSWilfPO
hdADY2dQ8/9WiOUyCzGO
=qrqO
-----END PGP SIGNATURE-----

--envbJBWh7q8WU6mo--