Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlocked_ioctl | KernelTrap

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlocked_ioctl

view
thread

!MAILaRCHIVE_VOTE_RePLACE

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

[view in full thread]

From: Andi Kleen <andi@...>

To: Rolf Eike Beer <eike-kernel@...>

Cc: Andi Kleen <andi@...>, <linux-kernel@...>, <kernel-janitors@...>, <paolo.ciarrocchi@...>, <gorcunov@...>

Subject: Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlocked_ioctl

Date: Thursday, January 10, 2008 - 5:25 am

> Can you explain the rationale behind that running on the BKL? What type of 

It used to always run with the BKL because everything used to 
and originally nobody wanted to review all ioctl handlers in tree to see if 
they can run with more fine grained locking. A lot probably can though.

quoted text> things needs to be protected so that this huge hammer is needed? What would 
> be an earlier point to release the BKL?

That depends on the driver. A lot don't need BKL at all and 
in others it can be easily eliminated. But it needs case-by-case
review of the locking situation.

The goal of the proposal here is just to make it more visible.

-Andi
--
unsubscribe noticeTo unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:

[JANITOR PROPOSAL] Switch ioctl functions to ->unlocked_ioctl, Andi Kleen, (Tue Jan 8, 12:40 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Rolf Eike Beer, (Thu Jan 10, 4:52 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Thu Jan 10, 5:25 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Rolf Eike Beer, (Thu Jan 10, 6:02 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Thu Jan 10, 6:06 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andre Noll, (Wed Jan 9, 6:34 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Richard Knutsson, (Wed Jan 9, 9:17 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andre Noll, (Wed Jan 9, 9:33 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Kevin Winchester, (Tue Jan 8, 7:50 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Tue Jan 8, 8:09 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Kevin Winchester, (Tue Jan 8, 8:17 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Tue Jan 8, 8:27 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Paolo Ciarrocchi, (Tue Jan 8, 3:58 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Pavel Machek, (Fri Jan 11, 4:33 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Matt Mackall, (Wed Jan 9, 4:12 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Alasdair G Kergon, (Wed Jan 9, 6:40 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Daniel Phillips, (Thu Jan 10, 5:49 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Alasdair G Kergon, (Thu Jan 10, 7:39 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Daniel Phillips, (Thu Jan 10, 6:55 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Wed Jan 9, 6:46 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Alasdair G Kergon, (Wed Jan 9, 6:45 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Christoph Hellwig, (Thu Jan 10, 4:34 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Chris Friesen, (Wed Jan 9, 6:58 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Alasdair G Kergon, (Wed Jan 9, 7:05 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Vadim Lobanov, (Wed Jan 9, 7:31 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Alasdair G Kergon, (Wed Jan 9, 8:00 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Vadim Lobanov, (Thu Jan 10, 12:59 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Tue Jan 8, 4:42 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Tue Jan 8, 7:06 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Paolo Ciarrocchi, (Tue Jan 8, 7:43 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Tue Jan 8, 8:03 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Paolo Ciarrocchi, (Tue Jan 8, 4:45 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Rik van Riel, (Tue Jan 8, 4:22 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Matthew Wilcox, (Tue Jan 8, 4:00 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Paolo Ciarrocchi, (Tue Jan 8, 4:03 pm)

supervising, text processing, semantic "patching" (Re: [JANI..., Oleg Verych, (Thu Mar 6, 10:54 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Matthew Wilcox, (Tue Jan 8, 4:16 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Matthew Wilcox, (Tue Jan 8, 4:21 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Dmitri Vorobiev, (Tue Jan 8, 7:55 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Paolo Ciarrocchi, (Tue Jan 8, 4:26 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Alexey Dobriyan, (Tue Jan 8, 2:52 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Tue Jan 8, 3:18 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Arnd Bergmann, (Tue Jan 8, 8:40 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Tue Jan 8, 8:47 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Junio C Hamano, (Wed Jan 9, 6:00 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Arnd Bergmann, (Tue Jan 8, 9:19 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Kevin Winchester, (Tue Jan 8, 9:31 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Andi Kleen, (Tue Jan 8, 9:41 pm)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Christoph Hellwig, (Wed Jan 9, 4:02 am)

Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlock..., Cyrill Gorcunov, (Tue Jan 8, 1:05 pm)

Navigation

Popular discussions


linux-kernel:
Tarkan Erimer	Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3
Greg KH	[GIT PATCH] driver core patches against 2.6.24
Mike Travis	[RFC 00/15] x86_64: Optimize percpu accesses
Dave Jones	agp / cpufreq.
linux-netdev:
Willy Tarreau	Re: [PATCH] tcp: splice as many packets as possible at once
Gerrit Renker	[PATCH 14/37] dccp: Tidy up setsockopt calls
David Miller	Re: [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock().
Natalie Protasevich	[BUG] New Kernel Bugs
git:

openbsd-misc:

Colocation donated by:

Who's online

There are currently 2 users and 601 guests online.

Online users

Syndicate