diff options
| author | Jason Ekstrand <jason@jlekstrand.net> | 2021-07-14 14:34:19 -0500 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2021-07-16 21:48:02 +0200 |
| commit | 0c6609bb20cf473f48403763aa9a9504ff95fa0f (patch) | |
| tree | 697d9a44506fa432138d7237833c5425ac465614 /include | |
| parent | dc194184d0ce1ba7837f91e0af20e95923049d4d (diff) | |
Revert "drm/i915: Skip over MI_NOOP when parsing"
This reverts a6c5e2aea704 ("drm/i915: Skip over MI_NOOP when parsing").
It complicates the batch parsing code a bit and increases indentation
for no reason other than fast-skipping a command that userspace uses
only rarely. Sure, there may be IGT tests that fill batches with NOOPs
but that's not a case we should optimize for in the kernel. We should
optimize for code clarity instead.
Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Jon Bloomfield <jon.bloomfield@intel.com>
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20210714193419.1459723-6-jason@jlekstrand.net
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions