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: Daniel Phillips <phillips@...>

To: Alasdair G Kergon <agk@...>

Cc: Matt Mackall <mpm@...>, Paolo Ciarrocchi <paolo.ciarrocchi@...>, Andi Kleen <andi@...>, <linux-kernel@...>, <kernel-janitors@...>, <gorcunov@...>

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

Date: Thursday, January 10, 2008 - 6:55 pm

On Thursday 10 January 2008 03:39, Alasdair G Kergon wrote:
quoted text> On Thu, Jan 10, 2008 at 01:49:15AM -0800, Daniel Phillips wrote:
> > So what stops you from changing to unlocked_ioctl for the main
> > device mapper ctl_ioctl?
>
> Nothing - patches to do this are queued for 2.6.25:

Nice.  This removes a deadlock we hit, where if creating a device mapper 
target blocks indefinitely (say on network IO) then nobody else can 
complete a device mapper operation because BKL is held.  If completing 
the device create depends on some other device mapper operation, then 
it is game over.

Our current workaround is to test for and drop BKL, ugh.  Thanks for the 
cleanup.

Regards,

Daniel
--
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:
Greg KH	[GIT PATCH] driver core patches against 2.6.24
Justin C. Sherrill	Re: pkgsrc bulk build and tiff
Jeremy Allison	Re: [RFC] Heads up on sys_fallocate()
Roland Dreier	Re: Integration of SCST in the mainstream Linux kernel
netbsd-tech-kern:
Matt Thomas	Re: Add a MAP_ALIGNED flag for mmap(2).
Vsevolod Stakhov	Unicode support in iso9660.
Jaromir Dolecek	Re: Speeding up fork/wait path
matthew green	re: merge of freebsd eventhandler
git:

dragonflybsd-user:
Petr Janda	KDE and OpenSSL = Broken
sam	Re: Loader not found
Erick Perez	Re: dragonfly pdf documentation
Michel Talon	Re: Compatability with FreeBSD Ports [debian package tools]

Colocation donated by:

Who's online

There are currently 1 user and 876 guests online.

Online users

domainreg5

Syndicate