On 09/15/2017 09:03 PM, Bjorn Helgaas wrote:
> On Fri, Sep 15, 2017 at 01:55:57PM -0600, Jens Axboe wrote:
>> On 09/15/2017 01:51 PM, Luca Coelho wrote:
>>> On Fri, 2017-09-15 at 13:48 -0600, Jens Axboe wrote:
On 09/15/2017 01:38 PM, Linus Torvalds wrote:
> On Fri, Sep 15, 2017 at 12:32
On Fri, Sep 15, 2017 at 01:55:57PM -0600, Jens Axboe wrote:
> On 09/15/2017 01:51 PM, Luca Coelho wrote:
> > On Fri, 2017-09-15 at 13:48 -0600, Jens Axboe wrote:
> >> On 09/15/2017 01:38 PM, Linus Torvalds wrote:
> >>> On Fri, Sep 15, 2017 at 12:32 PM, Jens Axboe wrote:
> >
>
On 09/15/2017 01:51 PM, Linus Torvalds wrote:
> On Fri, Sep 15, 2017 at 12:43 PM, Luca Coelho wrote:
>> On Fri, 2017-09-15 at 12:38 -0700, Linus Torvalds wrote:
>>>
>>> From some of the context it looks like commit 40f11adc7cd9 ("PCI:
>>> Avoid race while enabling upstream
On 09/15/2017 01:51 PM, Luca Coelho wrote:
> On Fri, 2017-09-15 at 13:48 -0600, Jens Axboe wrote:
>> On 09/15/2017 01:38 PM, Linus Torvalds wrote:
>>> On Fri, Sep 15, 2017 at 12:32 PM, Jens Axboe wrote:
>
> In any case, your patch introduces a regression on systems.
On Fri, Sep 15, 2017 at 12:43 PM, Luca Coelho wrote:
> On Fri, 2017-09-15 at 12:38 -0700, Linus Torvalds wrote:
>>
>> From some of the context it looks like commit 40f11adc7cd9 ("PCI:
>> Avoid race while enabling upstream bridges"), is that correct?
>
> Yes, that's the one. And
On Fri, 2017-09-15 at 13:48 -0600, Jens Axboe wrote:
> On 09/15/2017 01:38 PM, Linus Torvalds wrote:
> > On Fri, Sep 15, 2017 at 12:32 PM, Jens Axboe wrote:
> > > >
> > > > In any case, your patch introduces a regression on systems. Please get
> > > > it reverted now, and then
On 09/15/2017 01:38 PM, Linus Torvalds wrote:
> On Fri, Sep 15, 2017 at 12:32 PM, Jens Axboe wrote:
>>>
>>> In any case, your patch introduces a regression on systems. Please get
>>> it reverted now, and then you can come up with a new approach to fix the
>>> double enable of the
On 09/15/2017 01:36 PM, Luca Coelho wrote:
> On Fri, 2017-09-15 at 13:32 -0600, Jens Axboe wrote:
>> On 09/14/2017 02:36 PM, Jens Axboe wrote:
>>> On 09/14/2017 02:04 PM, Srinath Mannam wrote:
Hi Jens Axboe,
On Thu, Sep 14, 2017 at 11:14 PM, Jens Axboe wrote:
On Fri, 2017-09-15 at 12:38 -0700, Linus Torvalds wrote:
> On Fri, Sep 15, 2017 at 12:32 PM, Jens Axboe wrote:
> > >
> > > In any case, your patch introduces a regression on systems. Please get
> > > it reverted now, and then you can come up with a new approach to fix the
> > >
On Fri, Sep 15, 2017 at 12:32 PM, Jens Axboe wrote:
>>
>> In any case, your patch introduces a regression on systems. Please get
>> it reverted now, and then you can come up with a new approach to fix the
>> double enable of the upstream bridge.
>
> Who's sending in the revert? I
On Fri, 2017-09-15 at 13:32 -0600, Jens Axboe wrote:
> On 09/14/2017 02:36 PM, Jens Axboe wrote:
> > On 09/14/2017 02:04 PM, Srinath Mannam wrote:
> > > Hi Jens Axboe,
> > >
> > >
> > > On Thu, Sep 14, 2017 at 11:14 PM, Jens Axboe wrote:
> > > > On 09/14/2017 11:35 AM, Jens
On 09/14/2017 02:36 PM, Jens Axboe wrote:
> On 09/14/2017 02:04 PM, Srinath Mannam wrote:
>> Hi Jens Axboe,
>>
>>
>> On Thu, Sep 14, 2017 at 11:14 PM, Jens Axboe wrote:
>>> On 09/14/2017 11:35 AM, Jens Axboe wrote:
On 09/14/2017 11:28 AM, Srinath Mannam wrote:
> Hi
On 09/14/2017 02:04 PM, Srinath Mannam wrote:
> Hi Jens Axboe,
>
>
> On Thu, Sep 14, 2017 at 11:14 PM, Jens Axboe wrote:
>> On 09/14/2017 11:35 AM, Jens Axboe wrote:
>>> On 09/14/2017 11:28 AM, Srinath Mannam wrote:
Hi Bjorn,
On Thu, Sep 14, 2017 at 10:52 PM,
Hi Jens Axboe,
On Thu, Sep 14, 2017 at 11:14 PM, Jens Axboe wrote:
> On 09/14/2017 11:35 AM, Jens Axboe wrote:
>> On 09/14/2017 11:28 AM, Srinath Mannam wrote:
>>> Hi Bjorn,
>>>
>>> On Thu, Sep 14, 2017 at 10:52 PM, Jens Axboe wrote:
On 09/14/2017
On Thu, 2017-09-14 at 23:14 +0530, Srinath Mannam wrote:
>
> atomic_inc_return(>enable_cnt) in function
> pci_enable_device_flags enables passed pcie device.
> !pci_is_enabled(bridge) check in "if (bridge &&
> !pci_is_enabled(bridge))" checks for bridge device of previous pcie
> device.
> So it
On 09/14/2017 11:44 AM, Srinath Mannam wrote:
> Hi Jens Axboe,
>
> On Thu, Sep 14, 2017 at 11:05 PM, Jens Axboe wrote:
>> On 09/14/2017 11:28 AM, Srinath Mannam wrote:
>>> Hi Bjorn,
>>>
>>> On Thu, Sep 14, 2017 at 10:52 PM, Jens Axboe wrote:
On
On 09/14/2017 11:35 AM, Jens Axboe wrote:
> On 09/14/2017 11:28 AM, Srinath Mannam wrote:
>> Hi Bjorn,
>>
>> On Thu, Sep 14, 2017 at 10:52 PM, Jens Axboe wrote:
>>>
>>> On 09/14/2017 11:11 AM, Bjorn Helgaas wrote:
[+cc linux-pci]
On Thu, Sep 14, 2017 at 12:00 PM,
Hi Jens Axboe,
On Thu, Sep 14, 2017 at 11:05 PM, Jens Axboe wrote:
> On 09/14/2017 11:28 AM, Srinath Mannam wrote:
>> Hi Bjorn,
>>
>> On Thu, Sep 14, 2017 at 10:52 PM, Jens Axboe wrote:
>>>
>>> On 09/14/2017 11:11 AM, Bjorn Helgaas wrote:
[+cc linux-pci]
On 09/14/2017 11:28 AM, Srinath Mannam wrote:
> Hi Bjorn,
>
> On Thu, Sep 14, 2017 at 10:52 PM, Jens Axboe wrote:
>>
>> On 09/14/2017 11:11 AM, Bjorn Helgaas wrote:
>>> [+cc linux-pci]
>>>
>>> On Thu, Sep 14, 2017 at 12:00 PM, Jens Axboe wrote:
On
Hi Bjorn,
On Thu, Sep 14, 2017 at 10:52 PM, Jens Axboe wrote:
>
> On 09/14/2017 11:11 AM, Bjorn Helgaas wrote:
> > [+cc linux-pci]
> >
> > On Thu, Sep 14, 2017 at 12:00 PM, Jens Axboe wrote:
> >> On 09/12/2017 02:04 PM, Johannes Berg wrote:
> >>> On Tue,
On 09/14/2017 11:11 AM, Bjorn Helgaas wrote:
> [+cc linux-pci]
>
> On Thu, Sep 14, 2017 at 12:00 PM, Jens Axboe wrote:
>> On 09/12/2017 02:04 PM, Johannes Berg wrote:
>>> On Tue, 2017-09-12 at 13:43 -0600, Jens Axboe wrote:
>>>
CC'ing the guilty part and Bjorn. I'm assuming
[+cc linux-pci]
On Thu, Sep 14, 2017 at 12:00 PM, Jens Axboe wrote:
> On 09/12/2017 02:04 PM, Johannes Berg wrote:
>> On Tue, 2017-09-12 at 13:43 -0600, Jens Axboe wrote:
>>
>>> CC'ing the guilty part and Bjorn. I'm assuming it's the
>>> pci_is_enabled() check, since the rest of
On 09/12/2017 02:04 PM, Johannes Berg wrote:
> On Tue, 2017-09-12 at 13:43 -0600, Jens Axboe wrote:
>
>> CC'ing the guilty part and Bjorn. I'm assuming it's the
>> pci_is_enabled() check, since the rest of the patch shouldn't have
>> functional changes.
>
> and pci_enable_bridge() already checks
On Tue, 2017-09-12 at 13:43 -0600, Jens Axboe wrote:
> CC'ing the guilty part and Bjorn. I'm assuming it's the
> pci_is_enabled() check, since the rest of the patch shouldn't have
> functional changes.
and pci_enable_bridge() already checks if it's already enabled, but
still enables mastering in
On Tue, 2017-09-12 at 13:43 -0600, Jens Axboe wrote:
> On 09/12/2017 10:36 AM, Luca Coelho wrote:
> > On Tue, 2017-09-12 at 16:11 +, Coelho, Luciano wrote:
> > > Hi Jens,
> > >
> > > On Tue, 2017-09-12 at 09:48 -0600, Jens Axboe wrote:
> > > > Hi,
> > > >
> > > > I have no wifi in current
On 09/12/2017 10:36 AM, Luca Coelho wrote:
> On Tue, 2017-09-12 at 16:11 +, Coelho, Luciano wrote:
>> Hi Jens,
>>
>> On Tue, 2017-09-12 at 09:48 -0600, Jens Axboe wrote:
>>> Hi,
>>>
>>> I have no wifi in current git (8fac2f96ab8), it simply fails with:
>>>
>>> [4.363481] iwlwifi
On 09/12/2017 10:36 AM, Luca Coelho wrote:
> On Tue, 2017-09-12 at 16:11 +, Coelho, Luciano wrote:
>> Hi Jens,
>>
>> On Tue, 2017-09-12 at 09:48 -0600, Jens Axboe wrote:
>>> Hi,
>>>
>>> I have no wifi in current git (8fac2f96ab8), it simply fails with:
>>>
>>> [4.363481] iwlwifi
On Tue, 2017-09-12 at 16:11 +, Coelho, Luciano wrote:
> Hi Jens,
>
> On Tue, 2017-09-12 at 09:48 -0600, Jens Axboe wrote:
> > Hi,
> >
> > I have no wifi in current git (8fac2f96ab8), it simply fails with:
> >
> > [4.363481] iwlwifi :04:00.0: Direct firmware load for
> >
On 09/12/2017 10:11 AM, Coelho, Luciano wrote:
> Hi Jens,
>
> On Tue, 2017-09-12 at 09:48 -0600, Jens Axboe wrote:
>> Hi,
>>
>> I have no wifi in current git (8fac2f96ab8), it simply fails with:
>>
>> [4.363481] iwlwifi :04:00.0: Direct firmware load for
>> iwlwifi-8000C-34.ucode failed
Hi Jens,
On Tue, 2017-09-12 at 09:48 -0600, Jens Axboe wrote:
> Hi,
>
> I have no wifi in current git (8fac2f96ab8), it simply fails with:
>
> [4.363481] iwlwifi :04:00.0: Direct firmware load for
> iwlwifi-8000C-34.ucode failed with error -2
> [4.363733] iwlwifi :04:00.0:
Hi,
I have no wifi in current git (8fac2f96ab8), it simply fails with:
[4.363481] iwlwifi :04:00.0: Direct firmware load for
iwlwifi-8000C-34.ucode failed with error -2
[4.363733] iwlwifi :04:00.0: Direct firmware load for
iwlwifi-8000C-33.ucode failed with error -2
[
31 matches
Mail list logo