[PATCH] dvb-core: fix initialization of feeds list in demux filter

2010-02-07 Thread Francesco Lavra
A DVB demultiplexer device can be used to set up either a PES filter or a section filter. In the former case, the ts field of the feed union of struct dmxdev_filter is used, in the latter case the sec field of the same union is used. The ts field is a struct list_head, and is currently initialized

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter

2010-02-07 Thread Andy Walls
On Sun, 2010-02-07 at 13:49 +0100, Francesco Lavra wrote: > A DVB demultiplexer device can be used to set up either a PES filter or > a section filter. In the former case, the ts field of the feed union of > struct dmxdev_filter is used, in the latter case the sec field of the > same union is used.

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter

2010-02-07 Thread hermann pitton
Hi! Am Sonntag, den 07.02.2010, 10:13 -0500 schrieb Andy Walls: > On Sun, 2010-02-07 at 13:49 +0100, Francesco Lavra wrote: > > A DVB demultiplexer device can be used to set up either a PES filter or > > a section filter. In the former case, the ts field of the feed union of > > struct dmxdev_filt

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter

2010-02-07 Thread Mauro Carvalho Chehab
Francesco Lavra wrote: > A DVB demultiplexer device can be used to set up either a PES filter or > a section filter. In the former case, the ts field of the feed union of > struct dmxdev_filter is used, in the latter case the sec field of the > same union is used. > The ts field is a struct list_he

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter

2010-02-08 Thread Andreas Oberritter
Hello Mauro, Mauro Carvalho Chehab wrote: > Good catch, but it seems better to initialize both the mutex and the list head > at dvb_dmx_dev_init. Please test if the following patch fixes the issue. If > so, please > sign. please apply Francesco's original patch. Yours won't work, because "feed"

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread Chicken Shack
Am Montag, den 08.02.2010, 13:24 +0100 schrieb Andreas Oberritter: > Hello Mauro, > > Mauro Carvalho Chehab wrote: > > Good catch, but it seems better to initialize both the mutex and the list > > head > > at dvb_dmx_dev_init. Please test if the following patch fixes the issue. If > > so, please

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread Devin Heitmueller
Hello Mr. Shack, On Mon, Feb 8, 2010 at 8:43 AM, Chicken Shack wrote: > I vote for Andy Walls, Devin Heitmueller. There are a couple of capable > candidates here who can really do that job with a better output for the > whole Linux community. > There is ABSOLUTELY NO necessity to continue with Ma

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread Linus Torvalds
On Mon, 8 Feb 2010, Chicken Shack wrote: > > This is a SCANDAL, not fun! This is SCANDALOUS! I agree that this whole thread has been totally inappropriate from beginning to end. On all sides, btw. I would suggest that somebody who can't even use his own name should be the last to complain a

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread Mauro Carvalho Chehab
Linus Torvalds wrote: > - if somebody reports a regression, IT MUST BE FIXED. Not "discussed" for >two weeks. Not talked around. If we have a bisected commit that starts >oopsing with an existing setup that used to work, there is no >discussion. That commit absolutely _has_ to be r

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread Alain Kalker
Op maandag 08-02-2010 om 14:43 uur [tijdzone +0100], schreef Chicken Shack: > Now if I were a cynical or ranter or another kind of dumb primitive > persona non grata I would just add "Lol" or stuff like that and turn > myself away. > > But this is no fun here. > > It's nothing but a big proof tha

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread Chicken Shack
Am Montag, den 08.02.2010, 08:14 -0800 schrieb Linus Torvalds: > > On Mon, 8 Feb 2010, Chicken Shack wrote: > > > > This is a SCANDAL, not fun! This is SCANDALOUS! > > I agree that this whole thread has been totally inappropriate from > beginning to end. Yes, as a matter of frustration and th

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread Linus Torvalds
On Mon, 8 Feb 2010, Chicken Shack wrote: > ... Please stop emailing me. I'm simply not interested in your political troubles with the DVB people. I see with my own eyes why people have issues with your way of communication, and quite frankly, I'm not interested in discussing name-calling wit

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread Chicken Shack
Am Montag, den 08.02.2010, 11:09 -0800 schrieb Linus Torvalds: > > On Mon, 8 Feb 2010, Chicken Shack wrote: > > ... > > Please stop emailing me. I'm simply not interested in your political > troubles with the DVB people. > > I see with my own eyes why people have issues with your way of > comm

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread Rudy Zijlstra
Chicken Shack wrote: ok anonymous chicken you've managed a distinct first: i've installed a block filter on your email address. Rudy -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vg

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-08 Thread hermann pitton
Am Montag, den 08.02.2010, 08:14 -0800 schrieb Linus Torvalds: > > On Mon, 8 Feb 2010, Chicken Shack wrote: > > > > This is a SCANDAL, not fun! This is SCANDALOUS! > > I agree that this whole thread has been totally inappropriate from > beginning to end. The initial problem was, how to find s

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-09 Thread Chicken Shack
Am Dienstag, den 09.02.2010, 01:53 +0100 schrieb hermann pitton: > Am Montag, den 08.02.2010, 08:14 -0800 schrieb Linus Torvalds: > > > > On Mon, 8 Feb 2010, Chicken Shack wrote: > > > > > > This is a SCANDAL, not fun! This is SCANDALOUS! > > > > I agree that this whole thread has been totally i

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-09 Thread Chicken Shack
Am Dienstag, den 09.02.2010, 17:09 -0500 schrieb Richard Stallman: > "Banning you as a punishment seems foolishly harsh, too." > > The word "too" is there because, in my previous paragraph, I said it > seemed you had been overly harsh in your criticism. Everyone makes > mistakes. Being busy is n

Re: [PATCH] dvb-core: fix initialization of feeds list in demux filter (Was: Videotext application crashes the kernel due to DVB-demux patch)

2010-02-10 Thread hermann pitton
Am Dienstag, den 09.02.2010, 09:38 +0100 schrieb Chicken Shack: > Am Dienstag, den 09.02.2010, 01:53 +0100 schrieb hermann pitton: > > Am Montag, den 08.02.2010, 08:14 -0800 schrieb Linus Torvalds: > > > > > > On Mon, 8 Feb 2010, Chicken Shack wrote: > > > > > > > > This is a SCANDAL, not fun! Th