Messages in this thread | | | From | "Aguirre Rodriguez, Sergio Alberto" <> | Date | Tue, 24 Mar 2009 18:12:49 -0500 | Subject | RE: [PATCH 12/13][Resubmit][drivers/media] changed ioctls to unlocked |
| |
Hi Stoyan,
> -----Original Message----- > From: linux-media-owner@vger.kernel.org [mailto:linux-media- > owner@vger.kernel.org] On Behalf Of stoyboyker@gmail.com > Sent: Tuesday, March 24, 2009 3:39 PM > To: linux-kernel@vger.kernel.org > Cc: Stoyan Gaydarov; linux-media@vger.kernel.org > Subject: [PATCH 12/13][Resubmit][drivers/media] changed ioctls to unlocked > > From: Stoyan Gaydarov <stoyboyker@gmail.com> > > Signed-off-by: Stoyan Gaydarov <stoyboyker@gmail.com> > --- > drivers/media/dvb/bt8xx/dst_ca.c | 7 +++++-- > drivers/media/video/dabusb.c | 11 ++++++++--- > 2 files changed, 13 insertions(+), 5 deletions(-) > > diff --git a/drivers/media/dvb/bt8xx/dst_ca.c > b/drivers/media/dvb/bt8xx/dst_ca.c > index 0258451..d3487c5 100644 > --- a/drivers/media/dvb/bt8xx/dst_ca.c > +++ b/drivers/media/dvb/bt8xx/dst_ca.c > @@ -552,8 +552,10 @@ free_mem_and_exit: > return result; > } > > -static int dst_ca_ioctl(struct inode *inode, struct file *file, unsigned > int cmd, unsigned long ioctl_arg) > +static long dst_ca_ioctl(struct file *file, unsigned int cmd, unsigned > long ioctl_arg) > { > + lock_kernel(); > +
Why moving to unlocked_ioctl if you're acquiring the Big Kernel Lock anyways?
| |