Re: [PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-09 Thread Wei Yang
On Fri, Aug 07, 2015 at 06:59:58PM +1000, Alexey Kardashevskiy wrote: >On 08/07/2015 12:01 PM, Wei Yang wrote: >>On Thu, Aug 06, 2015 at 08:04:58PM +1000, Alexey Kardashevskiy wrote: >>>On 08/05/2015 11:25 AM, Wei Yang wrote: In current implementation, when VF BAR is bigger than 64MB, it uses 4

Re: [PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-07 Thread Alexey Kardashevskiy
On 08/07/2015 12:01 PM, Wei Yang wrote: On Thu, Aug 06, 2015 at 08:04:58PM +1000, Alexey Kardashevskiy wrote: On 08/05/2015 11:25 AM, Wei Yang wrote: In current implementation, when VF BAR is bigger than 64MB, it uses 4 M64 BAR in Single PE mode to cover the number of VFs required to be enabled

Re: [PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-07 Thread Alexey Kardashevskiy
On 08/07/2015 11:48 AM, Wei Yang wrote: On Thu, Aug 06, 2015 at 08:07:01PM +1000, Gavin Shan wrote: On Thu, Aug 06, 2015 at 05:36:02PM +0800, Wei Yang wrote: On Thu, Aug 06, 2015 at 03:20:25PM +1000, Gavin Shan wrote: On Wed, Aug 05, 2015 at 09:25:00AM +0800, Wei Yang wrote: In current implem

Re: [PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-06 Thread Wei Yang
On Thu, Aug 06, 2015 at 08:04:58PM +1000, Alexey Kardashevskiy wrote: >On 08/05/2015 11:25 AM, Wei Yang wrote: >>In current implementation, when VF BAR is bigger than 64MB, it uses 4 M64 >>BAR in Single PE mode to cover the number of VFs required to be enabled. >>By doing so, several VFs would be i

Re: [PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-06 Thread Wei Yang
On Thu, Aug 06, 2015 at 08:07:01PM +1000, Gavin Shan wrote: >On Thu, Aug 06, 2015 at 05:36:02PM +0800, Wei Yang wrote: >>On Thu, Aug 06, 2015 at 03:20:25PM +1000, Gavin Shan wrote: >>>On Wed, Aug 05, 2015 at 09:25:00AM +0800, Wei Yang wrote: In current implementation, when VF BAR is bigger than

Re: [PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-06 Thread Gavin Shan
On Thu, Aug 06, 2015 at 05:36:02PM +0800, Wei Yang wrote: >On Thu, Aug 06, 2015 at 03:20:25PM +1000, Gavin Shan wrote: >>On Wed, Aug 05, 2015 at 09:25:00AM +0800, Wei Yang wrote: >>>In current implementation, when VF BAR is bigger than 64MB, it uses 4 M64 >>>BAR in Single PE mode to cover the numbe

Re: [PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-06 Thread Alexey Kardashevskiy
On 08/05/2015 11:25 AM, Wei Yang wrote: In current implementation, when VF BAR is bigger than 64MB, it uses 4 M64 BAR in Single PE mode to cover the number of VFs required to be enabled. By doing so, several VFs would be in one VF Group and leads to interference between VFs in the same group. Th

Re: [PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-06 Thread Wei Yang
On Thu, Aug 06, 2015 at 03:20:25PM +1000, Gavin Shan wrote: >On Wed, Aug 05, 2015 at 09:25:00AM +0800, Wei Yang wrote: >>In current implementation, when VF BAR is bigger than 64MB, it uses 4 M64 >>BAR in Single PE mode to cover the number of VFs required to be enabled. >>By doing so, several VFs wo

Re: [PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-05 Thread Gavin Shan
On Wed, Aug 05, 2015 at 09:25:00AM +0800, Wei Yang wrote: >In current implementation, when VF BAR is bigger than 64MB, it uses 4 M64 >BAR in Single PE mode to cover the number of VFs required to be enabled. >By doing so, several VFs would be in one VF Group and leads to interference >between VFs in

[PATCH V2 3/6] powerpc/powernv: use one M64 BAR in Single PE mode for one VF BAR

2015-08-04 Thread Wei Yang
In current implementation, when VF BAR is bigger than 64MB, it uses 4 M64 BAR in Single PE mode to cover the number of VFs required to be enabled. By doing so, several VFs would be in one VF Group and leads to interference between VFs in the same group. This patch changes the design by using one M