All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: Mitar <mmitar@gmail.com>
Cc: Janne Grunau <j@jannau.net>, linux-media@vger.kernel.org
Subject: Re: [PATCH] Too slow libv4l MJPEG decoding with HD cameras
Date: Wed, 27 Oct 2010 19:43:01 +0200	[thread overview]
Message-ID: <4CC864A5.1070101@redhat.com> (raw)
In-Reply-To: <AANLkTim6ji=_bgdPWpv7608J8t6P5EfWZrhq8BRVWjnR@mail.gmail.com>

Hi,

On 10/27/2010 03:59 PM, Mitar wrote:
> Hi!
>
> On Wed, Oct 27, 2010 at 3:23 PM, Hans de Goede<hdegoede@redhat.com>  wrote:
>> If and only if libjpeg-turbo turns out to be much slower this is something
>> to consider. But the first thing to do here is see if we can solve this
>> in a way which is acceptable to all downstream users of libv4l, and thus
>> can be added in a non optional way (so make libv4l require libjpeg).
>
> I opted for avcodec as I have been told it is the fastest
> implementation around. But I have not really tested that claim. So for
> sure this should be tested. I tested just original tinyjpeg vs.
> avcodec implementation.
>
> I am sorry but I do not have time to do more work on this. Especially
> because libjpeg-turbo also seems to have problems with restart
> markers:
>
> http://libjpeg-turbo.virtualgl.org/About/Performance (at the end)
>
> which is the problem I am currently try to deal with also with ffmpeg:
>
> https://roundup.ffmpeg.org/issue2325
>

Well the problem is not entirely the same. libjpeg-turbo will degrade
a bit in performance when encountering these markers, where as if I
understand your ffmpeg bugreport properly ffmpeg does not properly
decode these images. This seems actually like an argument in favor of
giving libjpeg a try.

Are you sure you cannot make some time for this?

Regards,

Hans

  reply	other threads:[~2010-10-27 17:39 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-26 23:51 [PATCH] Too slow libv4l MJPEG decoding with HD cameras Mitar
2010-10-27  9:08 ` Hans de Goede
2010-10-27 10:49   ` Janne Grunau
2010-10-27 13:23     ` Hans de Goede
2010-10-27 13:59       ` Mitar
2010-10-27 17:43         ` Hans de Goede [this message]
2010-10-28 20:35           ` Mitar
2010-10-27 14:01     ` Mitar

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=4CC864A5.1070101@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=j@jannau.net \
    --cc=linux-media@vger.kernel.org \
    --cc=mmitar@gmail.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.