login
Login
/
Register
Search
Search this site:
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2008
»
May
»
19
Re: [PATCH] [2/11] Add unlocked_fasync
view
thread
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
[view in full thread]
From: Christoph Hellwig
Subject:
Re: [PATCH] [2/11] Add unlocked_fasync
Date: Monday, May 19, 2008 - 7:03 am
On Mon, May 19, 2008 at 02:31:11PM +0200, Andi Kleen wrote:
quoted text
> > Add a new fops entry point to allow fasync without BKL. While it's arguably > unclear this entry point is called often enough for it really matters > it was still relatively easy to do. And there are far less async users > in the tree than ioctls so it's likely they can be all converted > eventually and then the non unlocked async entry point could be dropped. > > There was still the problem of the actual flags change being > protected against other setters of flags. Instead of using BKL > for this use the i_mutex now. > > I also added a mutex_lock against one other flags change > that was lockless and could potentially lose updates. > > There are a couple of potential problems I added comments about on.
I'd rather do a properly flag day and move lock_kernel into the instances instead of adding this _unlocked gunk. --
unsubscribe notice
To 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:
[PATCH] [0/11] Repost of old VFS BKL patchkit
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [1/11] Remove BKL from remote_llseek
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [2/11] Add unlocked_fasync
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [3/11] Convert pipe over to unlocked_fasync
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [4/11] Convert socket fasync to unlocked_fasync
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [5/11] Convert fuse to unlocked_fasync
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [6/11] Convert bad_inode to unlocked_fasync
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [7/11] Convert DRM to unlocked_fasync
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [8/11] Use unlocked_fasync in random.c
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [9/11] Convert hpet to unlocked_fasync
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [10/11] Use unlocked_fasync in RTC
, Andi Kleen
, (Mon May 19, 5:31 am)
[PATCH] [11/11] Convert uio to fasync_unlocked
, Andi Kleen
, (Mon May 19, 5:31 am)
Re: [PATCH] [11/11] Convert uio to fasync_unlocked
, Andi Kleen
, (Mon May 19, 5:48 am)
Re: [PATCH] [11/11] Convert uio to fasync_unlocked
, Hans J. Koch
, (Mon May 19, 6:30 am)
Re: [PATCH] [1/11] Remove BKL from remote_llseek
, Trond Myklebust
, (Mon May 19, 6:58 am)
Re: [PATCH] [1/11] Remove BKL from remote_llseek
, Christoph Hellwig
, (Mon May 19, 7:02 am)
Re: [PATCH] [2/11] Add unlocked_fasync
, Christoph Hellwig
, (Mon May 19, 7:03 am)
Re: [PATCH] [1/11] Remove BKL from remote_llseek
, Andi Kleen
, (Mon May 19, 7:41 am)
Re: [PATCH] [2/11] Add unlocked_fasync
, Andi Kleen
, (Mon May 19, 7:43 am)
Re: [PATCH] [2/11] Add unlocked_fasync
, Alan Cox
, (Mon May 19, 8:12 am)
Re: [PATCH] [1/11] Remove BKL from remote_llseek
, Andi Kleen
, (Mon May 19, 8:13 am)
Re: [PATCH] [2/11] Add unlocked_fasync
, Alan Cox
, (Mon May 19, 8:22 am)
Re: [PATCH] [2/11] Add unlocked_fasync
, Andi Kleen
, (Mon May 19, 8:29 am)
Re: [PATCH] [2/11] Add unlocked_fasync
, Andi Kleen
, (Mon May 19, 8:45 am)
Re: [PATCH] [1/11] Remove BKL from remote_llseek
, Steve French
, (Mon May 19, 9:09 am)
Re: [PATCH] [2/11] Add unlocked_fasync
, Randy Dunlap
, (Mon May 19, 10:29 am)
Navigation
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Michael Trimarchi
Re: [PATCH] VFS: make file->f_pos access atomic on 32bit arch
Miklos Szeredi
[patch 14/15] vfs: more path_permission() conversions
Serge E. Hallyn
Re: [RFC v5][PATCH 7/8] Infrastructure for shared objects
Bernd Schmidt
Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3
Takashi Iwai
[PATCH 2/2] input: Add LED support to Synaptics device
git
:
Junio C Hamano
Re: mingw, windows, crlf/lf, and git
Eyvind Bernhardsen
Re: Where has "git ls-remote" reference pattern matching gone?
Shawn O. Pearce
Re: Switching from CVS to GIT
Todd Zullinger
Re: [PATCH 2/2] send-email: rfc2047-quote subject lines with non-ascii characters
Santi Béjar
Re: How to use git-fmt-merge-msg?
linux-netdev
:
Ramkrishna Vepa
[net-2.6 PATCH 1/10] Neterion: New driver: Driver help file
Mark Anthony
invitation / inquiry
Ingo Molnar
Re: [PATCH 08/16] dma-debug: add core checking functions
David Miller
Re: [PATCH 1/3] f_phonet: dev_kfree_skb instead of dev_kfree_skb_any in TX callback
Sascha Hauer
[PATCH 03/12] fec: do not typedef struct types
git-commits-head
:
Linux Kernel Mailing List
amba: struct device - replace bus_id with dev_name(), dev_set_name()
Linux Kernel Mailing List
MIPS: Yosemite: Convert SMP startup lock to arch spinlock.
Linux Kernel Mailing List
ARM: S5PC100: IRQ and timer
Linux Kernel Mailing List
davinci: edma: clear interrupt status for interrupt enabled channels only
Linux Kernel Mailing List
x86, mm, kprobes: fault.c, simplify notify_page_fault()
openbsd-misc
:
Daniel A. Ramaley
Re: [semi-OT] Can anyone recommend an OpenBSD-compatible colour laser printer?
Matthias Kilian
Re: can't get vesa @ 1280x800 or nv
Tobias Ulmer
Re: Problem after upgrade 4.5 to 4.6: ERR M
Philip Guenther
Re: SIGCHLD and libpthread.so
J.C. Roberts
Re: [semi-OT] Can anyone recommend an OpenBSD-compatible colour laser printer?
Colocation donated by:
Syndicate