Re: [PATCH v5 3/4] PCI: Add a new option for resource_alignment to reassign alignment

2016-09-29 Thread Yongji Xie
On 2016/9/29 19:54, Bjorn Helgaas wrote: On Thu, Sep 29, 2016 at 10:38:01AM +0800, Yongji Xie wrote: On 2016/9/29 6:42, Bjorn Helgaas wrote: On Tue, Sep 13, 2016 at 05:00:33PM +0800, Yongji Xie wrote: When using resource_alignment kernel parameter, the current implement reassigns the alignme

Re: [PATCH v5 3/4] PCI: Add a new option for resource_alignment to reassign alignment

2016-09-29 Thread Bjorn Helgaas
On Thu, Sep 29, 2016 at 10:38:01AM +0800, Yongji Xie wrote: > On 2016/9/29 6:42, Bjorn Helgaas wrote: > > >On Tue, Sep 13, 2016 at 05:00:33PM +0800, Yongji Xie wrote: > >>When using resource_alignment kernel parameter, the current > >>implement reassigns the alignment by changing resources' size >

Re: [PATCH v5 3/4] PCI: Add a new option for resource_alignment to reassign alignment

2016-09-28 Thread Yongji Xie
On 2016/9/29 6:42, Bjorn Helgaas wrote: On Tue, Sep 13, 2016 at 05:00:33PM +0800, Yongji Xie wrote: When using resource_alignment kernel parameter, the current implement reassigns the alignment by changing resources' size which can potentially break some drivers. For example, the driver uses th

Re: [PATCH v5 3/4] PCI: Add a new option for resource_alignment to reassign alignment

2016-09-28 Thread Bjorn Helgaas
On Tue, Sep 13, 2016 at 05:00:33PM +0800, Yongji Xie wrote: > When using resource_alignment kernel parameter, the current > implement reassigns the alignment by changing resources' size > which can potentially break some drivers. For example, the driver > uses the size to locate some register whose

[PATCH v5 3/4] PCI: Add a new option for resource_alignment to reassign alignment

2016-09-13 Thread Yongji Xie
When using resource_alignment kernel parameter, the current implement reassigns the alignment by changing resources' size which can potentially break some drivers. For example, the driver uses the size to locate some register whose length is related to the size. This patch adds a new option "nores