From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754665AbcGHLDp (ORCPT ); Fri, 8 Jul 2016 07:03:45 -0400 Received: from mx0b-0016f401.pphosted.com ([67.231.156.173]:6946 "EHLO mx0b-0016f401.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754362AbcGHLDh (ORCPT ); Fri, 8 Jul 2016 07:03:37 -0400 Date: Fri, 8 Jul 2016 18:58:54 +0800 From: Jisheng Zhang To: Daniel Lezcano , Arnd Bergmann , Olof Johansson CC: , , , , , Subject: Re: [PATCH v4 2/2] ARM: cpuidle: make arm_cpuidle_suspend() a bit more efficient Message-ID: <20160708185854.22e9fae9@xhacker> In-Reply-To: <20160708105012.GA32305@linaro.org> 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> 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-08_07:,, 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-1607080107 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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? Thanks