From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH] spi: qup: skip clk_disable_unprepare if the device is already runtime suspended Date: Fri, 2 Sep 2016 10:38:53 +0100 Message-ID: <20160902093853.GI3950@sirena.org.uk> References: <1472128408-7231-1-git-send-email-sudeep.holla@arm.com> <20160901202940.GU5967@sirena.org.uk> <630dd514-acff-5e83-d638-bda77876ebf2@arm.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="yaPAUYI/0vT2YKpA" Return-path: Received: from mezzanine.sirena.org.uk ([106.187.55.193]:56146 "EHLO mezzanine.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751225AbcIBJkV (ORCPT ); Fri, 2 Sep 2016 05:40:21 -0400 Content-Disposition: inline In-Reply-To: <630dd514-acff-5e83-d638-bda77876ebf2@arm.com> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Sudeep Holla Cc: linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, linux-soc@vger.kernel.org, linux-arm-msm@vger.kernel.org, Andy Gross , David Brown --yaPAUYI/0vT2YKpA Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Sep 02, 2016 at 09:42:04AM +0100, Sudeep Holla wrote: > On 01/09/16 21:29, Mark Brown wrote: > > On Thu, Aug 25, 2016 at 01:33:28PM +0100, Sudeep Holla wrote: > > > CPU: 3 PID: 1593 Comm: bash Tainted: G W 4.8.0-rc3 #14 > > > Hardware name: Qualcomm Technologies, Inc. APQ 8016 SBC (DT) > > > PC is at clk_core_unprepare+0x80/0x90 > > > LR is at clk_unprepare+0x28/0x40 > > > pc : [] lr : [] pstate: 60000145 > > Please think hard before including complete backtraces in upstream > > reports, they are very large and contain almost no useful information > > relative to their size so often obscure the relevant content in your > > message. If part of the backtrace is usefully illustrative then it's > > usually better to pull out the relevant sections. > I removed most of the addresses and just retained the symbols(somehow > the last line with pc and lr was left unintentionally). While you may > have the above opinion, other maintainers may differ. In future, I will > try to add it as a note just to describe the issue. Oh, *that's* why it looked so weird. Removing the addresses doesn't help here, the issue isn't that the addresses are confusing it's that you had a tiny commit message dwarfed by the backtrace preamble then a screenful of call stack which conveyed no meaningful information, including not just the entire callback path for a suspend (which doesn't tell us anything really, especially beyond the first frame) and going on to show the entire call stack from the sysfs write you used to trigger suspend which is even less relevant. This gives us 30 lines or so of splat (more than a screenful) for five lines of actual content with the important bit which describes what the change is supposed to be doing buried at the bottom. That's a really bad signal to noise ratio. What would've been better would be explaining why the change you are making fixes the problem. --yaPAUYI/0vT2YKpA Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEcBAEBCAAGBQJXyUikAAoJECTWi3JdVIfQUDQH/A0V2XzjJ1GuBfDGxbJ3PxzJ XoGxnBeyWij5obIcDPwD7FHagKOMZrRM0eDT3rKdHRlXDhXAk6lTW8Edb68DgYOe ZQC+4qL6KmhDbc3osF5Fw2S+9nOr5MO5BIIV2g5FHyET0lLIPhP1d1mMKq1CkWrF 0qa1Fr6e77C7agI5Xcw6Euz8CfHlWsPogWlTm2yC6pTt9zj89RtKmO/cV+MLxQtF ZA3xxvd13DuCm5TK2vfx8b8kLzzWoS0tI6BDagAwyZcxuoiOlgFpk7CRDizJIlML LwOgO2pHGyMWjDPA/rtLk11hR0iLQ3aHkPZqKhun+EspKSfkvaESBWtdGtqj6+M= =38mP -----END PGP SIGNATURE----- --yaPAUYI/0vT2YKpA--