All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Frank Schäfer" <fschaefer.oss@googlemail.com>
To: Antti Palosaari <crope@iki.fi>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: em28xx DEADLOCK reported by lock debug
Date: Sun, 22 Dec 2013 19:51:44 +0100	[thread overview]
Message-ID: <52B734C0.6010409@googlemail.com> (raw)
In-Reply-To: <52B72BEB.4010902@iki.fi>

Am 22.12.2013 19:14, schrieb Antti Palosaari:
> On 22.12.2013 20:02, Frank Schäfer wrote:
>> Am 22.12.2013 15:34, schrieb Antti Palosaari:
>>> On 22.12.2013 15:51, Frank Schäfer wrote:
>>>> Am 21.12.2013 20:55, schrieb Antti Palosaari:
>>>>> On 21.12.2013 18:51, Frank Schäfer wrote:
>>>>>> Hi Antti,
>>>>>>
>>>>>> thank you for reporting this issue.
>>>>>>
>>>>>> Am 18.12.2013 17:04, schrieb Antti Palosaari:
>>>>>>> That same lock debug deadlock is still there (maybe ~4 times I
>>>>>>> report
>>>>>>> it during 2 years). Is that possible to fix easily at all?
>>>>>>
>>>>>> Patches are always welcome. ;)
>>>>>
>>>>> haha, I cannot simply learn every driver I meet some problems...
>>>> Hint:
>>>>
>>>> If you report a bug ~4 times in 2 years but never get a reply, it
>>>> usually means
>>>> a) nobody cares
>>>> b) nobody has the resources (time, knowledge) to fix it.
>>>>
>>>> So you either have to live with this issue or to fix it yourself.
>>>
>>> OK, as you request me to fix it, I will fix that by making DVB USB v2
>>> driver for these em28xx devices I have added.
>>>
>>> It should not be very much work as em28xx protocol is still relatively
>>> easy.
>> How would that help to get those lockdep false warnings fixed ?
>> Btw: these warnings should appear for _all_ em28xx extensions (dvb,
>> input, audio).
>
> I am already looking to silence that v4l2 lockdep report. It is hard
> to say how much it is work as I simply don't know even reasons.
>
> I suspect that if I start learning and fixing em28xx driver it will
> take week or two as a workload. Writing new dvb-usb driver is only max
> 2 days of work and as a bonus you will get some missing features for
> free:
> 1) power-management
> 2) suspend/resume
> 3) PID filters
Sure, but we already have a driver for these devices.
I agree with you that em28xx is a big mess, but at least in this case it
doesn't do anything wrong.
Does this false warning really make you so nervous that you're willing
to spent 2 days for it ?
I appreciate that, but I suggest to spend these 2 days for fixing the
issue instead of just avoiding it.

Regards,
Frank

>
> regards
> Antti
>


  reply	other threads:[~2013-12-22 18:50 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-18 16:04 em28xx DEADLOCK reported by lock debug Antti Palosaari
2013-12-21 16:51 ` Frank Schäfer
2013-12-21 19:55   ` Antti Palosaari
2013-12-22 13:51     ` Frank Schäfer
2013-12-22 14:34       ` Antti Palosaari
2013-12-22 18:02         ` Frank Schäfer
2013-12-22 18:14           ` Antti Palosaari
2013-12-22 18:51             ` Frank Schäfer [this message]
2013-12-22 14:53       ` Mauro Carvalho Chehab
2013-12-22 18:12         ` Frank Schäfer

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=52B734C0.6010409@googlemail.com \
    --to=fschaefer.oss@googlemail.com \
    --cc=crope@iki.fi \
    --cc=linux-media@vger.kernel.org \
    /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.