Re: [Intel-gfx] [PATCH i-g-t v4 6/6] tests/kms_ccs: Test case for wrong aux buffer stride size

2017-10-30 Thread Ville Syrjälä
On Tue, Oct 03, 2017 at 09:40:59AM -0700, Ben Widawsky wrote: > On 17-09-27 15:34:19, Gabriel Krisman Bertazi wrote: > >Two scenarios tested: > > - unaligned stride > > - Stride too small > > > >Signed-off-by: Gabriel Krisman Bertazi > > Jason, could you provide your

Re: [Intel-gfx] [PATCH i-g-t v4 6/6] tests/kms_ccs: Test case for wrong aux buffer stride size

2017-10-30 Thread Ville Syrjälä
On Tue, Oct 03, 2017 at 01:57:13PM +0300, Arkadiusz Hiler wrote: > On Wed, Sep 27, 2017 at 03:34:19PM -0300, Gabriel Krisman Bertazi wrote: > > Two scenarios tested: > > - unaligned stride > > - Stride too small > > > > Signed-off-by: Gabriel Krisman Bertazi > >

Re: [Intel-gfx] [PATCH i-g-t v4 6/6] tests/kms_ccs: Test case for wrong aux buffer stride size

2017-10-09 Thread Gabriel Krisman Bertazi
Ben Widawsky writes: > On 17-09-27 15:34:19, Gabriel Krisman Bertazi wrote: >>Two scenarios tested: >> - unaligned stride >> - Stride too small >> >>Signed-off-by: Gabriel Krisman Bertazi > > Jason, could you provide your opinion on this? I've

Re: [Intel-gfx] [PATCH i-g-t v4 6/6] tests/kms_ccs: Test case for wrong aux buffer stride size

2017-10-03 Thread Ben Widawsky
On 17-09-27 15:34:19, Gabriel Krisman Bertazi wrote: Two scenarios tested: - unaligned stride - Stride too small Signed-off-by: Gabriel Krisman Bertazi Jason, could you provide your opinion on this? I've always felt the kernel interface shouldn't be validating

Re: [Intel-gfx] [PATCH i-g-t v4 6/6] tests/kms_ccs: Test case for wrong aux buffer stride size

2017-10-03 Thread Arkadiusz Hiler
On Wed, Sep 27, 2017 at 03:34:19PM -0300, Gabriel Krisman Bertazi wrote: > Two scenarios tested: > - unaligned stride > - Stride too small > > Signed-off-by: Gabriel Krisman Bertazi Fails on APL

[Intel-gfx] [PATCH i-g-t v4 6/6] tests/kms_ccs: Test case for wrong aux buffer stride size

2017-09-27 Thread Gabriel Krisman Bertazi
Two scenarios tested: - unaligned stride - Stride too small Signed-off-by: Gabriel Krisman Bertazi --- tests/kms_ccs.c | 23 ++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c index