Re: bluetooth related firmware loader spew on resume.

2014-11-27 Thread Takashi Iwai
At Wed, 26 Nov 2014 16:21:49 +0100, Takashi Iwai wrote: > > At Wed, 26 Nov 2014 15:27:57 +0100, > Oliver Neukum wrote: > > > > On Wed, 2014-11-26 at 15:12 +0100, Takashi Iwai wrote: > > > At Wed, 26 Nov 2014 23:05:20 +0900, > > > Marcel Holtmann wrote: > > > > > > > > Hi Oliver, > > > > > > > >

Re: bluetooth related firmware loader spew on resume.

2014-11-27 Thread Takashi Iwai
At Thu, 27 Nov 2014 10:46:05 +0100, Arend van Spriel wrote: > > On 11/27/14 10:29, Arend van Spriel wrote: > > On 11/27/14 10:17, Takashi Iwai wrote: > >> At Thu, 27 Nov 2014 09:59:12 +0100, > >> Arend van Spriel wrote: > >>> > >>> On 11/26/14 19:13, Takashi Iwai wrote: > At Wed, 26 Nov 2014

Re: bluetooth related firmware loader spew on resume.

2014-11-27 Thread Arend van Spriel
On 11/27/14 10:29, Arend van Spriel wrote: On 11/27/14 10:17, Takashi Iwai wrote: At Thu, 27 Nov 2014 09:59:12 +0100, Arend van Spriel wrote: On 11/26/14 19:13, Takashi Iwai wrote: At Wed, 26 Nov 2014 18:42:46 +0100, Arend van Spriel wrote: On 11/26/14 16:27, Takashi Iwai wrote: At Wed, 26

Re: bluetooth related firmware loader spew on resume.

2014-11-27 Thread Arend van Spriel
On 11/27/14 10:17, Takashi Iwai wrote: At Thu, 27 Nov 2014 09:59:12 +0100, Arend van Spriel wrote: On 11/26/14 19:13, Takashi Iwai wrote: At Wed, 26 Nov 2014 18:42:46 +0100, Arend van Spriel wrote: On 11/26/14 16:27, Takashi Iwai wrote: At Wed, 26 Nov 2014 17:26:15 +0200, Mihai Donțu wrote:

Re: bluetooth related firmware loader spew on resume.

2014-11-27 Thread Takashi Iwai
At Thu, 27 Nov 2014 09:59:12 +0100, Arend van Spriel wrote: > > On 11/26/14 19:13, Takashi Iwai wrote: > > At Wed, 26 Nov 2014 18:42:46 +0100, > > Arend van Spriel wrote: > >> > >> On 11/26/14 16:27, Takashi Iwai wrote: > >>> At Wed, 26 Nov 2014 17:26:15 +0200, > >>> Mihai Donțu wrote: > > >>

Re: bluetooth related firmware loader spew on resume.

2014-11-27 Thread Arend van Spriel
On 11/26/14 19:13, Takashi Iwai wrote: At Wed, 26 Nov 2014 18:42:46 +0100, Arend van Spriel wrote: On 11/26/14 16:27, Takashi Iwai wrote: At Wed, 26 Nov 2014 17:26:15 +0200, Mihai Donțu wrote: On Wed, 26 Nov 2014 16:19:49 +0100 Takashi Iwai wrote: At Wed, 26 Nov 2014 16:56:09 +0200, Miha

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Takashi Iwai
At Wed, 26 Nov 2014 18:42:46 +0100, Arend van Spriel wrote: > > On 11/26/14 16:27, Takashi Iwai wrote: > > At Wed, 26 Nov 2014 17:26:15 +0200, > > Mihai Donțu wrote: > >> > >> On Wed, 26 Nov 2014 16:19:49 +0100 > >> Takashi Iwai wrote: > >> > >>> At Wed, 26 Nov 2014 16:56:09 +0200, > >>> Mihai Do

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Arend van Spriel
On 11/26/14 16:27, Takashi Iwai wrote: At Wed, 26 Nov 2014 17:26:15 +0200, Mihai Donțu wrote: On Wed, 26 Nov 2014 16:19:49 +0100 Takashi Iwai wrote: At Wed, 26 Nov 2014 16:56:09 +0200, Mihai Donțu wrote: On Tue, 11 Nov 2014 13:12:28 -0500 Dave Jones wrote: Since the addition of 10d4c6736e

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Takashi Iwai
At Wed, 26 Nov 2014 17:26:15 +0200, Mihai Donțu wrote: > > On Wed, 26 Nov 2014 16:19:49 +0100 > Takashi Iwai wrote: > > > At Wed, 26 Nov 2014 16:56:09 +0200, > > Mihai Donțu wrote: > > > > > > On Tue, 11 Nov 2014 13:12:28 -0500 Dave Jones wrote: > > > > Since the addition of 10d4c6736ea "Blueto

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Mihai Donțu
On Wed, 26 Nov 2014 16:19:49 +0100 Takashi Iwai wrote: > At Wed, 26 Nov 2014 16:56:09 +0200, > Mihai Donțu wrote: > > > > On Tue, 11 Nov 2014 13:12:28 -0500 Dave Jones wrote: > > > Since the addition of 10d4c6736ea "Bluetooth: btusb: Add Broadcom patch > > > RAM support", I (and a number of othe

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Takashi Iwai
At Wed, 26 Nov 2014 15:27:57 +0100, Oliver Neukum wrote: > > On Wed, 2014-11-26 at 15:12 +0100, Takashi Iwai wrote: > > At Wed, 26 Nov 2014 23:05:20 +0900, > > Marcel Holtmann wrote: > > > > > > Hi Oliver, > > > > > > >> In order to paper over this, we may also remember the failing firmware > >

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Takashi Iwai
At Wed, 26 Nov 2014 16:56:09 +0200, Mihai Donțu wrote: > > On Tue, 11 Nov 2014 13:12:28 -0500 Dave Jones wrote: > > Since the addition of 10d4c6736ea "Bluetooth: btusb: Add Broadcom patch > > RAM support", I (and a number of other people[*]) have been seeing > > this trace on resume from suspend.

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Mihai Donțu
On Tue, 11 Nov 2014 13:12:28 -0500 Dave Jones wrote: > Since the addition of 10d4c6736ea "Bluetooth: btusb: Add Broadcom patch > RAM support", I (and a number of other people[*]) have been seeing > this trace on resume from suspend. > > WARNING: CPU: 1 PID: 8565 at drivers/base/firmware_class.c:11

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Oliver Neukum
On Wed, 2014-11-26 at 15:31 +0100, Takashi Iwai wrote: > Yes, that's what I mentioned in my reply. But, actually more puzzling > is that the WARNING shouldn't have been triggered at all. That is, > something is still racy there, so we'd need to fix it. Did it trigger before khubd was replaced b

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Takashi Iwai
At Wed, 26 Nov 2014 15:23:20 +0100, Oliver Neukum wrote: > > On Wed, 2014-11-26 at 11:53 +0100, Takashi Iwai wrote: > > At Wed, 26 Nov 2014 11:43:36 +0100, > > Oliver Neukum wrote: > > > > > > On Wed, 2014-11-26 at 11:31 +0100, Takashi Iwai wrote: > > > > At Wed, 26 Nov 2014 11:10:23 +0100, > > >

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Oliver Neukum
On Wed, 2014-11-26 at 15:12 +0100, Takashi Iwai wrote: > At Wed, 26 Nov 2014 23:05:20 +0900, > Marcel Holtmann wrote: > > > > Hi Oliver, > > > > >> In order to paper over this, we may also remember the failing firmware > > >> and avoid loading it. This might be an easer way than the endless > >

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Oliver Neukum
On Wed, 2014-11-26 at 11:53 +0100, Takashi Iwai wrote: > At Wed, 26 Nov 2014 11:43:36 +0100, > Oliver Neukum wrote: > > > > On Wed, 2014-11-26 at 11:31 +0100, Takashi Iwai wrote: > > > At Wed, 26 Nov 2014 11:10:23 +0100, > > > Oliver Neukum wrote: > > > > > > > > On Wed, 2014-11-26 at 09:52 +0100

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Oliver Neukum
On Wed, 2014-11-26 at 23:05 +0900, Marcel Holtmann wrote: > Hi Oliver, > > >> In order to paper over this, we may also remember the failing firmware > >> and avoid loading it. This might be an easer way than the endless > >> fight against UMH race... > > > > > > the full fix would be to impleme

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Takashi Iwai
At Wed, 26 Nov 2014 23:05:20 +0900, Marcel Holtmann wrote: > > Hi Oliver, > > >> In order to paper over this, we may also remember the failing firmware > >> and avoid loading it. This might be an easer way than the endless > >> fight against UMH race... > > > > > > the full fix would be to imp

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Marcel Holtmann
Hi Takashi, On Wed, 2014-11-26 at 09:52 +0100, Takashi Iwai wrote: > At Wed, 26 Nov 2014 14:15:27 +0900, > In order to paper over this, we may also remember the failing firmware > and avoid loading it. This might be an easer way than the endless > fight against UMH race

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Marcel Holtmann
Hi Oliver, >> In order to paper over this, we may also remember the failing firmware >> and avoid loading it. This might be an easer way than the endless >> fight against UMH race... > > > the full fix would be to implement reset_resume() for btusb. > It seems to me that setup() should be split

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Takashi Iwai
At Wed, 26 Nov 2014 11:43:36 +0100, Oliver Neukum wrote: > > On Wed, 2014-11-26 at 11:31 +0100, Takashi Iwai wrote: > > At Wed, 26 Nov 2014 11:10:23 +0100, > > Oliver Neukum wrote: > > > > > > On Wed, 2014-11-26 at 09:52 +0100, Takashi Iwai wrote: > > > > At Wed, 26 Nov 2014 14:15:27 +0900, > > >

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Oliver Neukum
On Wed, 2014-11-26 at 11:31 +0100, Takashi Iwai wrote: > At Wed, 26 Nov 2014 11:10:23 +0100, > Oliver Neukum wrote: > > > > On Wed, 2014-11-26 at 09:52 +0100, Takashi Iwai wrote: > > > At Wed, 26 Nov 2014 14:15:27 +0900, > > > > > In order to paper over this, we may also remember the failing firm

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Takashi Iwai
At Wed, 26 Nov 2014 11:10:23 +0100, Oliver Neukum wrote: > > On Wed, 2014-11-26 at 09:52 +0100, Takashi Iwai wrote: > > At Wed, 26 Nov 2014 14:15:27 +0900, > > > In order to paper over this, we may also remember the failing firmware > > and avoid loading it. This might be an easer way than the e

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Oliver Neukum
On Wed, 2014-11-26 at 09:52 +0100, Takashi Iwai wrote: > At Wed, 26 Nov 2014 14:15:27 +0900, > In order to paper over this, we may also remember the failing firmware > and avoid loading it. This might be an easer way than the endless > fight against UMH race... Hi, the full fix would be to impl

Re: bluetooth related firmware loader spew on resume.

2014-11-26 Thread Takashi Iwai
At Wed, 26 Nov 2014 14:15:27 +0900, Marcel Holtmann wrote: > > Hi Takashi, > > >> Since the addition of 10d4c6736ea "Bluetooth: btusb: Add Broadcom patch > >> RAM support", I (and a number of other people[*]) have been seeing > >> this trace on resume from suspend. > >> > >> WARNING: CPU: 1 PID:

Re: bluetooth related firmware loader spew on resume.

2014-11-25 Thread Marcel Holtmann
Hi Takashi, >> Since the addition of 10d4c6736ea "Bluetooth: btusb: Add Broadcom patch >> RAM support", I (and a number of other people[*]) have been seeing >> this trace on resume from suspend. >> >> WARNING: CPU: 1 PID: 8565 at drivers/base/firmware_class.c:1127 >> _request_firmware+0x4c1/0x7c

Re: bluetooth related firmware loader spew on resume.

2014-11-25 Thread Takashi Iwai
At Tue, 11 Nov 2014 13:12:28 -0500, Dave Jones wrote: > > Since the addition of 10d4c6736ea "Bluetooth: btusb: Add Broadcom patch > RAM support", I (and a number of other people[*]) have been seeing > this trace on resume from suspend. > > WARNING: CPU: 1 PID: 8565 at drivers/base/firmware_class.

bluetooth related firmware loader spew on resume.

2014-11-11 Thread Dave Jones
Since the addition of 10d4c6736ea "Bluetooth: btusb: Add Broadcom patch RAM support", I (and a number of other people[*]) have been seeing this trace on resume from suspend. WARNING: CPU: 1 PID: 8565 at drivers/base/firmware_class.c:1127 _request_firmware+0x4c1/0x7c0() CPU: 1 PID: 8565 Comm: kwor