Re: [PATCH -v2] PCI: update device mps when doing pci hotplug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2013/8/6 15:37, Li Zefan wrote:
> On 2013/8/6 15:23, Yijing Wang wrote:
>> v1->v2: Update patch log, remove Joe's reported-by, because his problem
>>         was mainly caused by BIOS incorrect setting. But this patch mainly
>> 		to fix the bug caused by device hot add. Conservatively, this 
>> 		version only update the mps problem when hot add. When the device
>> 		mps < parent mps found, this patch try to update device mps.
>> 		It seems unlikely device mps > parent mps after hot add device.
>> 		So we don't care that situation.
>>
>> This patch need to be applied to stable 3.4+
> 
> In this case...
> 
>>
>> Currently we don't update device's mps vaule when doing
>> pci device hot-add. The hot-added device's mps will be set
>> to default value (128B). But the upstream port device's mps
>> may be larger than 128B which was set by firmware during
>> system bootup. In this case the new added device may not
>> work normally. This patch try to update the hot added device
>> mps euqal to its parent mps, if device mpss < parent mps,
>> print warning.
>>
>> References: https://bugzilla.kernel.org/show_bug.cgi?id=60671
>> Reported-by: Yijing Wang <wangyijing@xxxxxxxxxx>
>> Signed-off-by: Yijing Wang <wangyijing@xxxxxxxxxx>
>> Cc: Jon Mason <jdmason@xxxxxxxx>
>> Cc: stable@xxxxxxxxxxxxxxx
> 
> The better tag is:
> 
> Cc: <stable@xxxxxxxxxxxxxxx> # 3.4+
> 
> Otherwise you won't be notified if it can be applied to the most recent
> stable version but failed in older versions.

Will update, thanks!

> 
> 
> .
> 


-- 
Thanks!
Yijing

--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux