Messages in this thread | | | Subject | Re: [PATCH] Fix proc_file_write missing ppos update | From | Stefani Seibold <> | Date | Sun, 30 Aug 2009 21:05:44 +0200 |
| |
Am Samstag, den 29.08.2009, 19:16 -0400 schrieb Christoph Hellwig: > On Sat, Aug 29, 2009 at 06:38:12PM +0200, Stefani Seibold wrote: > > The following fix a long standing issue in the proc_file_write function, > > which doesn't update the ppos file position pointer. > > The right fix is to get rid of the last remaining > read_proc_t/write_proc_t instances and switch everyone to implement file > operations. Alexey has been working on this on and off for a while. >
Switching all users of read_proc_t/write_proc_t to file operation is a huge job. About 180 files must be fixed.
But the main reason not to do this is because the breakage of "out of tree" drivers.
I like the current simplified proc interface. It saves a lot of code duplication because the basic operations will be handled inside the kernel and not in the driver.
There is no reason for Alexey to finish his work, submit it and maybe it will be accepted.
In the meantime i will see my patch as the preferred solution: a cleanup of the current interface.
| |