public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Roland Dreier <rdreier@cisco.com>
To: Andrew Morton <akpm@osdl.org>
Cc: Andy Fleming <afleming@freescale.com>,
	"Maciej W. Rozycki" <macro@linux-mips.org>,
	Ben Collins <ben.collins@ubuntu.com>,
	linux-kernel@vger.kernel.org, Linus Torvalds <torvalds@osdl.org>,
	Jeff Garzik <jeff@garzik.org>
Subject: Re: [PATCH] Export current_is_keventd() for libphy
Date: Tue, 05 Dec 2006 15:49:09 -0800	[thread overview]
Message-ID: <adairgpc39m.fsf@cisco.com> (raw)
In-Reply-To: <20061205135753.9c3844f8.akpm@osdl.org> (Andrew Morton's message of "Tue, 5 Dec 2006 13:57:53 -0800")

 > But running flush_scheduled_work() from within dev_close() is a very
 > sensible thing to do, and dev_close is called under rtnl_lock().

I can't argue with that -- this has actually bitten me in the past.

Hmm, I'll try to understand why we need rtnl_lock() to cover dev_close...

 - R.

  reply	other threads:[~2006-12-05 23:49 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-03  5:50 [PATCH] Export current_is_keventd() for libphy Ben Collins
2006-12-03  9:16 ` Andrew Morton
2006-12-04 19:17   ` Steve Fox
2006-12-05 18:05     ` Maciej W. Rozycki
2006-12-05 17:48   ` Maciej W. Rozycki
2006-12-05 18:07     ` Linus Torvalds
2006-12-05 19:31       ` Andrew Morton
2006-12-05 18:57     ` Andy Fleming
2006-12-06 12:31       ` Maciej W. Rozycki
2006-12-05 20:39     ` Andrew Morton
2006-12-05 20:59       ` Andy Fleming
2006-12-05 21:26         ` Andrew Morton
2006-12-05 21:37           ` Roland Dreier
2006-12-05 21:57             ` Andrew Morton
2006-12-05 23:49               ` Roland Dreier [this message]
2006-12-05 23:52               ` Roland Dreier
2006-12-06 15:25               ` Maciej W. Rozycki
2006-12-06 15:57                 ` Andrew Morton
2006-12-06 17:17                   ` Linus Torvalds
2006-12-06 17:43                     ` David Howells
2006-12-06 17:50                       ` Jeff Garzik
2006-12-06 18:07                         ` Linus Torvalds
2006-12-06 17:53                       ` Linus Torvalds
2006-12-06 17:58                         ` Linus Torvalds
2006-12-06 18:33                           ` Linus Torvalds
2006-12-06 18:37                             ` Linus Torvalds
2006-12-06 18:43                             ` David Howells
2006-12-06 19:02                               ` Linus Torvalds
2006-12-06 18:02                         ` David Howells
2006-12-07  1:21                     ` Linus Torvalds
2006-12-07  6:42                       ` Andrew Morton
2006-12-07  7:49                         ` Andrew Morton
2006-12-07 10:29                           ` David Howells
2006-12-07 10:42                             ` Andrew Morton
2006-12-07 17:05                               ` Jeff Garzik
2006-12-07 17:57                                 ` Andrew Morton
2006-12-07 18:17                                   ` Andrew Morton
2006-12-08 16:52                                   ` [PATCH] group xtime, xtime_lock, wall_to_monotonic, avenrun, calc_load_count fields together in ktimed Eric Dumazet
2006-12-09  5:46                                     ` Andrew Morton
2006-12-09  6:07                                       ` Randy Dunlap
2006-12-11 20:44                                       ` Eric Dumazet
2006-12-11 22:00                                         ` Andrew Morton
2006-12-13 21:26                                     ` [PATCH] Introduce time_data, a new structure to hold jiffies, xtime, xtime_lock, wall_to_monotonic, calc_load_count and avenrun Eric Dumazet
2006-12-15  5:24                                       ` Andrew Morton
2006-12-15 11:21                                         ` Eric Dumazet
2006-12-15 16:21                                       ` Eric Dumazet
2006-12-07 18:08                                 ` [PATCH] Export current_is_keventd() for libphy Maciej W. Rozycki
2006-12-07 18:59                                 ` Andy Fleming
2006-12-07 16:49                         ` Linus Torvalds
2006-12-07 17:52                           ` Andrew Morton
2006-12-07 18:01                             ` Linus Torvalds
2006-12-07 18:16                               ` Andrew Morton
2006-12-07 18:27                                 ` Linus Torvalds
2006-12-07 15:28                       ` Maciej W. Rozycki

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=adairgpc39m.fsf@cisco.com \
    --to=rdreier@cisco.com \
    --cc=afleming@freescale.com \
    --cc=akpm@osdl.org \
    --cc=ben.collins@ubuntu.com \
    --cc=jeff@garzik.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=macro@linux-mips.org \
    --cc=torvalds@osdl.org \
    /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