From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752762AbcGOHng (ORCPT ); Fri, 15 Jul 2016 03:43:36 -0400 Received: from mx0b-0016f401.pphosted.com ([67.231.156.173]:45310 "EHLO mx0b-0016f401.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751229AbcGOHne (ORCPT ); Fri, 15 Jul 2016 03:43:34 -0400 Date: Fri, 15 Jul 2016 15:38:40 +0800 From: Jisheng Zhang To: Russell King - ARM Linux CC: , Daniel Lezcano , "Arnd Bergmann" , Olof Johansson , , , , Subject: Re: [PATCH v4 2/2] ARM: cpuidle: make arm_cpuidle_suspend() a bit more efficient Message-ID: <20160715153840.6c91bc48@xhacker> In-Reply-To: <20160714153113.GC1041@n2100.armlinux.org.uk> References: <1467785755-3488-1-git-send-email-jszhang@marvell.com> <1467785755-3488-3-git-send-email-jszhang@marvell.com> <20160707134628.GB20553@linaro.org> <20160708141729.3fa61c10@xhacker> <20160708105012.GA32305@linaro.org> <20160708185854.22e9fae9@xhacker> <20160714153113.GC1041@n2100.armlinux.org.uk> X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.30; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2016-07-15_03:,, signatures=0 X-Proofpoint-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1604210000 definitions=main-1607150088 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 14 Jul 2016 16:31:13 +0100 Russell King - ARM Linux wrote: > On Fri, Jul 08, 2016 at 06:58:54PM +0800, Jisheng Zhang wrote: > > On Fri, 8 Jul 2016 12:50:12 +0200 Daniel Lezcano wrote: > > > > > On Fri, Jul 08, 2016 at 02:17:29PM +0800, Jisheng Zhang wrote: > > > > Dear Daniel, > > > > > > > > On Thu, 7 Jul 2016 15:46:28 +0200 Daniel Lezcano wrote: > > > > > > > > > On Wed, Jul 06, 2016 at 02:15:55PM +0800, Jisheng Zhang wrote: > > > > > > Currently, we check cpuidle_ops.suspend every time when entering a > > > > > > low-power idle state. But this check could be avoided in this hot path > > > > > > by moving it into arm_cpuidle_read_ops() to reduce arm_cpuidle_suspend > > > > > > overhead a bit. > > > > > > > > > > > > Signed-off-by: Jisheng Zhang > > > > > > --- > > > > > > > > > > Acked-by: Daniel Lezcano > > > > > > > > I'm not sure the upstream merging path this patch should follow, Per my > > > > understanding, I need to put it into Russell's PATCH system. > > > > > > Or alternatively through arm-soc > > > > > > > Got it. thanks. > > > > Dear Arnd, Olof, > > > > I have no pull request permission. what's your preference? Could you please > > advise? > > ... and because arm-soc people haven't responded, they've now ended up > in the patch system... So I've applied them to my tree in a separate > branch. > Thank you so much!