public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: hermann pitton <hermann-pitton@arcor.de>
To: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: Andreas Oberritter <obi@linuxtv.org>,
	Andy Walls <awalls@radix.net>,
	Chicken Shack <chicken.shack@gmx.de>,
	linux-media@vger.kernel.org, akpm@linux-foundation.org,
	torvalds@linux-foundation.org
Subject: Re: Need to discuss method for multiple, multiple-PID TS's from same demux (Re: Videotext application crashes the kernel due to DVB-demux patch)
Date: Fri, 05 Feb 2010 22:46:20 +0100	[thread overview]
Message-ID: <1265406380.4064.9.camel@localhost> (raw)
In-Reply-To: <4B6C88AD.4010708@redhat.com>

Hi,

Am Freitag, den 05.02.2010, 19:07 -0200 schrieb Mauro Carvalho Chehab:
> Andreas Oberritter wrote:
> > Andy Walls wrote:
> 
> >>> As Honza noted, these ioctls are used by enigma2 and, in general, by
> >>> software running on Dream Multimedia set top boxes.
> >> Right, so reverting the patch is not an option.
> >>
> >> It also makes implementing multiple dvr0.n nodes for a demux0 device
> >> node probably a waste of time at this point.
> > 
> > I think so, too. But I guess it's always worth discussing alternatives.
> 
> If this discussion happened before 2.6.32 release, and provided that a different
> implementation were agreed, things would be easier, as a different solution like
> your proposal could be decided and used.
> 
> Now, we have already a regression on a stable kernel, and solving it by
> creating another regression is not something smart to do.
> 
> >From what I understood, the regression appeared on an old, orphan
> application with a non-official patch applied on it. Other applications with
> similar features weren't affected. On the other hand, if the patch got reverted, 
> we'll break a maintained application that is used on a great number of devices,
> and whose features depend on the new ioctls.
> 
> We are too late in -rc cycle, so probably there's not enough time for
> writing, test, validate any new API in time for 2.6.33 and write some compat
> layer to emulate those two ioctls with a different implementation.
> 
> So, removing those two ioctls is not an option anymore.
> 
> 
> Cheers,
> Mauro

during the still ongoing v4l to v4l2 conversion, all major apps did ship
with their own headers.

Since we keep backward compat, that previously unknown to me
alevt-dvb-t, agreed it is a nice to have, should compile against the
older headers instead latest kernel headers, until someone maintains it
again and takes advantage of later improvements.

Untested, but usually we see just such.

Cheers,
Hermann





  reply	other threads:[~2010-02-05 21:46 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-01  9:56 Videotext application crashes the kernel due to DVB-demux patch Chicken Shack
2010-02-01 12:41 ` Andy Walls
2010-02-02  2:00   ` Andy Walls
2010-02-02  9:11     ` Chicken Shack
2010-02-02 12:52       ` Andy Walls
2010-02-03  1:01         ` hermann pitton
2010-02-04 12:54           ` Andy Walls
2010-02-04 14:07             ` Chicken Shack
2010-02-05  2:21               ` Need to discuss method for multiple, multiple-PID TS's from same demux (Re: Videotext application crashes the kernel due to DVB-demux patch) Andy Walls
2010-02-05  2:37                 ` hermann pitton
2010-02-05 11:39                 ` Chicken Shack
2010-02-05 12:19                   ` HoP
2010-02-05 18:29                     ` Andy Walls
2010-02-05 13:19                 ` Andreas Oberritter
2010-02-05 13:28                   ` Mauro Carvalho Chehab
2010-02-05 13:58                     ` Chicken Shack
2010-02-05 15:31                     ` Chicken Shack
2010-02-05 19:22                   ` Andy Walls
2010-02-05 20:27                     ` Andreas Oberritter
2010-02-05 21:07                       ` Mauro Carvalho Chehab
2010-02-05 21:46                         ` hermann pitton [this message]
2010-02-05 22:32                         ` Chicken Shack
2010-02-05 23:12                           ` hermann pitton
2010-02-05 23:39                             ` Chicken Shack
2010-02-06  0:25                               ` hermann pitton
2010-02-06  8:55                                 ` "However, if you don't want to lose your freedom, you had better not follow him." " Chicken Shack
2010-02-07  3:58                                   ` hermann pitton
2010-02-07 12:11                                     ` Chicken Shack
2010-02-07 14:43                                     ` Andy Walls
2010-02-07 15:29                                       ` Chicken Shack
2010-02-07 18:10                                         ` HoP
2010-02-07 18:46                                           ` Chicken Shack
2010-02-07 19:13                                             ` HoP
2010-02-07 19:41                                               ` Chicken Shack
2010-02-06 12:02                                 ` Need to discuss method for multiple, multiple-PID TS's from same demux " BOUWSMA Barry
2010-03-11  4:00                                   ` hermann pitton
2010-02-01 13:02 ` Videotext application crashes the kernel due to DVB-demux patch Mauro Carvalho Chehab
2010-02-01 13:59   ` Chicken Shack

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=1265406380.4064.9.camel@localhost \
    --to=hermann-pitton@arcor.de \
    --cc=akpm@linux-foundation.org \
    --cc=awalls@radix.net \
    --cc=chicken.shack@gmx.de \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.com \
    --cc=obi@linuxtv.org \
    --cc=torvalds@linux-foundation.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox