| Date | Tue, 8 Jan 2008 15:22:45 -0500 | From | Rik van Riel <> | Subject | Re: [JANITOR PROPOSAL] Switch ioctl functions to ->unlocked_ioctl |
| |
On Tue, 8 Jan 2008 20:58:04 +0100 "Paolo Ciarrocchi" <paolo.ciarrocchi@gmail.com> wrote:
> -static const struct file_operations rtc_fops = { > +static long rtc_fioctl(struct file_operations rtc_fops) > +{ > + lock_kernel(); > .owner = THIS_MODULE, > .llseek = no_llseek, > .read = rtc_read, > .poll = rtc_poll, > - .ioctl = rtc_ioctl, > + .unlocked_ioctl = rtc_ioctl, > .open = rtc_open, > .release = rtc_release, > .fasync = rtc_fasync, > + unlock_kernel(); > };
This is a struct with function pointers, not a function.
No wonder it doesn't compile after you tried turning it into one :)
-- All rights reversed.
|