Re: [Intel-gfx] [PATCH i-g-t] Revert tests/gem_ctx_param_basic: fix invalid params

2015-08-24 Thread Ander Conselvan De Oliveira
On Fri, 2015-08-21 at 16:26 +0300, Ander Conselvan De Oliveira wrote: On Fri, 2015-08-07 at 15:53 +0300, David Weinehall wrote: On Thu, Aug 06, 2015 at 11:33:00PM +0200, Daniel Vetter wrote: This reverts commit 0b45b0746f45deea11670a8b2c949776bbbef55c. The point of testing for

Re: [Intel-gfx] [PATCH i-g-t] Revert tests/gem_ctx_param_basic: fix invalid params

2015-08-21 Thread Ander Conselvan De Oliveira
On Fri, 2015-08-07 at 15:53 +0300, David Weinehall wrote: On Thu, Aug 06, 2015 at 11:33:00PM +0200, Daniel Vetter wrote: This reverts commit 0b45b0746f45deea11670a8b2c949776bbbef55c. The point of testing for LAST_FLAG + 1 is to catch abi extensions - despite our best efforts we really

Re: [Intel-gfx] [PATCH i-g-t] Revert tests/gem_ctx_param_basic: fix invalid params

2015-08-10 Thread David Weinehall
On Fri, Aug 07, 2015 at 10:04:47AM -0300, Paulo Zanoni wrote: 2015-08-06 18:33 GMT-03:00 Daniel Vetter daniel.vet...@ffwll.ch: This reverts commit 0b45b0746f45deea11670a8b2c949776bbbef55c. The point of testing for LAST_FLAG + 1 is to catch abi extensions - despite our best efforts we

Re: [Intel-gfx] [PATCH i-g-t] Revert tests/gem_ctx_param_basic: fix invalid params

2015-08-07 Thread Daniel Vetter
On Fri, Aug 07, 2015 at 03:53:57PM +0300, David Weinehall wrote: On Thu, Aug 06, 2015 at 11:33:00PM +0200, Daniel Vetter wrote: This reverts commit 0b45b0746f45deea11670a8b2c949776bbbef55c. The point of testing for LAST_FLAG + 1 is to catch abi extensions - despite our best efforts we

Re: [Intel-gfx] [PATCH i-g-t] Revert tests/gem_ctx_param_basic: fix invalid params

2015-08-07 Thread David Weinehall
On Thu, Aug 06, 2015 at 11:33:00PM +0200, Daniel Vetter wrote: This reverts commit 0b45b0746f45deea11670a8b2c949776bbbef55c. The point of testing for LAST_FLAG + 1 is to catch abi extensions - despite our best efforts we really suck at properly reviewing for test coverage when extending ABI.

Re: [Intel-gfx] [PATCH i-g-t] Revert tests/gem_ctx_param_basic: fix invalid params

2015-08-07 Thread Paulo Zanoni
2015-08-06 18:33 GMT-03:00 Daniel Vetter daniel.vet...@ffwll.ch: This reverts commit 0b45b0746f45deea11670a8b2c949776bbbef55c. The point of testing for LAST_FLAG + 1 is to catch abi extensions - despite our best efforts we really suck at properly reviewing for test coverage when extending

[Intel-gfx] [PATCH i-g-t] Revert tests/gem_ctx_param_basic: fix invalid params

2015-08-06 Thread Daniel Vetter
This reverts commit 0b45b0746f45deea11670a8b2c949776bbbef55c. The point of testing for LAST_FLAG + 1 is to catch abi extensions - despite our best efforts we really suck at properly reviewing for test coverage when extending ABI. The real bug here is that David Weinhall hasn't submitted updated