Re: Bug#699470: crystalhd-dkms: Kernel null pointer BUG in crystalhd_dioq_fetch_wait()

2013-03-01 Thread Andres Mejia
It looks like the crystalhd drivers are buggy with newer kernel
releases. I opt for removing the dkms package. I will do this over the
weekend.

On Thu, Feb 28, 2013 at 4:52 PM, thomas schorpp
thomas.scho...@gmail.com wrote:
 On 28.02.2013 20:41, Julien Cristau wrote:

 On Thu, Jan 31, 2013 at 19:25:50 +0100, tom schorpp wrote:

 Package: crystalhd-dkms
 Version: 1:0.0~git20110715.fdd2f19-7
 Severity: critical
 Tags: patch
 Justification: breaks the whole system

 Reproducible NULL pointer BUG at
 crystalhd-0.0~git20110715.fdd2f19/driver/linux/crystalhd_misc.c:515,
 triggered by adobe flash plugin from dmo repo, ffmpeg, mplayer, bino or
 other, mostly on heavy ioq usage
 or after FETCH_TIMEOUT and/or unclosed driver HANDLEs.

 Does the maintainer or somebody on pkg-multimedia plan on fixing this RC
 bug?  If not I'll consider a NMU removing the dkms package from
 crystalhd source.

 Cheers,
 Julien


 Known linux-media Maintainers

 STAGING - CRYSTAL HD VIDEO DECODER
 M:Naren Sankar nsan...@broadcom.com
 M:Jarod Wilson ja...@wilsonet.com
 M:Scott Davilla davi...@4pi.com
 M:Manu Abraham abraham.m...@gmail.com

 seem to have left the Broadcom's legacy driver code, focusing on the new
 linux-media staging driver, but limited time,
 one stated lately on the linux-media/LKML, nothing read from the others.
 I could adapt it to new kernel releases the next 2-3 years, but nothing
 more, not a experienced kernel driver coder,
 no debian package/policy maintaining motivation because I do not use dkms or
 debian kernels packages.

 If my last patch applies to Your codebase in the debian git repository (mine
 is from git.linuxtv.org, according to the
 git changelog more advanced in the gstreamer plugin, merge?) You may
 consider it

 hotfixed

 and release with known multithreading (gstreamer based transcoders/matroska
 demuxers) and PM ACPI S3 issues?
 Has not crashed here any more, nor notable side effects with usual playback
 use cases, including Totem (gstreamer).

 y
 tom

 ___
 pkg-multimedia-maintainers mailing list
 pkg-multimedia-maintainers@lists.alioth.debian.org
 http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers



--
~ Andres

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#699470: crystalhd-dkms: Kernel null pointer BUG in crystalhd_dioq_fetch_wait()

2013-03-01 Thread thomas schorpp

On 01.03.2013 18:36, Andres Mejia wrote:

It looks like the crystalhd drivers are buggy with newer kernel
releases. I opt for removing the dkms package. I will do this over the
weekend.


Top posting on list and removal announce without valid bug report for Your 
claimed issue and any confirmation?

Driver does not behave different from linux 3.2...3.7.1 here, see my posts on 
the linux-media list, xbmc people?

y
tom




On Thu, Feb 28, 2013 at 4:52 PM, thomas schorpp
thomas.scho...@gmail.com wrote:

On 28.02.2013 20:41, Julien Cristau wrote:


On Thu, Jan 31, 2013 at 19:25:50 +0100, tom schorpp wrote:


Package: crystalhd-dkms
Version: 1:0.0~git20110715.fdd2f19-7
Severity: critical
Tags: patch
Justification: breaks the whole system

Reproducible NULL pointer BUG at
crystalhd-0.0~git20110715.fdd2f19/driver/linux/crystalhd_misc.c:515,
triggered by adobe flash plugin from dmo repo, ffmpeg, mplayer, bino or
other, mostly on heavy ioq usage
or after FETCH_TIMEOUT and/or unclosed driver HANDLEs.


Does the maintainer or somebody on pkg-multimedia plan on fixing this RC
bug?  If not I'll consider a NMU removing the dkms package from
crystalhd source.

Cheers,
Julien



Known linux-media Maintainers

STAGING - CRYSTAL HD VIDEO DECODER
M:Naren Sankar nsan...@broadcom.com
M:Jarod Wilson ja...@wilsonet.com
M:Scott Davilla davi...@4pi.com
M:Manu Abraham abraham.m...@gmail.com

seem to have left the Broadcom's legacy driver code, focusing on the new
linux-media staging driver, but limited time,
one stated lately on the linux-media/LKML, nothing read from the others.
I could adapt it to new kernel releases the next 2-3 years, but nothing
more, not a experienced kernel driver coder,
no debian package/policy maintaining motivation because I do not use dkms or
debian kernels packages.

If my last patch applies to Your codebase in the debian git repository (mine
is from git.linuxtv.org, according to the
git changelog more advanced in the gstreamer plugin, merge?) You may
consider it

hotfixed

and release with known multithreading (gstreamer based transcoders/matroska
demuxers) and PM ACPI S3 issues?
Has not crashed here any more, nor notable side effects with usual playback
use cases, including Totem (gstreamer).

y
tom


___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#699470: crystalhd-dkms: Kernel null pointer BUG in crystalhd_dioq_fetch_wait()

2013-03-01 Thread thomas schorpp

On 01.03.2013 18:36, Andres Mejia wrote:

It looks like the crystalhd drivers are buggy with newer kernel
releases. I opt for removing the dkms package. I will do this over the
weekend.


Driver is maintainable and supported up to at least Linux 3.8 series, when 
we'll have 4.0 in debian stable, 2015?

schorpp@tom3:/mnt/data/usr/local/src/crystalhd/driver/linux$ make clean
rm -f *.map *.list *.o *.ko crystalhd.mod.c crystalhd_lnx.o crystalhd_misc.o 
crystalhd_cmds.o crystalhd_hw.o crystalhd_linkfuncs.o crystalhd_fleafuncs.o 
crystalhd_flea_ddr.o
schorpp@tom3:/mnt/data/usr/local/src/crystalhd/driver/linux$
schorpp@tom3:/mnt/data/usr/local/src/crystalhd/driver/linux$ make
make -C /lib/modules/3.8.1/build 
SUBDIRS=/mnt/data/usr/local/src/crystalhd/driver/linux modules
make[1]: Entering directory `/mnt/data/usr/local/src/linux-laststable'
  CC [M]  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd_lnx.o
  CC [M]  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd_misc.o
  CC [M]  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd_cmds.o
  CC [M]  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd_hw.o
  CC [M]  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd_linkfuncs.o
  CC [M]  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd_fleafuncs.o
  CC [M]  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd_flea_ddr.o
  LD [M]  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd.o
  Building modules, stage 2.
  MODPOST 1 modules
  CC  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd.mod.o
  LD [M]  /mnt/data/usr/local/src/crystalhd/driver/linux/crystalhd.ko
make[1]: Leaving directory `/mnt/data/usr/local/src/linux-laststable'
schorpp@tom3:/mnt/data/usr/local/src/crystalhd/driver/linux$

Builds without warnings with my patches.

schorpp@tom3:~$ uname -a
Linux tom3 3.8.1 #12 SMP PREEMPT Fri Mar 1 20:41:00 CET 2013 x86_64 GNU/Linux
schorpp@tom3:~$ hellobcm
starting up
Running DIL (3.22.0) Version
DtsDeviceOpen: Opening HW in mode 0
Clock set to 180
Setting Color Mode to 1
try calls done
Unable to open input file
DtsAllocIoctlData Error
schorpp@tom3:~$ dmesg |grep crystal
[6.426460] Loading crystalhd v3.10.1
[6.426515] crystalhd :03:00.0: Starting Device:0x1612
[6.429600] crystalhd :03:00.0: irq 51 for MSI/MSI-X
[   99.896730] crystalhd :03:00.0: Opening new user[0] handle
[  102.604770] crystalhd :03:00.0: Closing user[0] handle via ioctl with 
mode 1c200
[  243.648468] crystalhd :03:00.0: Opening new user[0] handle
[  246.325373] crystalhd :03:00.0: Closing user[0] handle via ioctl with 
mode 1c200
[  334.105537] crystalhd :03:00.0: Opening new user[0] handle
[  336.776957] crystalhd :03:00.0: Closing user[0] handle via ioctl with 
mode 1c200
[  510.322855] crystalhd :03:00.0: Opening new user[0] handle
[  510.577299] crystalhd :03:00.0: Closing user[0] handle with mode 
[  510.781740] crystalhd :03:00.0: Opening new user[0] handle
[  511.036308] crystalhd :03:00.0: Closing user[0] handle with mode 
[  511.240825] crystalhd :03:00.0: Opening new user[0] handle
[  513.801093] crystalhd :03:00.0: [FMT CH] PIB:0 ff 420 6 2d0 162 2d0 0 0 0
[  513.921604] crystalhd :03:00.0: MISSING 3 PICTURES
[  514.587208] crystalhd :03:00.0: Closing user[0] handle via ioctl with 
mode 1c200
[  535.989772] crystalhd :03:00.0: Opening new user[0] handle
[  536.244314] crystalhd :03:00.0: Closing user[0] handle with mode 
[  536.448772] crystalhd :03:00.0: Opening new user[0] handle
[  536.703282] crystalhd :03:00.0: Closing user[0] handle with mode 
[  536.907816] crystalhd :03:00.0: Opening new user[0] handle
[  539.435257] crystalhd :03:00.0: [FMT CH] PIB:0 ff 420 6 2d0 162 2d0 0 0 0
[  539.508317] crystalhd :03:00.0: MISSING 3 PICTURES
[  616.455975] crystalhd :03:00.0: Closing user[0] handle via ioctl with 
mode 1c200
schorpp@tom3:~$

Loads and runs with Totem and MPlayer.

root@tom3:~# lspci -vvv -s 03:00.0 |grep Sta
Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast TAbort- TAbort- MAbort- 
SERR+ PERR- INTx-
Status: D0 NoSoftRst+ PME-Enable- DSel=0 DScale=1 PME-
DevSta:CorrErr+ UncorrErr+ FatalErr- UnsuppReq+ AuxPwr- TransPend-
LnkSta:Speed 2.5GT/s, Width x1, TrErr- Train- SlotClk+ DLActive- 
BWMgmt- ABWMgmt-
UESta:DLP- SDES- TLP- FCP- CmpltTO- CmpltAbrt- UnxCmplt- RxOF- 
MalfTLP- ECRC- UnsupReq+ ACSViol-
CESta:RxErr+ BadTLP- BadDLLP- Rollover- Timeout- NonFatalErr+
Status:InProgress-
Status:NegoPending- InProgress-
root@tom3:~#

Some non-fatal pci errors, but lspci is usually reporting incorrectly here or 
my old pci-e host is pretty incompatible.

So no technical reasons to drop the package?

y
tom

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org

Re: Bug#699470: crystalhd-dkms: Kernel null pointer BUG in crystalhd_dioq_fetch_wait()

2013-03-01 Thread Julien Cristau
On Fri, Mar  1, 2013 at 21:38:54 +0100, thomas schorpp wrote:

 So no technical reasons to drop the package?
 
Until and unless the driver is in mainline, there's every reason to drop
it.

Cheers,
Julien


signature.asc
Description: Digital signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: Bug#699470: crystalhd-dkms: Kernel null pointer BUG in crystalhd_dioq_fetch_wait()

2013-03-01 Thread Julien Cristau
On Fri, Mar  1, 2013 at 12:36:03 -0500, Andres Mejia wrote:

 It looks like the crystalhd drivers are buggy with newer kernel
 releases. I opt for removing the dkms package. I will do this over the
 weekend.
 
Thanks.

Julien


signature.asc
Description: Digital signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: Bug#699470: crystalhd-dkms: Kernel null pointer BUG in crystalhd_dioq_fetch_wait()

2013-03-01 Thread thomas schorpp

On 01.03.2013 21:55, Julien Cristau wrote:

On Fri, Mar  1, 2013 at 21:38:54 +0100, thomas schorpp wrote:


So no technical reasons to drop the package?


Until and unless the driver is in mainline, there's every reason to drop
it.
 


Well, then drop all the other non-mainline drivers dkms packages, too.

This is getting unserious, I wish You all much fun with more root holes in
closed source vdpau GPU drivers.

Thanks to Broadcom for the nice effort.

Bye,
tom


___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#699470: crystalhd-dkms: Kernel null pointer BUG in crystalhd_dioq_fetch_wait()

2013-03-01 Thread Andres Mejia
On Fri, Mar 1, 2013 at 3:55 PM, Julien Cristau jcris...@debian.org wrote:
 On Fri, Mar  1, 2013 at 21:38:54 +0100, thomas schorpp wrote:

 So no technical reasons to drop the package?

 Until and unless the driver is in mainline, there's every reason to drop
 it.

 Cheers,
 Julien

I just checked the crystalhd driver in the mainline kernel. The driver
seems to be much better maintained over there than at linuxtv.org. See
[1].

I'm going to drop the driver from linuxtv.org. If after I drop the
package someone has some compelling reason to use the driver from
linuxtv.org, they can submit another bug to the crystalhd package and
explain why the linuxtv.org driver should be used.

1. 
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/log/drivers/staging/crystalhd

--
~ Andres

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#699470: crystalhd-dkms: Kernel null pointer BUG in crystalhd_dioq_fetch_wait()

2013-02-28 Thread Julien Cristau
On Thu, Jan 31, 2013 at 19:25:50 +0100, tom schorpp wrote:

 Package: crystalhd-dkms
 Version: 1:0.0~git20110715.fdd2f19-7
 Severity: critical
 Tags: patch
 Justification: breaks the whole system
 
 Reproducible NULL pointer BUG at 
 crystalhd-0.0~git20110715.fdd2f19/driver/linux/crystalhd_misc.c:515, 
 triggered by adobe flash plugin from dmo repo, ffmpeg, mplayer, bino or 
 other, mostly on heavy ioq usage 
 or after FETCH_TIMEOUT and/or unclosed driver HANDLEs.
 
Does the maintainer or somebody on pkg-multimedia plan on fixing this RC
bug?  If not I'll consider a NMU removing the dkms package from
crystalhd source.

Cheers,
Julien


signature.asc
Description: Digital signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: Bug#699470: crystalhd-dkms: Kernel null pointer BUG in crystalhd_dioq_fetch_wait()

2013-02-28 Thread thomas schorpp

On 28.02.2013 20:41, Julien Cristau wrote:

On Thu, Jan 31, 2013 at 19:25:50 +0100, tom schorpp wrote:


Package: crystalhd-dkms
Version: 1:0.0~git20110715.fdd2f19-7
Severity: critical
Tags: patch
Justification: breaks the whole system

Reproducible NULL pointer BUG at 
crystalhd-0.0~git20110715.fdd2f19/driver/linux/crystalhd_misc.c:515,
triggered by adobe flash plugin from dmo repo, ffmpeg, mplayer, bino or other, 
mostly on heavy ioq usage
or after FETCH_TIMEOUT and/or unclosed driver HANDLEs.


Does the maintainer or somebody on pkg-multimedia plan on fixing this RC
bug?  If not I'll consider a NMU removing the dkms package from
crystalhd source.

Cheers,
Julien



Known linux-media Maintainers

STAGING - CRYSTAL HD VIDEO DECODER
M:Naren Sankar nsan...@broadcom.com
M:Jarod Wilson ja...@wilsonet.com
M:Scott Davilla davi...@4pi.com
M:Manu Abraham abraham.m...@gmail.com

seem to have left the Broadcom's legacy driver code, focusing on the new 
linux-media staging driver, but limited time,
one stated lately on the linux-media/LKML, nothing read from the others.
I could adapt it to new kernel releases the next 2-3 years, but nothing more, 
not a experienced kernel driver coder,
no debian package/policy maintaining motivation because I do not use dkms or 
debian kernels packages.

If my last patch applies to Your codebase in the debian git repository (mine is 
from git.linuxtv.org, according to the
git changelog more advanced in the gstreamer plugin, merge?) You may consider it

hotfixed

and release with known multithreading (gstreamer based transcoders/matroska 
demuxers) and PM ACPI S3 issues?
Has not crashed here any more, nor notable side effects with usual playback use 
cases, including Totem (gstreamer).

y
tom

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers