On 11/11/2016 02:55 PM, Mauro Carvalho Chehab wrote:
Em Thu, 10 Nov 2016 12:15:39 +0100
Benjamin,
Could you please test it?
Thanks!
Mauro
Hi, I the device is in use now. Can't test it with a new tree. I am
using an old tree for it.
MvH
Benjamin Larsson
--
To unsubscribe from this list: se
/* rest of packet length low */
> @@ -890,7 +889,7 @@ static int af9005_rc_query(struct dvb_usb_device *d, u32
> * event, int *state)
> }
>
> ret:
> - mutex_unlock(&st->data_mutex);
> + mutex_unlock(&d->data_mutex);
> return ret;
>
- THIS_MODULE, &d, adapter_nr);
-
- if (ret < 0)
- return ret;
-
- st = d->priv;
- mutex_init(&st->data_mutex);
-
- return 0;
+ return dvb_usb_device_init(intf, &af9005_properties,
+
gt; 24 10 eb 1a 49 c7 44 24 08 02 00 00 00 c6 43 RIP []
> __mutex_lock_slowpath+0x6f/0x100 RSP
> CR2:
>
> So, move it to the struct dvb_usb_device and initialize it
> before calling the driver's callbacks.
>
> Reported-by: Jörg Otte
> Signed-of
CR2:
So, move it to the struct dvb_usb_device and initialize it
before calling the driver's callbacks.
Reported-by: Jörg Otte
Signed-off-by: Mauro Carvalho Chehab
diff --git a/drivers/media/usb/dvb-usb/cinergyT2-core.c
b/drivers/media/usb/dvb-usb/cinergyT2-core.c
index
Em Wed, 9 Nov 2016 19:57:58 +
Malcolm Priestley escreveu:
> > Yeah, I avoided serializing the logic that detects if the firmware is
> > loaded, but forgot that the power control had the same issue. The
> > newer dvb usb drivers use the dvb-usb-v2, so I didn't touch this
> > code for a while.
Em Wed, 9 Nov 2016 11:07:35 -0800
Linus Torvalds escreveu:
> On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote:
> >
> > Tried patch with no success. Again a NULL ptr dereferece.
>
> That patch was pure garbage, I think. Pretty much all the other
> drivers that use the same approach will have th
get following crash in dvb-usb-cinergyT2 module.
Looks like it's commit 5ef8ed0e5608f ("[media] cinergyT2-core: don't
do DMA on stack"), which movced the DMA data array from the stack to
the "private" pointer. In the process it also added serialization in
the form o
On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote:
>
> Tried patch with no success. Again a NULL ptr dereferece.
That patch was pure garbage, I think. Pretty much all the other
drivers that use the same approach will have the same issue. Adding
that init function just for the semaphore is crazy.
I
2016-11-08 21:22 GMT+01:00 Mauro Carvalho Chehab :
> Em Tue, 8 Nov 2016 10:42:03 -0800
> Linus Torvalds escreveu:
>
>> On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote:
>> > Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module.
>>
>> Looks lik
Em Tue, 8 Nov 2016 22:15:24 +0100
Benjamin Larsson escreveu:
> On 11/08/2016 09:22 PM, Mauro Carvalho Chehab wrote:
> > Em Tue, 8 Nov 2016 10:42:03 -0800
> > Linus Torvalds escreveu:
> >
> >> On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote:
> >>> Since
On 11/08/2016 09:22 PM, Mauro Carvalho Chehab wrote:
Em Tue, 8 Nov 2016 10:42:03 -0800
Linus Torvalds escreveu:
On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote:
Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module.
Looks like it's commit 5ef8ed0e5608f ("[media] cine
Em Tue, 8 Nov 2016 10:42:03 -0800
Linus Torvalds escreveu:
> On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote:
> > Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module.
>
> Looks like it's commit 5ef8ed0e5608f ("[media] cinergyT2-core: don't
> do
On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote:
> Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module.
Looks like it's commit 5ef8ed0e5608f ("[media] cinergyT2-core: don't
do DMA on stack"), which movced the DMA data array from the stack to
the "private
Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module.
dvb-usb: found a 'TerraTec/qanu USB2.0 Highspeed DVB-T Receiver' in warm state.
BUG: unable to handle kernel NULL pointer dereference at (null)
IP: [] __mutex_lock_slowpath+0x6f/0x100
PGD 0
Oops: 0002 [#1] S
Em Tue, 11 Oct 2016 23:41:53 +0200 (CEST)
Julia Lawall escreveu:
> On Tue, 11 Oct 2016, Mauro Carvalho Chehab wrote:
>
> > Em Tue, 11 Oct 2016 15:16:24 +0200 (CEST)
> > Julia Lawall escreveu:
> >
> > > On Tue, 11 Oct 2016, Julia Lawall wrote:
> > >
> > > > It looks like a lock may be needed
.@01.org
> > Cc: Julia Lawall
> > Subject:
> >
> > [linux-review:Mauro-Carvalho-Chehab/Don-t-use-stack-for-DMA-transers-on-medi
> > a-usb-drivers/20161011-182408 3/31]
> > drivers/media/usb/dvb-usb/cinergyT2-core.c:174:2-8: preceding lock on
&g
> julia
> > >
> > > -- Forwarded message --
> > > Date: Tue, 11 Oct 2016 21:06:18 +0800
> > > From: kbuild test robot
> > > To: kbu...@01.org
> > > Cc: Julia Lawall
> > > Subject:
> > >
> > >
Cc: Julia Lawall
> Subject:
>
> [linux-review:Mauro-Carvalho-Chehab/Don-t-use-stack-for-DMA-transers-on-medi
> a-usb-drivers/20161011-182408 3/31]
> drivers/media/usb/dvb-usb/cinergyT2-core.c:174:2-8: preceding lock on line
> 169
>
> CC: kbuild-...@01.org
> TO: Ma
-drivers/20161011-182408 3/31]
drivers/media/usb/dvb-usb/cinergyT2-core.c:174:2-8: preceding lock on line
169
CC: kbuild-...@01.org
TO: Mauro Carvalho Chehab
CC: linux-media@vger.kernel.org
CC: 0day robot
tree: https://github.com/0day-ci/linux
Mauro-Carvalho-Chehab/Don-t-use-stack-for
Thierry Merle wrote:
> another user has got the same problem, except that the led still lights up.
> Can you tell us what it the firmware version in your device?
> You can see it by doing lsusb -vvv, for the Cinergy T2 entry this is the
> "bcdDevice" line.
> I have the 1.06 firmware version and it
Hello Johannes,
Johannes Engel wrote:
> Hello!
>
> Switching to the new kernel 2.6.28 including the new driver for my Terratec
> Cinergy T² made the thing almost unusable.
> Neither mplayer nor scan resp. w_scan is able to tune the card anymore, not
> even the led lights up anymore. Sometimes t
Hello!
Switching to the new kernel 2.6.28 including the new driver for my Terratec
Cinergy T² made the thing almost unusable.
Neither mplayer nor scan resp. w_scan is able to tune the card anymore, not
even the led lights up anymore. Sometimes tzap manages to get out a proper
signal, but not re
23 matches
Mail list logo