summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* atmel: Don't use create_proc_read_entry()David Howells2013-04-291-38/+31
| | | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Simon Kelley <simon@thekelleys.org.uk> cc: John W. Linville <linville@tuxdriver.com> cc: linux-wireless@vger.kernel.org cc: netdev@vger.kernel.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* ds1620: Don't use create_proc_read_entry()David Howells2013-04-291-14/+19
| | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Arnd Bergmann <arnd@arndb.de> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* efirtc: Don't use create_proc_read_entry()David Howells2013-04-291-43/+40
| | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Arnd Bergmann <arnd@arndb.de> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* genrtc: Don't use create_proc_read_entry()David Howells2013-04-291-25/+23
| | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Arnd Bergmann <arnd@arndb.de> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* hp_sdc_rtc: Don't use create_proc_read_entry()David Howells2013-04-291-32/+26
| | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Brian S. Julin <bri@calyx.com> cc: Helge Deller <deller@gmx.de> cc: linux-m68k@lists.linux-m68k.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* nubus: Don't use create_proc_read_entry()David Howells2013-04-293-60/+80
| | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: linux-m68k@lists.linux-m68k.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* fsl_udc: Don't use create_proc_read_entry()David Howells2013-04-291-81/+43
| | | | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Li Yang <leoli@freescale.com> cc: Felipe Balbi <balbi@ti.com> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: linux-usb@vger.kernel.org cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* goku_udc: Don't use create_proc_read_entry()David Howells2013-04-291-50/+39
| | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Felipe Balbi <balbi@ti.com> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: linux-usb@vger.kernel.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* wlags49_h2: Don't use create_proc_read_entry()David Howells2013-04-291-159/+161
| | | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Henk de Groot <pe1dnn@amsat.org> cc: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: linux-wireless@vger.kernel.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* cxt1e1: Don't use create_proc_read_entry()David Howells2013-04-293-302/+175
| | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Bob Beers <bob.beers@gmail.com> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: devel@driverdev.osuosl.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* csr: Don't use create_proc_read_entry()David Howells2013-04-293-332/+189
| | | | | | | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: Randy Dunlap <rdunlap@xenotime.net> cc: Priit Laes <plaes@plaes.org> cc: devel@driverdev.osuosl.org cc: Mikko Virkkilä <mikko.virkkila@bluegiga.com> cc: Lauri Hintsala <Lauri.Hintsala@bluegiga.com> cc: Riku Mettälä <riku.mettala@bluegiga.com> cc: Veli-Pekka Peltola <veli-pekka.peltola@bluegiga.com> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* comedi: Don't use create_proc_read_entry()David Howells2013-04-291-21/+31
| | | | | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: David Schleef <ds@schleef.org> cc: Ian Abbott <abbotti@mev.co.uk> cc: Mori Hess <fmhess@users.sourceforge.net> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: H Hartley Sweeten <hsweeten@visionengravers.com> cc: devel@driverdev.osuosl.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* ft1000: Don't use create_proc_read_entry()David Howells2013-04-292-136/+123
| | | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Marek Belisko <marek.belisko@gmail.com> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: Ondrej Zary <linux@rainbow-software.org> cc: devel@driverdev.osuosl.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* rtl8187se: Don't use create_proc_read_entry()David Howells2013-04-291-71/+63
| | | | | | | | | | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Whilst we're at it, reduce the number of show functions where we can share them. Question: Do any of the registers read by proc_get_registers() have side effects upon reading? If so, locking will be required. Signed-off-by: David Howells <dhowells@redhat.com> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: Maxim Mikityanskiy <maxtram95@gmail.com> cc: YAMANE Toshiaki <yamanetoshi@gmail.com> cc: Bill Pemberton <wfp5p@virginia.edu> cc: Andrea Merello <andreamrl@tiscali.it> cc: linux-wireless@vger.kernel.org cc: devel@driverdev.osuosl.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* rtl8192u: Don't use create_proc_read_entry()David Howells2013-04-291-123/+91
| | | | | | | | | | | | | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Whilst we're at it, reduce the number of show functions where we can share them. Note: proc_get_stats_ap() should probably use seq_file iteration rather than list_for_each_entry(). Further note: There appears to be a lot of locking missing in this file to defend against concurrent access by the driver doing normal operations. Notably, ieee->network_list traversal and RWCAM/RCAMO command/response access. Further, do any of the registers read have side effects upon reading? Signed-off-by: David Howells <dhowells@redhat.com> cc: Jerry Chuang <jerry-chuang@realtek.com> cc: Mauro Carvalho Chehab <mchehab@redhat.com> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: linux-wireless@vger.kernel.org cc: devel@driverdev.osuosl.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* procfs: Mark create_proc_read_entry deprecatedDavid Howells2013-04-291-2/+4
| | | | | | | | Mark create_proc_read_entry deprecated. proc_create[_data]() should be used instead. Signed-off-by: David Howells <dhowells@redhat.com> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* xt_hashlimit: allocate a copy of name explicitly, don't rely on procfs gutsAl Viro2013-04-291-3/+11
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* mISDN: grabbing/dropping reference to THIS_MODULE in open/release is racyAl Viro2013-04-291-2/+1
| | | | | | ... when you have no ->owner set. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* mISDN: fix mISDN_read()/mISDN_read() raceAl Viro2013-04-291-9/+14
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* mISDN: fix misdn_add_timer()/misdn_del_timer() raceAl Viro2013-04-291-9/+5
| | | | | | | | | do add_timer() *before* unlocking dev->lock, or unpleasant things can happen if misdn_del_timer() on another CPU finds the sucker, calls del_timer_sync() (which does nothing, since we hadn't started the timer yet) and frees it, just as we get around to add_timer()... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* mISDN: fix races between misdn_del_timer() and timer callbackAl Viro2013-04-291-13/+9
| | | | | | | mark the victim with negative ->id if misdn_del_timer() finds it on the list, have timer callback *not* move ones so marked to dev->expired Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* mISDN: fix the races with timers going off just as they are deletedAl Viro2013-04-291-2/+12
| | | | | | | | | timer callback in timerdev.c both accesses struct mISDNtimer it's called for *and* moves it to dev->expired. We need del_timer_sync(), or we risk kfree() freeing it right under dev_expire_timer() *and* dev->expired getting corrupted. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* dvb_frontend: don't mess with ->f_op in ->release()Al Viro2013-04-291-4/+1
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* dvb_net: don't mess with ->f_op in ->release()Al Viro2013-04-291-4/+1
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* lis3lv02d: don't wank with fasync() on ->release()Al Viro2013-04-291-1/+0
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* mpt3sas: don't wank with fasync on ->release()Al Viro2013-04-291-14/+0
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* mpt2sas: don't wank with fasync on ->release()Al Viro2013-04-291-14/+0
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* fix a leak in /proc/schedstatsAl Viro2013-04-291-6/+1
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* pmcraid: don't wank with fasync in ->release()Al Viro2013-04-291-14/+0
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* mptctl: don't wank with fasync in ->release()Al Viro2013-04-291-8/+0
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* lpfc: fix races for miscdevice open vs. rmmodAl Viro2013-04-291-32/+1
| | | | | | | | mind you, I'm not sure WTF would anybody _need_ that miscdevice at all - no IO is possible for it, opening it only pins the module down and is seriously racy, at that. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* yurex: don't wank with fasync on ->release()...Al Viro2013-04-291-2/+0
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* fanotify: don't wank with FASYNC on ->release()Al Viro2013-04-291-3/+0
| | | | | | ... it's done already by __fput() Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* sock_close() couldn't have been called with NULL inode since at least 2.1.earlyAl Viro2013-04-291-9/+0
| | | | | | ... if not since 0.99 or so. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* tomoyo_close_control: don't bother with return valueAl Viro2013-04-293-8/+4
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* dmasound_core: saner arguments for sq_fsync()Al Viro2013-04-291-3/+3
| | | | | | it is not (and it has never been) an ->fsync() instance... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* hppfs: get rid of ->fsync()Al Viro2013-04-291-7/+0
| | | | | | | it has grown by accident - directories there do *not* use page cache, so there's nothing to write. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* hppfs: fix the leaks on close()Al Viro2013-04-291-1/+12
| | | | | | we need to close the underlying procfs file and free ->private_data Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* new helper: read_code()Al Viro2013-04-296-67/+42
| | | | | | | | switch binfmts that use ->read() to that (and to kernel_read() in several cases in binfmt_flat - sure, it's nommu, but still, doing ->read() into kmalloc'ed buffer...) Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* cx25821: sanitize cx25821_openfile_audio() a bit...Al Viro2013-04-131-68/+28
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* cx25821: sanitize cx25821_get_audio_data() a bitAl Viro2013-04-131-58/+25
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* try a saner locking for pde_opener...Al Viro2013-04-093-44/+24
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* deal with races between remove_proc_entry() and proc_reg_release()Al Viro2013-04-092-53/+34
| | | | | | | * serialize the call of ->release() on per-pdeo mutex * don't remove pdeo from per-pde list until we are through with it Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* procfs: preparations for remove_proc_entry() race fixesAl Viro2013-04-094-214/+135
| | | | | | | | * leave ->proc_fops alone; make ->pde_users negative instead * trim pde_opener * move relevant code in fs/proc/inode.c Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* procfs: Clean up huge if-statement in __proc_file_read()David Howells2013-04-091-51/+47
| | | | | | | | | Switch huge if-statement in __proc_file_read() around. This then puts the single line loop break immediately after the if-statement and allows us to de-indent the huge comment and make it take fewer lines. The code following the if-statement then follows naturally from the call to dp->read_proc(). Signed-off-by: David Howells <dhowells@redhat.com>
* vt6655: slightly clean reading config fileAl Viro2013-04-091-75/+40
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* vt6656: slightly sanitized reading configAl Viro2013-04-091-43/+20
| | | | | | | Just reading - parsing the results is left alone (and unspeakably lousy). Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* sound_firmware: don't bother with filp_close()Al Viro2013-04-091-4/+5
| | | | | | | it's opened read-only and never installed into any descriptor tables; fput() will do just as well. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* pxa3xx-gcu: stop embedding file_operations into device-private objectAl Viro2013-04-091-7/+8
| | | | | | no need to do that anymore... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* pxa3xx-gcu: quite playing silly buggers with ->f_opAl Viro2013-04-091-9/+12
| | | | | | | | misc device gets ->private_data pointing to struct miscdevice on open(), so we can use that to get to per-device structure instead of relying on file_operations being copied into it. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
OpenPOWER on IntegriCloud