From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D0C0BC433E0 for ; Thu, 6 Aug 2020 12:26:53 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id A455022DFA for ; Thu, 6 Aug 2020 12:26:53 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="VJ5d5pM6"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="IRZhR4rj" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org A455022DFA Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:Message-ID:References:In-Reply-To:Subject:To:From: Date:MIME-Version:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=UvmSeAj+kowxMB+MbkbVHCxb+2iZIbDB7ecmGNA8dvs=; b=VJ5d5pM6JbDyDPd5hczQGP8zG UB+JJ9/x2Yrh+D0zfRZt7+MGc5MhCL1jT7tEMe057quOP36h+e93mhjnKN8vrvMyWCMfYSpjCb6Oz NpCiX47B4cq314arRp/AZ6OEvRnPa9+5O8den9fskXIF1ahVCa6HTu4d9OiQk47mvWgeFTkf8cALq 5baF+ZG4Xbbe7Hifxw71mEfkuVSpRMonDlbOgqnDEaIrF9I/mtTPoIO/J/pc8ljvLCxR92hV7GhVr wQm7D/VM81Y4Ai/O1bmYAiGtl34gkesVch1tjj0K2cIPsGrJ7KOjKoziKKKNIlCOTVXpzk/1vOp1v dxSgpk7Hw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1k3exo-0005FP-MG; Thu, 06 Aug 2020 12:25:41 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1k3exm-0005Ez-PO for linux-arm-kernel@lists.infradead.org; Thu, 06 Aug 2020 12:25:39 +0000 Received: from disco-boy.misterjones.org (disco-boy.misterjones.org [51.254.78.96]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id CF96B22CF6; Thu, 6 Aug 2020 12:25:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1596716738; bh=jpYTCCwR4fZP8SyH14e+ACdHwa6e8RH80YNL1hLi6L8=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=IRZhR4rjAD9KvhCFxHEGYa6XFLYelDK0rOrJrpXDwilVmtTt1wRyxDSC4Wswclshs B9Q7GB075Ei7Z37D1jXxrm1iIYpDK/dls9Z2XQoCLXwexQHnOD9Fobkp8o7ddWlAJ0 IUPhOkDgbXgI0PhArND5uIY0mRYirezqMMpSjl2A= Received: from disco-boy.misterjones.org ([51.254.78.96] helo=www.loen.fr) by disco-boy.misterjones.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1k3exk-000Gxu-7i; Thu, 06 Aug 2020 13:25:36 +0100 MIME-Version: 1.0 Date: Thu, 06 Aug 2020 13:25:35 +0100 From: Marc Zyngier To: Jason Liu Subject: Re: [PATCH 1/1] arm64: kexec: no need to do irq_chip->irq_mask if it already masked In-Reply-To: References: <20200804085657.10776-1-jason.hui.liu@nxp.com> <20200804113850.GB15199@bogus> <3c63ae0cc3a7b5bad5d4638a9870340e@kernel.org> User-Agent: Roundcube Webmail/1.4.5 Message-ID: <1e4496c263e486be3438f2797630164a@kernel.org> X-Sender: maz@kernel.org X-SA-Exim-Connect-IP: 51.254.78.96 X-SA-Exim-Rcpt-To: jason.hui.liu@nxp.com, sudeep.holla@arm.com, catalin.marinas@arm.com, will@kernel.org, sashal@kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200806_082538_983328_E5D45B31 X-CRM114-Status: GOOD ( 25.34 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: sashal@kernel.org, catalin.marinas@arm.com, linux-kernel@vger.kernel.org, Sudeep Holla , will@kernel.org, linux-arm-kernel@lists.infradead.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 2020-08-06 11:05, Jason Liu wrote: >> -----Original Message----- >> From: Marc Zyngier [...] >> > No, this patch is not papering over a much deeper issue in the driver. >> > This is just to make things better for the ARM64 kexec. >> >> Yes, I'm sure it is... However: >> >> request_irq() >> > clock off> >> if (chip->irq_mask && !irqd_irq_masked(&desc->irq_data)) >> >> >> This is because the PM in the irqsteer driver is completely busted: >> request_irq() should get a reference on the driver to prevent it from >> being >> suspended. Since you don't implement it correctly, this doesn't happen >> and >> your "improvement" doesn't help at all. > > The request_irq will get a reference to prevent the irqchip from being > suspended due to it call > irq_chip_pm_get(). I am pretty sure we have implemented correctly and > that is also the common Linux code. Then it seems you cannot read your own driver. At no point do you set the parent_device that would give you a fighting chance to get the device clocked and powered on. How does it work? Magic? > In order to save power and let the irqchip enter into runtime SUSPEND > mode, the driver will call free_irq() > When it was not used(idle). Then free_irq() will decrease the > reference and let the irqchip enter suspend state. The reference count on *what*? There is nothing to take a reference on. So yes, you understand how the core kernel works. But you don't seem to notice that there is no link between the irq and the device that implements the controller. > So, when the irqchip entered suspend, which means there is no user for > the irqchip and all the irqs were DISABLED + MASKED. > Due to the runtimePM support for the irqchip, when kexec runs, it will > sometimes meet the situation that the irqchip is suspend due to > no users for it. So from either the performance(time cost) or coding > logic, the machine_kexec_mask_interrupts() should not do > double mask for the irqs which already masked. I strongly suggest you start by fixing the damn driver first. In the meantime, NAK to this patch. M. -- Jazz is not dead. It just smells funny... _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel