On Tuesday, June 02, 2015 03:23:35 PM Abdiel Janulgue wrote: > > On 06/02/2015 09:31 AM, Kenneth Graunke wrote: > > On Monday, June 01, 2015 03:14:30 PM Abdiel Janulgue wrote: > >> This is needed since kernel doesn't support RS context save and > >> restore on BDW yet. So manually disable hw-generated binding tables > >> when done using it in the batch. Otherwise the GPU would no longer > >> accept software binding tables submitted by other clients including > >> but not limited to the Xorg driver. > >> > >> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> > >> --- > >> src/mesa/drivers/dri/i965/intel_batchbuffer.c | 11 +++++++++++ > >> src/mesa/drivers/dri/i965/intel_batchbuffer.h | 3 ++- > >> 2 files changed, 13 insertions(+), 1 deletion(-) > > > > This seems fairly awful. The kernel should prevent userspace from > > breaking other userspace...in the hardware context world, this really > > doesn't feel like our job. > > > > Why didn't you just update your kernel patch for Broadwell? i.e. make > > > > drm/i915: Enable Resource Streamer state save/restore in HSW > > > > do: > > + if (IS_HASWELL(ring->dev) || INTEL_INFO(ring->dev)->gen >= 8) > > > > instead of: > > + if (IS_HASWELL(ring->dev)) > > > > It looks like the MI_SET_CONTEXT RS save/restore bits you used on > > Haswell still exist on Broadwell. Do they not work or something? > > > > > > I was hoping to have a follow-up for GEN8 as well once the initial > kernel patches get merged :)
We should do it all at once - it should be trivial to support both.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
