From: Arnd Bergmann <arnd@arndb•de>
To: Dave Airlie <airlied@gmail•com>
Cc: Theodore Kilgore <kilgota@banach•math.auburn.edu>,
Steven Rostedt <rostedt@goodmis•org>, Greg KH <greg@kroah•com>,
codalist@telemann•coda.cs.cmu.edu, autofs@linux•kernel.org,
Samuel Ortiz <samuel@sortiz•org>, Jan Kara <jack@suse•cz>,
Mikulas Patocka <mikulas@artax•karlin.mff.cuni.cz>,
Jan Harkes <jaharkes@cs•cmu.edu>,
netdev@vger•kernel.org, Anders Larsen <al@alarsen•net>,
linux-kernel@vger•kernel.org, dri-devel@lists•freedesktop.org,
Bryan Schumaker <bjschuma@netapp•com>,
Christoph Hellwig <hch@infradead•org>,
ksummit-2010-discuss@lists•linux-foundation.org,
Petr Vandrovec <vandrove@vc•cvut.cz>,
Arnaldo Carvalho de Melo <acme@ghostprotocols•net>,
linux-fsdevel@vger•kernel.org,
Evgeniy Dushistov <dushistov@mail•ru>,
Ingo Molnar <mingo@elte•hu>,
Andrew Hendry <andrew.hendry@gmail•com>,
linux-media@vger•kernel.org
Subject: Re: [Ksummit-2010-discuss] [v2] Remaining BKL users, what to do
Date: Tue, 19 Oct 2010 09:26:54 +0200 [thread overview]
Message-ID: <201010190926.54635.arnd@arndb.de> (raw)
In-Reply-To: <AANLkTi=oAeuz8ZxcOMpf=3MVY=WMt0BwHiGCUxO7OAEV@mail.gmail.com>
On Tuesday 19 October 2010 06:52:32 Dave Airlie wrote:
> > I might be able to find some hardware still lying around here that uses an
> > i810. Not sure unless I go hunting it. But I get the impression that if
> > the kernel is a single-CPU kernel there is not any problem anyway? Don't
> > distros offer a non-smp kernel as an installation option in case the user
> > needs it? So in reality how big a problem is this?
>
> Not anymore, which is my old point of making a fuss. Nowadays in the
> modern distro world, we supply a single kernel that can at runtime
> decide if its running on SMP or UP and rewrite the text section
> appropriately with locks etc. Its like magic, and something like
> marking drivers as BROKEN_ON_SMP at compile time is really wrong when
> what you want now is a runtime warning if someone tries to hotplug a
> CPU with a known iffy driver loaded or if someone tries to load the
> driver when we are already in SMP mode.
We could make the driver run-time non-SMP by adding
if (num_present_cpus() > 1) {
pr_err("i810 no longer supports SMP\n");
return -EINVAL;
}
to the init function. That would cover the vast majority of the
users of i810 hardware, I guess.
Arnd
next prev parent reply other threads:[~2010-10-19 7:26 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-16 14:32 Remaining BKL users, what to do Arnd Bergmann
2010-09-16 14:49 ` Steven Rostedt
2010-09-16 18:32 ` Jens Axboe
2010-09-17 18:46 ` Arnd Bergmann
2010-09-16 15:04 ` Jan Kara
2010-09-16 21:26 ` Anton Altaparmakov
2010-09-17 10:45 ` Arnd Bergmann
2010-09-17 13:32 ` Christoph Hellwig
2010-09-17 13:50 ` Arnd Bergmann
2010-09-17 14:02 ` Christoph Hellwig
2010-09-17 14:56 ` Arnd Bergmann
2010-09-16 15:07 ` Alan Cox
2010-09-16 20:08 ` David Miller
2010-09-16 16:09 ` Anders Larsen
2010-09-16 16:57 ` Samuel Ortiz
2010-09-16 20:08 ` David Miller
2010-09-20 1:25 ` [autofs] " Ian Kent
2010-10-18 15:42 ` [v2] " Arnd Bergmann
2010-10-18 16:19 ` Christoph Hellwig
2010-10-18 17:38 ` Arnd Bergmann
2010-10-18 18:43 ` [Ksummit-2010-discuss] " Greg KH
2010-10-18 23:00 ` Dave Airlie
2010-10-19 0:40 ` Greg KH
2010-10-19 0:57 ` Dave Airlie
2010-10-19 2:24 ` Greg KH
2010-10-19 2:45 ` Dave Airlie
2010-10-19 3:33 ` Steven Rostedt
2010-10-19 4:03 ` Dave Airlie
2010-10-19 5:00 ` Theodore Kilgore
2010-10-19 4:52 ` Dave Airlie
2010-10-19 7:26 ` Arnd Bergmann [this message]
2010-10-19 12:39 ` Steven Rostedt
2010-10-19 13:54 ` Paul Mundt
2010-10-19 13:26 ` Arnd Bergmann
2010-10-19 20:50 ` Dave Airlie
2010-10-20 16:14 ` Ville Syrjälä
2010-10-19 18:24 ` Valdis.Kletnieks
2010-10-19 19:37 ` Greg KH
2010-10-19 19:40 ` Oliver Neukum
2010-10-19 20:29 ` Greg KH
2010-10-19 20:38 ` Jiri Kosina
2010-10-19 20:41 ` Alan Cox
2010-10-19 20:48 ` Arnd Bergmann
2010-10-19 20:44 ` Arnd Bergmann
2010-10-20 4:43 ` Dave Young
2010-10-20 6:50 ` Arnd Bergmann
2010-11-02 1:21 ` Pavel Machek
2010-11-03 6:58 ` Pekka Enberg
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=201010190926.54635.arnd@arndb.de \
--to=arnd@arndb$(echo .)de \
--cc=acme@ghostprotocols$(echo .)net \
--cc=airlied@gmail$(echo .)com \
--cc=al@alarsen$(echo .)net \
--cc=andrew.hendry@gmail$(echo .)com \
--cc=autofs@linux$(echo .)kernel.org \
--cc=bjschuma@netapp$(echo .)com \
--cc=codalist@telemann$(echo .)coda.cs.cmu.edu \
--cc=dri-devel@lists$(echo .)freedesktop.org \
--cc=dushistov@mail$(echo .)ru \
--cc=greg@kroah$(echo .)com \
--cc=hch@infradead$(echo .)org \
--cc=jack@suse$(echo .)cz \
--cc=jaharkes@cs$(echo .)cmu.edu \
--cc=kilgota@banach$(echo .)math.auburn.edu \
--cc=ksummit-2010-discuss@lists$(echo .)linux-foundation.org \
--cc=linux-fsdevel@vger$(echo .)kernel.org \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-media@vger$(echo .)kernel.org \
--cc=mikulas@artax$(echo .)karlin.mff.cuni.cz \
--cc=mingo@elte$(echo .)hu \
--cc=netdev@vger$(echo .)kernel.org \
--cc=rostedt@goodmis$(echo .)org \
--cc=samuel@sortiz$(echo .)org \
--cc=vandrove@vc$(echo .)cvut.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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