Daniel Vetter <daniel@xxxxxxxx> writes: > On Thu, Oct 3, 2013 at 12:03 AM, Francisco Jerez <currojerez@xxxxxxxxxx> wrote: >> + case I915_PARAM_HAS_ATOMICS: >> + value = 1; >> + break; > > Generally when we do kernel fixes for gpu hangs like that we don't add > parameters (would drown in them otherwise) but simply queue it up to > -fixes and slap a cc: stable on it. Gpu hang fixes are critical enough > imo for that treatment, even when it's for brand new userspace code. > > Any specific reason why we shouldn't follow this approach here? I'd > make the patch simpler and we could dump a bit of userspace code, too. Well, what it means is that people who pull new mesa on their old kernel will reliably get GPU hangs when running piglit, which is something we've avoided in the past when enabling new features (I915_PARAM_HAS_GEN7_SOL_RESET for example).
Attachment:
pgpXyS_yrzT1z.pgp
Description: PGP signature
_______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
