Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlocked_ioctl

Arnd Bergmann
2008-01-09T01:20:05Z

On Wednesday 09 January 2008, Andi Kleen wrote:
> I imagined it would check for
>
> +struct file_operations ... = {
> +      ...
> +       .ioctl = ...
>
> That wouldn't catch the case of someone adding only .ioctl to an
> already existing file_operations which is not visible in the patch context,
> but that should be hopefully rare. The more common case is adding
> completely new operations

Right, this would work fine. We can probably even have a list of
data structures that work like file_operations in this regard.