|Summary:||Aperture space rollback snafu|
|Product:||Mesa||Reporter:||Chris Wilson <chris>|
|Component:||Drivers/DRI/i965||Assignee:||Intel 3D Bugs Mailing List <intel-3d-bugs>|
|Status:||RESOLVED MOVED||QA Contact:||Intel 3D Bugs Mailing List <intel-3d-bugs>|
|i915 platform:||i915 features:|
Description Chris Wilson 2018-05-26 16:30:31 UTC
Since we may update context state as we compute the batch, that state is preserved as we rollback the batch for a failed aperture check -- the subsequent emission then skips the atoms as the context state is believed to match requirements. E.g. gen7_upload_urb(). Affects: genX(blorp_exec) brw_dispatch_compute_common brw_draw_single_prim The aperture rollback has been in place since commit 3d851ae48865ac6a20a615e6e06d913c2265cda5 Author: Eric Anholt <firstname.lastname@example.org> Date: Sat Oct 22 09:37:57 2011 -0700 i965: Use the batch save/reset code to avoid needing the BO validate step. 7.10-branchpoint-6137-g3d851a! The issue with believing global state for urb in particular seems to be from commit d5e217dbfda2a87e149bdc8586c25143fc0ac183 Author: Jason Ekstrand <email@example.com> Date: Fri Aug 18 16:10:39 2017 -0700 i965: Stop looking at NewDriverState when emitting 3DSTATE_URB I'm not sure if that's the only instance.
Comment 1 GitLab Migration User 2019-09-25 19:11:23 UTC
-- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/mesa/mesa/issues/1728.