All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: linux-pm <linux-pm@lists.linux-foundation.org>,
	linux-acpi <linux-acpi@vger.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Zhang Rui <rui.zhang@intel.com>, Len Brown <lenb@kernel.org>,
	Arjan van de Ven <arjan@linux.intel.com>
Subject: Re: [RFC][PATCH 0/3] PM: Asynchronous suspend and resume
Date: Thu, 13 Aug 2009 20:28:52 +0200	[thread overview]
Message-ID: <200908132028.53094.rjw@sisk.pl> (raw)
In-Reply-To: <Pine.LNX.4.44L0.0908131031060.2940-100000@iolanthe.rowland.org>

On Thursday 13 August 2009, Alan Stern wrote:
> On Wed, 12 Aug 2009, Rafael J. Wysocki wrote:
> 
> > I have an idea.
> > 
> > Every such dependency involves two devices, one of which is a "master"
> > and the second of which is a "slave", meaning that the "slave" have to be
> > suspended before the "master" and cannot be resumed before it.  In principle
> > we could give each device two lists of "dependency objects", one containing
> > "dependency objects" where the device is the "master" and the other containing
> > "dependency objects" where the device is the "slave".  Then, each "dependency
> > object" could be represented as
> > 
> > struct pm_connection {
> >     struct device *master;
> >     struct list_head master_hook;
> >     struct device *slave;
> >     struct list_head slave_hook;
> > };
> > 
> > Add some locking, helpers for adding / removing "dependency objects" etc.
> > and it should work.  Instead of checking the parent, walk the list of
> > "masters", instead of walking the list of children, walk the list of "slaves".
> 
> If the set of pm_connection objects is reasonably small then the
> master_hook and slave_hook aren't needed; you can just read through the
> entire list.  Leaving out parent-child connections, which we already
> know, would help shrink the set.

Do you mean a global list for the entire system?  I'm not sure if that wouldn't
hurt performance (please note that it also affects runtime PM).

We could use counters of "suspended slaves" and "active masters" to optimize
things, but still updating these counters could take time (one has to find all
devices dependent on given one and update a counter for each of them).

> The layout of the pm_connection objects could then be improved
> slightly.  Each object could contain a variable-sized array of device
> pointers together with two integers, M and S.  The first M pointers
> would be masters and the remaining S pointers would be slaves.  This
> could be useful when, for example, a large number of devices all depend
> on one particular power device.

I'm not sure.  If we use the coutners, this case will be quite easy to handle.

Thanks,
Rafael

  parent reply	other threads:[~2009-08-13 18:28 UTC|newest]

Thread overview: 153+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-12 20:18 [RFC][PATCH 0/3] PM: Asynchronous suspend and resume Rafael J. Wysocki
2009-08-12 20:20 ` [RFC][PATCH 1/3] PM: Asynchronous resume of devices Rafael J. Wysocki
2009-08-12 20:20   ` Rafael J. Wysocki
2009-08-14 16:33   ` Pavel Machek
2009-08-14 16:33   ` Pavel Machek
2009-08-15 20:59     ` Rafael J. Wysocki
2009-08-15 20:59     ` Rafael J. Wysocki
2009-08-22  9:24       ` Pavel Machek
2009-08-22 21:45         ` Rafael J. Wysocki
2009-08-22 21:45         ` Rafael J. Wysocki
2009-08-22  9:24       ` Pavel Machek
2009-08-12 20:21 ` [RFC][PATCH 2/3] PM: Asynchronous suspend " Rafael J. Wysocki
2009-08-14 16:35   ` Pavel Machek
2009-08-15 21:04     ` Rafael J. Wysocki
2009-08-15 21:04     ` Rafael J. Wysocki
2009-08-22  9:25       ` Pavel Machek
2009-08-22  9:25       ` Pavel Machek
2009-08-22 21:46         ` Rafael J. Wysocki
2009-08-22 21:46         ` Rafael J. Wysocki
2009-08-14 16:35   ` Pavel Machek
2009-08-12 20:21 ` Rafael J. Wysocki
2009-08-12 20:22 ` [RFC][PATCH 3/3] PM: Asynchronous suspend and resume for ACPI battery Rafael J. Wysocki
2009-08-12 20:22   ` Rafael J. Wysocki
2009-08-12 21:12 ` [RFC][PATCH 0/3] PM: Asynchronous suspend and resume Alan Stern
2009-08-12 21:43   ` Rafael J. Wysocki
2009-08-12 21:43   ` Rafael J. Wysocki
2009-08-13  8:18     ` Zhang Rui
2009-08-13  8:18     ` Zhang Rui
2009-08-13  8:18       ` Zhang Rui
2009-08-13 18:08       ` Rafael J. Wysocki
2009-08-13 18:08         ` Rafael J. Wysocki
2009-08-14  3:24         ` Zhang Rui
2009-08-14  3:24         ` Zhang Rui
2009-08-14 11:58           ` Rafael J. Wysocki
2009-08-14 11:58           ` Rafael J. Wysocki
2009-08-14 11:58             ` Rafael J. Wysocki
2009-08-13 18:08       ` Rafael J. Wysocki
2009-08-13 14:45     ` Alan Stern
2009-08-13 14:45     ` Alan Stern
2009-08-13 18:28       ` Rafael J. Wysocki
2009-08-13 18:28       ` Rafael J. Wysocki [this message]
2009-08-13 18:39         ` Alan Stern
2009-08-13 21:10           ` Rafael J. Wysocki
2009-08-13 21:10           ` Rafael J. Wysocki
2009-08-13 18:39         ` Alan Stern
2009-08-13 21:53     ` Rafael J. Wysocki
2009-08-13 21:53     ` Rafael J. Wysocki
2009-08-14 14:45       ` Alan Stern
2009-08-14 19:12         ` Rafael J. Wysocki
2009-08-14 19:12         ` Rafael J. Wysocki
2009-08-14 21:21           ` Alan Stern
2009-08-14 21:31             ` Rafael J. Wysocki
2009-08-14 21:37               ` Alan Stern
2009-08-14 21:37               ` Alan Stern
2009-08-16 10:29               ` [linux-pm] " Rafael J. Wysocki
2009-08-16 10:29                 ` Rafael J. Wysocki
2009-08-16 10:29               ` Rafael J. Wysocki
2009-08-14 21:31             ` Rafael J. Wysocki
2009-08-14 21:21           ` Alan Stern
2009-08-14 14:45       ` Alan Stern
2009-08-12 21:12 ` Alan Stern
2009-08-14 16:33 ` Pavel Machek
2009-08-14 16:33 ` Pavel Machek
2009-08-15 21:00   ` Rafael J. Wysocki
2009-08-15 21:00   ` Rafael J. Wysocki
2009-08-17  0:15 ` [RFC][PATCH 0/7] PM: Asynchronous suspend and resume (updated) Rafael J. Wysocki
2009-08-17  0:15   ` Rafael J. Wysocki
2009-08-17  0:16   ` [RFC][PATCH 1/7] PM: Update kerneldoc comments in drivers/base/power/main.c Rafael J. Wysocki
2009-08-19 21:57     ` Rafael J. Wysocki
2009-08-19 21:57     ` Rafael J. Wysocki
2009-08-19 22:00       ` Randy Dunlap
2009-08-19 22:00       ` Randy Dunlap
2009-08-19 22:06       ` Greg KH
2009-08-19 22:06       ` Greg KH
2009-08-19 22:28       ` Alan Stern
2009-08-19 23:14         ` Rafael J. Wysocki
2009-08-19 23:14         ` Rafael J. Wysocki
2009-08-20 14:00           ` Alan Stern
2009-08-20 14:00           ` Alan Stern
2009-08-19 22:28       ` Alan Stern
2009-08-26 15:44     ` Pavel Machek
2009-08-26 15:44     ` Pavel Machek
2009-08-17  0:16   ` Rafael J. Wysocki
2009-08-17  0:17   ` [RFC][PATCH 2/7] PM: Framework for representing PM links between devices Rafael J. Wysocki
2009-08-21 22:27     ` [RFC][PATCH 2/7 update] " Rafael J. Wysocki
2009-08-21 22:27     ` Rafael J. Wysocki
2009-08-17  0:17   ` [RFC][PATCH 2/7] " Rafael J. Wysocki
2009-08-17  0:18   ` [RFC][PATCH 3/7] PM: Asynchronous resume of I/O devices Rafael J. Wysocki
2009-08-23 22:09     ` [RFC][PATCH 3/7 updated] " Rafael J. Wysocki
2009-08-23 22:09     ` Rafael J. Wysocki
2009-08-17  0:18   ` [RFC][PATCH 3/7] " Rafael J. Wysocki
2009-08-17  0:19   ` [RFC][PATCH 4/7] PM: Asynchronous suspend " Rafael J. Wysocki
2009-08-17  0:19   ` Rafael J. Wysocki
2009-08-17  0:20   ` [RFC][PATCH 5/7] PM: Asynchronous suspend and resume of PCI devices Rafael J. Wysocki
2009-08-18  6:57     ` Zhang Rui
2009-08-18  6:57       ` Zhang Rui
2009-08-18 13:47       ` Alan Stern
2009-08-18 13:47       ` Alan Stern
2009-08-18  6:57     ` Zhang Rui
2009-08-17  0:20   ` Rafael J. Wysocki
2009-08-17  0:20   ` [RFC][PATCH 6/7] PM: Asynchronous suspend and resume of ACPI devices Rafael J. Wysocki
2009-08-17  0:20   ` Rafael J. Wysocki
2009-08-17  0:21   ` [RFC][PATCH 7/7] PM: Asynchronous suspend and resume of i8042 Rafael J. Wysocki
2009-08-17  0:21   ` Rafael J. Wysocki
2009-08-18  7:03     ` Zhang Rui
2009-08-18  7:03       ` Zhang Rui
2009-08-18 19:57       ` Rafael J. Wysocki
2009-08-18 23:40         ` Rafael J. Wysocki
2009-08-18 23:40         ` Rafael J. Wysocki
2009-08-18 23:40           ` Rafael J. Wysocki
2009-08-18 19:57       ` Rafael J. Wysocki
2009-08-26 15:43       ` Pavel Machek
2009-08-26 15:43       ` Pavel Machek
2009-08-18  7:03     ` Zhang Rui
2009-08-18  1:59   ` [RFC][PATCH 0/7] PM: Asynchronous suspend and resume (updated) Zhang Rui
2009-08-18  1:59   ` Zhang Rui
2009-08-18  7:16   ` Zhang Rui
2009-08-18  7:16     ` Zhang Rui
2009-08-18 20:01     ` Rafael J. Wysocki
2009-08-18 20:01     ` Rafael J. Wysocki
2009-08-18 20:01       ` Rafael J. Wysocki
2009-08-18 23:58       ` Rafael J. Wysocki
2009-08-18 23:58         ` Rafael J. Wysocki
2009-08-19  1:05         ` Zhang Rui
2009-08-19  1:05         ` Zhang Rui
2009-08-19  1:05           ` Zhang Rui
2009-08-19 21:02           ` Rafael J. Wysocki
2009-08-19 21:02           ` Rafael J. Wysocki
2009-08-21  7:40             ` Zhang Rui
2009-08-21  7:40             ` Zhang Rui
2009-08-18 23:58       ` Rafael J. Wysocki
2009-08-26 15:44     ` Pavel Machek
2009-08-26 15:44     ` Pavel Machek
2009-08-18  7:16   ` Zhang Rui
2009-08-18 14:04   ` Alan Stern
2009-08-18 14:04   ` Alan Stern
2009-08-18 19:56     ` Rafael J. Wysocki
2009-08-18 20:22       ` Alan Stern
2009-08-18 20:22       ` Alan Stern
2009-08-18 22:33         ` Rafael J. Wysocki
2009-08-19 14:07           ` Alan Stern
2009-08-19 14:07           ` Alan Stern
2009-08-19 21:17             ` Rafael J. Wysocki
2009-08-19 21:17             ` Rafael J. Wysocki
2009-08-19 22:34               ` Alan Stern
2009-08-20 15:56                 ` Rafael J. Wysocki
2009-08-20 15:56                 ` Rafael J. Wysocki
2009-08-19 22:34               ` Alan Stern
2009-08-18 22:33         ` Rafael J. Wysocki
2009-08-18 19:56     ` Rafael J. Wysocki
2009-08-26 13:20     ` Pavel Machek
2009-08-26 13:20     ` Pavel Machek
  -- strict thread matches above, loose matches on Subject: below --
2009-08-12 20:18 [RFC][PATCH 0/3] PM: Asynchronous suspend and resume Rafael J. Wysocki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200908132028.53094.rjw@sisk.pl \
    --to=rjw@sisk.pl \
    --cc=arjan@linux.intel.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=rui.zhang@intel.com \
    --cc=stern@rowland.harvard.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.