summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* 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>
* silicom-bypass: Remove device_open/close()David Howells2013-04-091-24/+0
| | | | | | | Remove device_open/close() functions as they don't really do anything and remove Device_Open as it isn't counted atomically and the value isn't used. Signed-off-by: David Howells <dhowells@redhat.com>
* proc: Kill create_proc_entry()David Howells2013-04-092-18/+8
| | | | | | | Kill create_proc_entry() in favour of create_proc_read_entry(), proc_create() and proc_create_data(). Signed-off-by: David Howells <dhowells@redhat.com>
* wlags49_h2: Don't use create_proc_entry()David Howells2013-04-091-11/+0
| | | | | | | | create_proc_entry() shouldn't be used. Rather proc_create_data() should be used. The proc_write() function is only used by #if'd out code, so delete it for now. Signed-off-by: David Howells <dhowells@redhat.com>
* nubus: Don't use create_proc_entry()David Howells2013-04-091-2/+3
| | | | | | | | Don't use create_proc_entry() in nubus_proc_subdir(). The files created aren't given any way to use them, so for the moment use create_proc_read_entry() with a NULL accessor and generate a compile-time warning. Signed-off-by: David Howells <dhowells@redhat.com>
* UM: Adjust printk in create_proc_mconsole()David Howells2013-04-091-2/+1
| | | | | | | Adjust printk in create_proc_mconsole() to reflect it is now using proc_create() not create_proc_mconsole(). Signed-off-by: David Howells <dhowells@redhat.com>
* constify a bunch of struct file_operations instancesAl Viro2013-04-0913-17/+17
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* bluetooth: kill unused fops field in struct bt_sock_listAl Viro2013-04-091-1/+0
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* bluetooth: fix race in bt_procfs_init()Al Viro2013-04-091-7/+1
| | | | | | | use proc_create_data() rather than set ->data after the file has been created Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* bluetooth: kill unused 'module' argument of bt_procfs_init()Al Viro2013-04-099-10/+10
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* bluetooth: don't bother with ->owner for procfs fopsAl Viro2013-04-091-7/+8
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* x25: use proc_remove_subtree()Al Viro2013-04-091-33/+14
| | | | | | | ... and don't bother with ->owner, while we are at it - procfs fops do not need it. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* create_proc_cpu_mask() doesn't need an argument...Al Viro2013-04-093-8/+6
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* lparcfg: don't bother saving pointer to proc_dir_entryAl Viro2013-04-091-9/+2
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* prominfo_proc fixesAl Viro2013-04-091-30/+12
| | | | | | | * check for proc_mkdir() failures * use remove_proc_subtree() Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* srm_env: don't bother with pointer to srm_env_tAl Viro2013-04-091-24/+9
| | | | | | | Since the only thing in it the methods actually care about is variable id, just store that directly. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* srm_env: use proc_remove_subtree()Al Viro2013-04-091-57/+7
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* procfs: new helper - PDE_DATA(inode)Al Viro2013-04-0995-302/+243
| | | | | | | | | | The only part of proc_dir_entry the code outside of fs/proc really cares about is PDE(inode)->data. Provide a helper for that; static inline for now, eventually will be moved to fs/proc, along with the knowledge of struct proc_dir_entry layout. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* last_radio_log: switch to proc_create()Al Viro2013-04-091-16/+4
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* procfs: kill ->write_proc()Al Viro2013-04-092-26/+0
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* bfin_adv7393: switch to proc_create_data()Al Viro2013-04-091-24/+19
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* xtensa simdisk: switch to proc_create_data()Al Viro2013-04-091-13/+23
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* rtl8192u: switch to proc_create()Al Viro2013-04-091-31/+22
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* scsi_proc: make proc_scsi_host_open() preallocate a bigger bufferAl Viro2013-04-091-1/+2
| | | | | | | | | Some of the ->show_info() instances really spew a lot; it's not a problem wrt correctness (seq_read() will grow buffer and call the sucker again), but in this case it makes sense to start with a somewhat bigger one - they often do exceed one page worth of output. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* new helper: single_open_size()Al Viro2013-04-092-0/+19
| | | | | | | | | | | | | | | Same as single_open(), but preallocates the buffer of given size. Doesn't make any sense for sizes up to PAGE_SIZE and doesn't make sense if output of show() exceeds PAGE_SIZE only rarely - seq_read() will take care of growing the buffer and redoing show(). If you _know_ that it will be large, it might make more sense to look into saner iterator, rather than go with single-shot one. If that's impossible, single_open_size() might be for you. Again, don't use that without a good reason; occasionally that's really the best way to go, but very often there are better solutions. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* scsi: bury ->proc_info()Al Viro2013-04-092-73/+9
| | | | | | all users converted to ->show_info()/->write_info() Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* atari_scsi: switch to ->show_info()Al Viro2013-04-093-87/+62
| | | | Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>