All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: corbet@lwn.net (Jonathan Corbet)
Cc: video4linux-list@redhat.com, linux-kernel@vger.kernel.org,
	mchehab@infradead.org
Subject: Re: [PATCH] video4linux: Push down the BKL
Date: Fri, 23 May 2008 16:39:56 +0100	[thread overview]
Message-ID: <20080523163956.6e93746c@core> (raw)
In-Reply-To: <9027.1211551014@vena.lwn.net>

> On the other hand, the next level of BKL pushdown would be painful as
> all hell, given the massive number of callbacks in the V4L2 API.  So I'm
> thinking it might be justified to create a video_ioctl2_locked() for
> V4L2 drivers which are not yet known to be safe in the absence of the
> BKL.  The amount of extra code would be quite small, and it would let
> safe drivers operate BKL-free.

The problem is that currently they are almost all unsafe - I did a quick
survey as part of the changes. Pushing it down to the video2_ioctl is a
starting point, but the v4l layer is going to need a lot of love and its
own gradual migration.

Right now we've gone from BKL buried in fs to BKL at top of v4l layer
which is indeed only a starting point. I'd assumed the same as you are
think a new video_ioctl2 and switching drivers one by one (I was assuming
adding video2_ioctl_unlocked())

Alan

WARNING: multiple messages have this Message-ID (diff)
From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: corbet@lwn.net (Jonathan Corbet)
Cc: video4linux-list@redhat.com, linux-kernel@vger.kernel.org,
	mchehab@infradead.org
Subject: Re: [PATCH] video4linux: Push down the BKL
Date: Fri, 23 May 2008 16:39:56 +0100	[thread overview]
Message-ID: <20080523163956.6e93746c@core> (raw)
In-Reply-To: <9027.1211551014@vena.lwn.net>

> On the other hand, the next level of BKL pushdown would be painful as
> all hell, given the massive number of callbacks in the V4L2 API.  So I'm
> thinking it might be justified to create a video_ioctl2_locked() for
> V4L2 drivers which are not yet known to be safe in the absence of the
> BKL.  The amount of extra code would be quite small, and it would let
> safe drivers operate BKL-free.

The problem is that currently they are almost all unsafe - I did a quick
survey as part of the changes. Pushing it down to the video2_ioctl is a
starting point, but the v4l layer is going to need a lot of love and its
own gradual migration.

Right now we've gone from BKL buried in fs to BKL at top of v4l layer
which is indeed only a starting point. I'd assumed the same as you are
think a new video_ioctl2 and switching drivers one by one (I was assuming
adding video2_ioctl_unlocked())

Alan

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

  reply	other threads:[~2008-05-23 15:55 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-22 21:37 [PATCH] video4linux: Push down the BKL Alan Cox
2008-05-22 21:37 ` Alan Cox
2008-05-23  2:08 ` Andy Walls
2008-05-23  2:08   ` Andy Walls
2008-05-23  6:16   ` Hans Verkuil
2008-05-23  6:16     ` Hans Verkuil
2008-05-23  6:28     ` Hans Verkuil
2008-05-23  6:28       ` Hans Verkuil
2008-05-26 16:39     ` Mauro Carvalho Chehab
2008-05-26 16:39       ` Mauro Carvalho Chehab
2008-05-23  9:09   ` Alan Cox
2008-05-23  9:09     ` Alan Cox
2008-05-26 16:34     ` Mauro Carvalho Chehab
2008-05-26 16:34       ` Mauro Carvalho Chehab
2008-05-26 16:46       ` Hans Verkuil
2008-05-26 16:46         ` Hans Verkuil
2008-05-26 21:14         ` Mauro Carvalho Chehab
2008-05-26 21:14           ` Mauro Carvalho Chehab
2008-06-01  2:34         ` [PATCH] cx18: convert driver to video_ioctl2() (Re: [PATCH] video4linux: Push down the BKL) Andy Walls
2008-06-01 10:15           ` Hans Verkuil
2008-06-01 19:01             ` Andy Walls
2008-06-03 21:20               ` Mauro Carvalho Chehab
2008-06-03 22:08                 ` [ivtv-devel] " Alan Cox
2008-06-04  0:44                   ` Andy Walls
2008-06-04 10:02                     ` Alan Cox
2008-05-23 13:56 ` [PATCH] video4linux: Push down the BKL Jonathan Corbet
2008-05-23 15:39   ` Alan Cox [this message]
2008-05-23 15:39     ` Alan Cox
2008-05-23 16:09     ` Jonathan Corbet
2008-05-23 18:58       ` Alan Cox
2008-05-23 18:58         ` Alan Cox
2008-05-23 19:05 ` Hans Verkuil
2008-05-23 19:05   ` Hans Verkuil
2008-05-25 23:46 ` Mike Isely
2008-05-25 23:46   ` Mike Isely
2008-05-26 16:59 ` Mauro Carvalho Chehab
2008-05-26 16:59   ` Mauro Carvalho Chehab
2008-05-26 20:23   ` Alan Cox
2008-05-26 20:23     ` Alan Cox
2008-05-26 21:10     ` Mauro Carvalho Chehab
2008-05-26 21:10       ` Mauro Carvalho Chehab
2008-05-26 22:01       ` Alan Cox
2008-05-26 22:01         ` Alan Cox
2008-05-27 13:10         ` Mauro Carvalho Chehab
2008-05-27 13:10           ` Mauro Carvalho Chehab
2008-05-27 15:41           ` Jonathan Corbet
2008-05-27 16:31             ` Mauro Carvalho Chehab
2008-05-27 16:31               ` Mauro Carvalho Chehab
2008-05-27 16:37               ` Jonathan Corbet
2008-05-27 18:59                 ` Mauro Carvalho Chehab
2008-05-27 18:59                   ` Mauro Carvalho Chehab
2008-05-27 19:26                   ` Devin Heitmueller
2008-05-27 19:26                     ` Devin Heitmueller
2008-05-27 21:00                     ` Mauro Carvalho Chehab
2008-05-27 21:00                       ` Mauro Carvalho Chehab
2008-05-27 21:22                       ` Devin Heitmueller
2008-05-27 21:22                         ` Devin Heitmueller
2008-05-27 23:48                       ` Andy Walls
2008-05-27 23:48                         ` Andy Walls
2008-05-28  0:46                         ` Devin Heitmueller
2008-05-28  0:46                           ` Devin Heitmueller
2008-05-28  2:37                           ` Andy Walls
2008-05-28  2:37                             ` Andy Walls
2008-05-28  2:47                             ` Devin Heitmueller
2008-05-28  2:47                               ` Devin Heitmueller
2008-05-28 23:30                               ` Andy Walls
2008-05-28  8:34                             ` Alan Cox
2008-05-28  8:34                               ` Alan Cox
2008-05-28  6:13                           ` Hans Verkuil
2008-05-28  6:13                             ` Hans Verkuil
2008-05-27 19:50                   ` Arjan van de Ven
2008-05-27 20:24                     ` Mauro Carvalho Chehab
2008-05-27 20:24                       ` Mauro Carvalho Chehab
2008-05-27 18:14               ` Alan Cox
2008-05-27 18:14                 ` Alan Cox

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080523163956.6e93746c@core \
    --to=alan@lxorguk.ukuu.org.uk \
    --cc=corbet@lwn.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mchehab@infradead.org \
    --cc=video4linux-list@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.