Re: [Intel-gfx] [PATCH v2] drm/i915/ctx: Remove bad invariant

2013-02-12 Thread Daniel Vetter
On Mon, Feb 11, 2013 at 01:31:27PM -0800, Ben Widawsky wrote: > It's not that the assertion is incorrect, but rather that we can call > do_destroy early in loading, and we will falsely BUG(). > > Since contexts have been in for a while now, and in the internal APIs > are pretty stable, it should b

[Intel-gfx] [PATCH v2] drm/i915/ctx: Remove bad invariant

2013-02-11 Thread Ben Widawsky
It's not that the assertion is incorrect, but rather that we can call do_destroy early in loading, and we will falsely BUG(). Since contexts have been in for a while now, and in the internal APIs are pretty stable, it should be fairly safe to remove this. v2: Remove unused dev_priv, and dev Sign