2007-01-08 21:37:27

by Kevin Fenzi

[permalink] [raw]
Subject: Re: [Bluez-devel] 2 plugz issues

>Kevin,
>>
>>> 2. Once I comment out that code from 1 above, my headset works fine
>>> and sounds ok. However, if I do anything that sends a DTMF from my
>>> sip phone (I am using twinkle) it causes twinkle to go into a 100%
>>> cpu spin, and once to totally lock up my machine. Any idea why DTMF
>>> would cause that? Is there any way I can test or get more info to
>>> track down the problem?
>>
>> please try it with plugz/patches/sco-flowcontrol-v3.0.diff applied.
>>

>I would second Brad remark. With an unpatched kernel you will
>experience kernel crashes. The test you removed from the code was
>supposed to check for the above kernel patch !! ;-)

Yeah, ok. You may want to change that message from the generic sounding

Unable to set number of SCO buffers : please upgrade your Kernel!

to something like:

Unable to set number of SCO buffers: Please apply kernel patches from
patches directory in the plugz source to your kernel and reboot with
that kernel.

I'll try and find time to build a custom kernel and test it, but that
might take a while. I try and avoid building custom kernels these
days. ;(

>
>Btw, i would like to update the project documentation, to explicitely
>say the kernel patch is mandatory.

If this patch is needed, is there any move to get it pushed to
upstream? One of the reasons I was very happy about this new setup was
that I didn't need to compile an out of tree module with every kernel
update, but now it's worse if I have to make a new kernel with this
patch. ;(

>
>Cheers,
>
>Fabien

kevin


Attachments:
signature.asc (189.00 B)
(No filename) (347.00 B)
(No filename) (164.00 B)
Download all attachments

2007-01-08 22:32:51

by Brad Midgley

[permalink] [raw]
Subject: Re: [Bluez-devel] 2 plugz issues

Kevin

> If this patch is needed, is there any move to get it pushed to
> upstream? One of the reasons I was very happy about this new setup was
> that I didn't need to compile an out of tree module with every kernel
> update, but now it's worse if I have to make a new kernel with this
> patch. ;(

yes that is the idea. Fabien has been adapting it according to Marcel's
feedback but ultimately Marcel needs to give it the nod.

Brad

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel