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=-6.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,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 3C749C0650F for ; Mon, 5 Aug 2019 09:59:26 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 106AD217D9 for ; Mon, 5 Aug 2019 09:59:26 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="ZVw3d0EJ" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 106AD217D9 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=rjwysocki.net Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-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=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=tGHjIoNIJgn4+9JByY0sVA+8t4QeggbhtRfQFmJ1dv0=; b=ZVw3d0EJnpz8pF Er+PJ1dtY2ZpMY4F67DIOzF5AxEjhu2HtwImozs7R/B09D7AgL+USjbjHlYESL+S2z9Wd6WbnEXBF fdPmGKLjlYjG6w3viGv4ewt1BTfdsmU5f5VW3LXlxzOt34ihinUl/nq2qGJfGjrVV27MLsHRoRMWO hKcUH0W5M1AJ9HjTGmj9CScsetXN/AYZ4bCxQ8rh1lCEDB6lLHBuQ1JXW8vHid7BGjFqu8Ay3SUxY s0rrTI+SRKhafg4LCh7DpGv56xn/yKz5tSd6dShGPaGwwKC0QCjXO2KEmslOCkswWtau7MEN1ggAd jtBdCjUjs5ssNXKIM8AQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1huZlv-00042h-EZ; Mon, 05 Aug 2019 09:59:19 +0000 Received: from cloudserver094114.home.pl ([79.96.170.134]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1huZls-00041l-8S for linux-arm-kernel@lists.infradead.org; Mon, 05 Aug 2019 09:59:17 +0000 Received: from 79.184.254.29.ipv4.supernova.orange.pl (79.184.254.29) (HELO kreacher.localnet) by serwer1319399.home.pl (79.96.170.134) with SMTP (IdeaSmtpServer 0.83.275) id a3463300b674835e; Mon, 5 Aug 2019 11:59:09 +0200 From: "Rafael J. Wysocki" To: Ran Wang Subject: Re: [PATCH v5 1/3] PM: wakeup: Add routine to help fetch wakeup source object. Date: Mon, 05 Aug 2019 11:59:09 +0200 Message-ID: <4158639.B12JYek7R7@kreacher> In-Reply-To: <20190724074722.12270-1-ran.wang_1@nxp.com> References: <20190724074722.12270-1-ran.wang_1@nxp.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190805_025916_463013_E8EE3A0D X-CRM114-Status: GOOD ( 19.53 ) 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: Mark Rutland , Li Biwen , Len Brown , Greg Kroah-Hartman , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Li Yang , devicetree@vger.kernel.org, Rob Herring , Pavel Machek , linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wednesday, July 24, 2019 9:47:20 AM CEST Ran Wang wrote: > Some user might want to go through all registered wakeup sources > and doing things accordingly. For example, SoC PM driver might need to > do HW programming to prevent powering down specific IP which wakeup > source depending on. So add this API to help walk through all registered > wakeup source objects on that list and return them one by one. > > Signed-off-by: Ran Wang > --- > Change in v5: > - Update commit message, add decription of walk through all wakeup > source objects. > - Add SCU protection in function wakeup_source_get_next(). > - Rename wakeup_source member 'attached_dev' to 'dev' and move it up > (before wakeirq). > > Change in v4: > - None. > > Change in v3: > - Adjust indentation of *attached_dev;. > > Change in v2: > - None. > > drivers/base/power/wakeup.c | 24 ++++++++++++++++++++++++ > include/linux/pm_wakeup.h | 3 +++ > 2 files changed, 27 insertions(+) > > diff --git a/drivers/base/power/wakeup.c b/drivers/base/power/wakeup.c > index ee31d4f..2fba891 100644 > --- a/drivers/base/power/wakeup.c > +++ b/drivers/base/power/wakeup.c > @@ -14,6 +14,7 @@ > #include > #include > #include > +#include > #include > #include > > @@ -226,6 +227,28 @@ void wakeup_source_unregister(struct wakeup_source *ws) > } > } > EXPORT_SYMBOL_GPL(wakeup_source_unregister); > +/** > + * wakeup_source_get_next - Get next wakeup source from the list > + * @ws: Previous wakeup source object, null means caller want first one. > + */ > +struct wakeup_source *wakeup_source_get_next(struct wakeup_source *ws) > +{ > + struct list_head *ws_head = &wakeup_sources; > + struct wakeup_source *next_ws = NULL; > + int idx; > + > + idx = srcu_read_lock(&wakeup_srcu); > + if (ws) > + next_ws = list_next_or_null_rcu(ws_head, &ws->entry, > + struct wakeup_source, entry); > + else > + next_ws = list_entry_rcu(ws_head->next, > + struct wakeup_source, entry); > + srcu_read_unlock(&wakeup_srcu, idx); > + This is incorrect. The SRCU cannot be unlocked until the caller of this is done with the object returned by it, or that object can be freed while it is still being accessed. Besides, this patch conflicts with some general wakeup sources changes in the works, so it needs to be deferred and rebased on top of those changes. > + return next_ws; > +} > +EXPORT_SYMBOL_GPL(wakeup_source_get_next); > > /** > * device_wakeup_attach - Attach a wakeup source object to a device object. > @@ -242,6 +265,7 @@ static int device_wakeup_attach(struct device *dev, struct wakeup_source *ws) > return -EEXIST; > } > dev->power.wakeup = ws; > + ws->dev = dev; > if (dev->power.wakeirq) > device_wakeup_attach_irq(dev, dev->power.wakeirq); > spin_unlock_irq(&dev->power.lock); > diff --git a/include/linux/pm_wakeup.h b/include/linux/pm_wakeup.h > index 9102760..fc23c1a 100644 > --- a/include/linux/pm_wakeup.h > +++ b/include/linux/pm_wakeup.h > @@ -23,6 +23,7 @@ struct wake_irq; > * @name: Name of the wakeup source > * @entry: Wakeup source list entry > * @lock: Wakeup source lock > + * @dev: The device it attached to > * @wakeirq: Optional device specific wakeirq > * @timer: Wakeup timer list > * @timer_expires: Wakeup timer expiration > @@ -42,6 +43,7 @@ struct wakeup_source { > const char *name; > struct list_head entry; > spinlock_t lock; > + struct device *dev; > struct wake_irq *wakeirq; > struct timer_list timer; > unsigned long timer_expires; > @@ -88,6 +90,7 @@ extern void wakeup_source_add(struct wakeup_source *ws); > extern void wakeup_source_remove(struct wakeup_source *ws); > extern struct wakeup_source *wakeup_source_register(const char *name); > extern void wakeup_source_unregister(struct wakeup_source *ws); > +extern struct wakeup_source *wakeup_source_get_next(struct wakeup_source *ws); > extern int device_wakeup_enable(struct device *dev); > extern int device_wakeup_disable(struct device *dev); > extern void device_set_wakeup_capable(struct device *dev, bool capable); > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel