From: Linus Torvalds <torvalds@linux-foundation.org>
To: Hugh Dickins <hugh@veritas.com>
Cc: Alan Stern <stern@rowland.harvard.edu>,
Cornelia Huck <cornelia.huck@de.ibm.com>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Oliver Neukum <oneukum@suse.de>,
Maneesh Soni <maneesh@in.ibm.com>,
gregkh@suse.de, Richard Purdie <rpurdie@rpsys.net>,
James Bottomley <James.Bottomley@SteelEye.com>,
Kernel development list <linux-kernel@vger.kernel.org>
Subject: Re: 2.6.21-rc suspend regression: sysfs deadlock
Date: Thu, 15 Mar 2007 09:51:33 -0700 (PDT) [thread overview]
Message-ID: <Pine.LNX.4.64.0703150944250.3816@woody.linux-foundation.org> (raw)
In-Reply-To: <Pine.LNX.4.64.0703151555240.26565@blonde.wat.veritas.com>
On Thu, 15 Mar 2007, Hugh Dickins wrote:
>
> sysfs_access_in_other_task() left me wondering what this "other" task
> was, and what kind of "access" it's trying to get - or is the calling
> task the other, and it's trying to access something it wouldn't
> directly have access to?
For naming clashes, I'd suggest:
- try to name according to *why* something is done, not necessarily what
it does.
For example, is it really in "another task"? Maybe it's just an
on-demand thread of the same task? Do you actually care how the
deferred work is done?
- avoid being vague. I agree with not liking the name much, and the
"other" thing bothers me. Like Hugh, it makes me ask "_What_ other
task?"
So I would suggest not concentrating on some implementation issue, but on
the reason why you need it in the first place. Namely that you want to
defer the actual action to avoid deadlock due to recursive locking. So
that "why do I actually do this" thing implies something like
"sysfs_store_async()" or "sysfs_store_deferred()" or maybe actually
concentrate on the locking angle and say something like
"sysfs_store_needs_to_reacquire_lock()".
(That last one wasn't really serious - it's too long and cumbersome, but
it's an example of not caring _how_ you do it, just abotu what you want
done).
Linus
next prev parent reply other threads:[~2007-03-15 16:52 UTC|newest]
Thread overview: 61+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-08 13:05 refcounting drivers' data structures used in sysfs buffers Oliver Neukum
2007-03-08 16:02 ` Alan Stern
2007-03-09 0:45 ` Oliver Neukum
2007-03-09 16:32 ` Alan Stern
2007-03-09 16:44 ` Oliver Neukum
2007-03-09 17:02 ` Dmitry Torokhov
2007-03-09 17:18 ` Oliver Neukum
2007-03-09 17:34 ` Dmitry Torokhov
2007-03-09 19:32 ` Alan Stern
2007-03-09 20:05 ` Oliver Neukum
2007-03-09 20:27 ` Alan Stern
2007-03-09 20:39 ` Oliver Neukum
2007-03-09 20:08 ` Alan Stern
2007-03-09 20:48 ` Oliver Neukum
2007-03-10 19:19 ` Alan Stern
2007-03-12 8:54 ` Oliver Neukum
2007-03-12 14:57 ` Alan Stern
2007-03-12 15:23 ` Oliver Neukum
2007-03-12 15:42 ` Dmitry Torokhov
2007-03-12 15:59 ` Oliver Neukum
2007-03-12 16:21 ` Alan Stern
2007-03-12 18:25 ` Oliver Neukum
2007-03-12 19:31 ` Alan Stern
2007-03-12 19:49 ` Oliver Neukum
2007-03-12 20:03 ` Alan Stern
2007-03-12 20:15 ` Oliver Neukum
2007-03-12 20:31 ` Dmitry Torokhov
2007-03-12 20:45 ` Alan Stern
2007-03-12 21:31 ` Richard Purdie
2007-03-13 15:00 ` 2.6.21-rc suspend regression: sysfs deadlock Alan Stern
2007-03-13 18:42 ` Cornelia Huck
2007-03-13 21:20 ` Linus Torvalds
2007-03-14 16:12 ` Alan Stern
2007-03-14 18:43 ` Cornelia Huck
2007-03-14 19:23 ` Alan Stern
2007-03-15 10:27 ` Cornelia Huck
2007-03-15 12:31 ` Hugh Dickins
2007-03-15 13:02 ` Oliver Neukum
2007-03-15 13:22 ` Dmitry Torokhov
2007-03-15 13:59 ` Hugh Dickins
2007-03-15 14:27 ` Alan Stern
2007-03-15 15:32 ` Cornelia Huck
2007-03-15 16:29 ` Hugh Dickins
2007-03-15 16:51 ` Linus Torvalds [this message]
2007-03-15 19:50 ` [PATCH] sysfs and driver core: add callback helper, used by SCSI and S390 Alan Stern
2007-03-15 19:51 ` [PATCH] sysfs: reinstate exclusion between method calls and attribute unregistration Alan Stern
2007-03-13 19:00 ` 2.6.21-rc suspend regression: sysfs deadlock Hugh Dickins
2007-03-13 20:09 ` Alan Stern
2007-03-13 20:55 ` Hugh Dickins
2007-03-13 21:08 ` Dmitry Torokhov
2007-03-13 21:20 ` Alan Stern
-- strict thread matches above, loose matches on Subject: below --
2007-03-10 20:44 Alan Stern
2007-03-06 19:20 Hugh Dickins
2007-03-06 20:16 ` Oliver Neukum
2007-03-07 1:56 ` Linus Torvalds
2007-03-07 14:38 ` Oliver Neukum
2007-03-07 15:56 ` Dmitry Torokhov
2007-03-07 16:52 ` Linus Torvalds
2007-03-07 16:59 ` Oliver Neukum
2007-03-07 18:02 ` Linus Torvalds
2007-03-07 18:16 ` Oliver Neukum
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=Pine.LNX.4.64.0703150944250.3816@woody.linux-foundation.org \
--to=torvalds@linux-foundation.org \
--cc=James.Bottomley@SteelEye.com \
--cc=cornelia.huck@de.ibm.com \
--cc=dmitry.torokhov@gmail.com \
--cc=gregkh@suse.de \
--cc=hugh@veritas.com \
--cc=linux-kernel@vger.kernel.org \
--cc=maneesh@in.ibm.com \
--cc=oneukum@suse.de \
--cc=rpurdie@rpsys.net \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).