All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Devin Heitmueller <dheitmueller@kernellabs.com>
Cc: "Rémi Denis-Courmont" <remi@remlab.net>, linux-media@vger.kernel.org
Subject: Re: uvcvideo USERPTR mode busted?
Date: Thu, 21 Mar 2013 13:41:25 +0100	[thread overview]
Message-ID: <151543510.hNXiaIfEAr@avalon> (raw)
In-Reply-To: <CAGoCfixOsn4eTp7NYmJSK-LJTqF677LXf8fgTzrz4KFgPN7znw@mail.gmail.com>

On Sunday 03 March 2013 09:57:14 Devin Heitmueller wrote:
> On Sun, Mar 3, 2013 at 4:37 AM, Rémi Denis-Courmont <remi@remlab.net> wrote:
> >         Hello,
> > 
> > Trying to use USERPTR buffers with UVC, user space gets stuck either in
> > poll(POLLIN) or in ioctl(VIDIOC_DQBUF). It seems the UVC driver never ever
> > returns a frame in USERPTR mode. The symptoms are identical with kernel
> > versions 3.6, 3.7 and 3.8. I also tested 3.2, but it did not support
> > USERPTR.
> > 
> > Tested hardware was Logitech HD Pro Webcam C920 with YUY2 pixel format.
> > The same hardware and the same driver work fine with MMAP buffers.
> > The same USERPTR userspace code works fine with the vivi test device...
> > 
> > Did any have any better luck?

I've just tested USERPTR with a Logitech C905 on a 3.7.10 kernel using yavta 
without any issue.

> Hi Remi,
> 
> I've used userptr mode with the C920 on an ARM platform (with YUYV mode and
> not MPEG).  It's worth noting that there is actually a bug I hit where if
> the memory you pass is not aligned on a page boundary then you will get
> garbage video.  I have a fix or this but haven't submitted it upstream yet.

Please submit it at some point :-)

Is it a uvcvideo issue or a videobuf2 issue ?

> So it should work, aside from the bug I found.
> 
> Have you tried testing with v42l-ctl's streaming command?  That would help
> identify whether it's something special about your code or whether it's the
> driver.  Don't get me wrong, it's almost certainly a driver issue in either
> case, but it would help narrow down the issue if you're using v4l2-ctl as
> that app is really simple and readily available to the driver developers.

-- 
Regards,

Laurent Pinchart


  reply	other threads:[~2013-03-21 12:40 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-03  9:37 uvcvideo USERPTR mode busted? Rémi Denis-Courmont
2013-03-03 14:57 ` Devin Heitmueller
2013-03-21 12:41   ` Laurent Pinchart [this message]
2013-03-21 13:28     ` Devin Heitmueller

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=151543510.hNXiaIfEAr@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=dheitmueller@kernellabs.com \
    --cc=linux-media@vger.kernel.org \
    --cc=remi@remlab.net \
    /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.