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=-10.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable 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 9EC82C432BE for ; Wed, 11 Aug 2021 00:18:14 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 7C55160FD9 for ; Wed, 11 Aug 2021 00:18:14 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235809AbhHKASe (ORCPT ); Tue, 10 Aug 2021 20:18:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58958 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235845AbhHKAS1 (ORCPT ); Tue, 10 Aug 2021 20:18:27 -0400 Received: from mail-ot1-x334.google.com (mail-ot1-x334.google.com [IPv6:2607:f8b0:4864:20::334]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1E679C061799 for ; Tue, 10 Aug 2021 17:18:05 -0700 (PDT) Received: by mail-ot1-x334.google.com with SMTP id z9-20020a9d62c90000b0290462f0ab0800so1241114otk.11 for ; Tue, 10 Aug 2021 17:18:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=tC3Mrv9dp9wt6xL+7sHY8TagV0MgWFNV+pJOGnVORVo=; b=FDn7/X54nL1aG3m/4vDJtAKvaaviRB7169EeCZwuzP0shJzDftJhls7H32FjLkR4tK TY0MO1Z+yupNXglfBUjzGPkdnNUoM9reDecGvC2JBQ2aUKhIZBhJf2yC9FksyUUsC8Rq i5GRR3+HOtcJ+Znq6y8W95C/6oqcPcBxeVQZLrF8nQIOI/lWu/0GlgBMh/o/DQujQvMt orLScabv/n3cpxDica+yDQ+iJXHXdMYutH2DCv3IR82bgKnpfMCBjBEkKLWYNyR6cm8s wCnGf+ducxIz14qCU7uLIoawYJIRAcDS+BddSJ7VLsN1rTB5lCozGt2merKrzy5YNmX9 fOZA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=tC3Mrv9dp9wt6xL+7sHY8TagV0MgWFNV+pJOGnVORVo=; b=YRlD63UvxoMFR1hfRmmMBW0cWk22sk9dRrI7+qrLeEaO7juNIyMuSg7aR/uCYpR+GW TI/+GGocBemVvm9/SGXpbu5Z2ikkSMdq7hPdNrpb2zNiqJq36JGOrAkZ8HcK1P5dhl/R 9J0N3f936c03JDXuPku50fNDd1YCqF+o29AKfyWNGWLF9xQKq1kebP8Ng4UFb8fEH/Dw crQoDDZbuwgy5RI4nVIN6QAtr1C855lKtbn/zbpCN5NNIveT0tMUwd2aUDsIGxcW7iFi mQw2Ply4A3xRZYo2Me+6FNdfxYAg+Ggi3J5c3zvlEX2aunCqJl8E1iAr50GP7ZzARENh MuuA== X-Gm-Message-State: AOAM531RbM7KwXiCmlUlHJD2rDEPgvV+8BGH57ZHLrzFsrnieQVbrkB+ 7Ar/ZhEue5rfv1SY7nSPkOds0w== X-Google-Smtp-Source: ABdhPJwkyONF/aUoE8sR15S4yNOA7sm6WRXOMuaxwi+kTEm/PcLMaps34StDjqMYdEhwRL4FwUMRSg== X-Received: by 2002:a9d:2609:: with SMTP id a9mr6767199otb.365.1628641084446; Tue, 10 Aug 2021 17:18:04 -0700 (PDT) Received: from builder.lan (104-57-184-186.lightspeed.austtx.sbcglobal.net. [104.57.184.186]) by smtp.gmail.com with ESMTPSA id 12sm4176912otg.14.2021.08.10.17.18.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Aug 2021 17:18:04 -0700 (PDT) Date: Tue, 10 Aug 2021 19:18:02 -0500 From: Bjorn Andersson To: Stephen Boyd Cc: Mike Tipton , djakov@kernel.org, agross@kernel.org, saravanak@google.com, okukatla@codeaurora.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Alex Elder Subject: Re: [PATCH v2 4/4] interconnect: qcom: icc-rpmh: Add BCMs to commit list in pre_aggregate Message-ID: References: <20210721175432.2119-1-mdtipton@codeaurora.org> <20210721175432.2119-5-mdtipton@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org On Tue 10 Aug 18:31 CDT 2021, Stephen Boyd wrote: > Quoting Mike Tipton (2021-07-21 10:54:32) > > We're only adding BCMs to the commit list in aggregate(), but there are > > cases where pre_aggregate() is called without subsequently calling > > aggregate(). In particular, in icc_sync_state() when a node with initial > > BW has zero requests. Since BCMs aren't added to the commit list in > > these cases, we don't actually send the zero BW request to HW. So the > > resources remain on unnecessarily. > > > > Add BCMs to the commit list in pre_aggregate() instead, which is always > > called even when there are no requests. > > > > Fixes: 976daac4a1c5 ("interconnect: qcom: Consolidate interconnect RPMh support") > > Signed-off-by: Mike Tipton > > --- > > This patch breaks reboot for me on sc7180 Lazor > FWIW, it prevents at least SM8150 from booting (need to check my other boards as well), because its no longer okay to have the interconnect providers defined without having all client paths specified. Regards, Bjorn