[0/4] drm/i915: uAPI clean-ups part 2

Message

Jason Ekstrand March 19, 2021, 10:38 p.m. UTC

This patch series is the second installment of my quest to clean up the
i915 uAPI.  The first three patches delete interfaces which have only ever
been used by i-g-t and never any real userspace.  In the case of NO_ZEROMAP,
it's unclear exactly how this happened.  There were userspace patches for
it back in the day, they just never landed.  For the others, there was
never any userspace beyond i-g-t and they never should have landed in the
first place.

The last patch moves the SINGLE_TIMELINE API to an emulation using syncobj
to help simplify some of our locking infrastructure.

Test-with: [e-mail]

Jason Ekstrand (4):
  drm/i915: Drop I915_CONTEXT_PARAM_RINGSIZE
  drm/i915: Drop I915_CONTEXT_PARAM_NO_ZEROMAP
  drm/i915: Drop the CONTEXT_CLONE API
  drm/i915: Implement SINGLE_TIMELINE with a syncobj

 drivers/gpu/drm/i915/Makefile                 |   1 -
 drivers/gpu/drm/i915/gem/i915_gem_context.c   | 372 ++----------------
 .../gpu/drm/i915/gem/i915_gem_context_types.h |   9 +-
 .../gpu/drm/i915/gem/i915_gem_execbuffer.c    |  23 +-
 drivers/gpu/drm/i915/gt/intel_context_param.c |  63 ---
 drivers/gpu/drm/i915/gt/intel_context_param.h |  14 -
 include/uapi/drm/i915_drm.h                   |  40 +-
 7 files changed, 55 insertions(+), 467 deletions(-)
 delete mode 100644 drivers/gpu/drm/i915/gt/intel_context_param.c
 delete mode 100644 drivers/gpu/drm/i915/gt/intel_context_param.h

Comments

Jani Nikula March 22, 2021, 11:55 a.m. UTC | #1

On Fri, 19 Mar 2021, Patchwork <[e-mail]> wrote:
> == Series Details ==
>
> Series: drm/i915: uAPI clean-ups part 2
> URL   : https://patchwork.freedesktop.org/series/88196/
> State : failure
>
> == Summary ==
>
> Applying: drm/i915: Drop I915_CONTEXT_PARAM_RINGSIZE
> Applying: drm/i915: Drop I915_CONTEXT_PARAM_NO_ZEROMAP
> Applying: drm/i915: Drop the CONTEXT_CLONE API
> Applying: drm/i915: Implement SINGLE_TIMELINE with a syncobj
> error: sha1 information is lacking or useless (drivers/gpu/drm/i915/gem/i915_gem_context.c).
> error: could not build fake ancestor
> hint: Use 'git am --show-current-patch=diff' to see the failed patch
> Patch failed at 0004 drm/i915: Implement SINGLE_TIMELINE with a syncobj
> When you have resolved this problem, run "git am --continue".
> If you prefer to skip this patch, run "git am --skip" instead.
> To restore the original branch and stop patching, run "git am --abort".

CI runs on top of drm-tip branch. What's this based on?

BR,
Jani.

Jason Ekstrand March 22, 2021, 4:11 p.m. UTC | #2

On Mon, Mar 22, 2021 at 6:55 AM Jani Nikula <[e-mail]> wrote:
>
> On Fri, 19 Mar 2021, Patchwork <[e-mail]> wrote:
> > == Series Details ==
> >
> > Series: drm/i915: uAPI clean-ups part 2
> > URL   : https://patchwork.freedesktop.org/series/88196/
> > State : failure
> >
> > == Summary ==
> >
> > Applying: drm/i915: Drop I915_CONTEXT_PARAM_RINGSIZE
> > Applying: drm/i915: Drop I915_CONTEXT_PARAM_NO_ZEROMAP
> > Applying: drm/i915: Drop the CONTEXT_CLONE API
> > Applying: drm/i915: Implement SINGLE_TIMELINE with a syncobj
> > error: sha1 information is lacking or useless (drivers/gpu/drm/i915/gem/i915_gem_context.c).
> > error: could not build fake ancestor
> > hint: Use 'git am --show-current-patch=diff' to see the failed patch
> > Patch failed at 0004 drm/i915: Implement SINGLE_TIMELINE with a syncobj
> > When you have resolved this problem, run "git am --continue".
> > If you prefer to skip this patch, run "git am --skip" instead.
> > To restore the original branch and stop patching, run "git am --abort".
>
> CI runs on top of drm-tip branch. What's this based on?

drm/topic/i915-gem-next

--Jason