linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RFCv3 PATCH 00/33] Core and vb2 enhancements
@ 2012-06-28  6:47 Hans Verkuil
  2012-06-28  6:47 ` [RFCv3 PATCH 01/33] v4l2-ioctl.c: move a block of code down, no other changes Hans Verkuil
  2012-07-20  9:55 ` [RFCv3 PATCH 00/33] Core and vb2 enhancements Guennadi Liakhovetski
  0 siblings, 2 replies; 39+ messages in thread
From: Hans Verkuil @ 2012-06-28  6:47 UTC (permalink / raw)
  To: linux-media
  Cc: Tomasz Stanislawski, Pawel Osciak, Guennadi Liakhovetski,
	Laurent Pinchart, Hans de Goede, Mauro Carvalho Chehab

Hi all,

This is the third version of this patch series.

The first version is here:

http://www.mail-archive.com/linux-media@vger.kernel.org/msg47558.html

Changes since RFCv2:

- Rebased to staging/for_v3.6.

- Incorporated Laurent's review comments in patch 22: vb2-core: refactor reqbufs/create_bufs.

Changes since RFCv1:

- Incorporated all review comments from Hans de Goede and Laurent Pinchart (Thanks!)
  except for splitting off the vb2 helper functions into a separate source. I decided
  to keep it together with the vb2-core code.

- Improved commit messages, added more comments to the code.

- The owner filehandle and the queue lock are both moved to struct vb2_queue since
  these are a property of the queue.

- The debug function has a new 'write_only' boolean: some debug functions can only
  print a subset of the arguments if it is called by an _IOW ioctl. The previous
  patch series split this up into two functions. Handling the debug function for
  a write-only ioctl is annoying at the moment: you have to print the arguments
  before calling the ioctl since the ioctl can overwrite arguments. I am considering
  changing the op argument to const for such ioctls and see if any driver is
  actually messing around with the contents of such structs. If we can guarantee
  that drivers do not change the argument struct, then we can simplify the debug
  code.

- All debugging is now KERN_DEBUG instead of KERN_INFO.

I still have one outstanding question: should anyone be able to call mmap() or
only the owner of the vb2 queue? Right now anyone can call mmap().

Comments are welcome, but if I don't see any in the next 2-3 days, then I'll make
a pull request for this on Sunday.

Regards,

        Hans


^ permalink raw reply	[flat|nested] 39+ messages in thread

end of thread, other threads:[~2012-07-20 10:14 UTC | newest]

Thread overview: 39+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-06-28  6:47 [RFCv3 PATCH 00/33] Core and vb2 enhancements Hans Verkuil
2012-06-28  6:47 ` [RFCv3 PATCH 01/33] v4l2-ioctl.c: move a block of code down, no other changes Hans Verkuil
2012-06-28  6:47   ` [RFCv3 PATCH 02/33] v4l2-ioctl.c: introduce INFO_FL_CLEAR to replace switch Hans Verkuil
2012-06-28  6:47   ` [RFCv3 PATCH 03/33] v4l2-ioctl.c: v4l2-ioctl: add debug and callback/offset functionality Hans Verkuil
2012-06-28  6:47   ` [RFCv3 PATCH 04/33] v4l2-ioctl.c: remove an unnecessary #ifdef Hans Verkuil
2012-06-28  6:47   ` [RFCv3 PATCH 05/33] v4l2-ioctl.c: use the new table for querycap and i/o ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 06/33] v4l2-ioctl.c: use the new table for priority ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 07/33] v4l2-ioctl.c: use the new table for format/framebuffer ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 08/33] v4l2-ioctl.c: use the new table for overlay/streamon/off ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 09/33] v4l2-ioctl.c: use the new table for std/tuner/modulator ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 10/33] v4l2-ioctl.c: use the new table for queuing/parm ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 11/33] v4l2-ioctl.c: use the new table for control ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 12/33] v4l2-ioctl.c: use the new table for selection ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 13/33] v4l2-ioctl.c: use the new table for compression ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 14/33] v4l2-ioctl.c: use the new table for debug ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 15/33] v4l2-ioctl.c: use the new table for preset/timings ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 16/33] v4l2-ioctl.c: use the new table for the remaining ioctls Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 17/33] v4l2-ioctl.c: finalize table conversion Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 18/33] v4l2-dev.c: add debug sysfs entry Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 19/33] v4l2-ioctl: remove v4l_(i2c_)print_ioctl Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 20/33] ivtv: don't mess with vfd->debug Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 21/33] cx18: " Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 22/33] vb2-core: refactor reqbufs/create_bufs Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 23/33] vb2-core: add support for count == 0 in create_bufs Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 24/33] Spec: document CREATE_BUFS behavior if count == 0 Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 25/33] v4l2-dev/ioctl.c: add vb2_queue support to video_device Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 26/33] videobuf2-core: add helper functions Hans Verkuil
2012-06-28  9:27     ` Scott Jiang
2012-06-28 12:17       ` Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 27/33] vivi: remove pointless g/s_std support Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 28/33] vivi: embed struct video_device instead of allocating it Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 29/33] vivi: use vb2 helper functions Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 30/33] vivi: add create_bufs/preparebuf support Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 31/33] v4l2-dev.c: also add debug support for the fops Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 32/33] pwc: use the new vb2 helpers Hans Verkuil
2012-06-28  6:48   ` [RFCv3 PATCH 33/33] pwc: v4l2-compliance fixes Hans Verkuil
2012-07-20  9:55 ` [RFCv3 PATCH 00/33] Core and vb2 enhancements Guennadi Liakhovetski
2012-07-20 10:08   ` Hans Verkuil
2012-07-20 10:14   ` Laurent Pinchart

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).