Re: [PATCH] f_op->poll() without lock_kernel()

From: Alexander Viro (viro@math.psu.edu)
Date: Fri Apr 21 2000 - 17:13:16 EST


On Fri, 21 Apr 2000, David S. Miller wrote:

> Date: Fri, 21 Apr 2000 23:49:45 +0200
> From: Manfred Spraul <manfreds@colorfullife.com>
>
> How did you grep? You missed
>
> drivers/char/qpmouse.c:
> poll_pq() looks good, but read_qp() needs lock_kernel.
> [while(!queue_empty()) { get_from_queue()} ]
>
> I used "*_poll", and filtered out the noise by hand.
>
> I think poll_pq should be renamed to pq_poll, so that it uses a naming
> convention in sync with what everyone else does.

What _should_ be done is the conversion of initializers to new syntax.
BTW, IWBNI the same would be done for proto/proto_ops. Dave, would you
accept the net/*/* side of such patch? I can send it in an hour or so.
Block devices, fs_struct, super_operations, file_operations in
filesystems, dentry_operations and inode_operations are already there.
Linus, will you accept the patch for character devices? Let's finish the
stinker and make sure that _all_ methods tables are initialized
        a) statically
        b) via the field: value; syntax.
Will make for much simpler searches...
                                                        Al

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Apr 23 2000 - 21:00:19 EST