[Intel-xe] [RFC PATCH 0/2] Draft: Update Xe uAPI for 32-bit compat, invasively

Christopher Snowhill kode54 at gmail.com
Fri May 26 00:32:19 UTC 2023


This attempt brings in Maarten Lankhorst's original uAPI update, with
suggested changes made to make up the difference. As long as we're
attempting to bring up 32-bit compat capability, we may as well go
whole hog and just break things. Better that way than to deal with the
mess later.

Also introduces field validation against all of the padding and
reserved fields, which must be zero, in a separate commit.

This is the invasive update, as a draft, for comment and examination.
It's probably not going to be applied any time soon, but it's here,
and it's been updated based on the comments against the original
invasive attempt.

v3:
  Redid everything, maybe this one will be a little more permanent.
v2:
  Removed extensions checks where there were none originally. (José)
  Moved extraneous parentheses to the correct places. (Lucas)

Suggested-by: Ryan Houdek <sonicadvance1 at gmail.com>
Signed-off-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
Signed-off-by: Christopher Snowhill <kode54 at gmail.com>

Christopher Snowhill (2):
  drm/xe: Reshuffle all of the uAPI structs
  drm/xe: Validate uAPI padding and reserved fields

 drivers/gpu/drm/xe/xe_bo.c              |  6 ++-
 drivers/gpu/drm/xe/xe_engine.c          | 18 +++++--
 drivers/gpu/drm/xe/xe_exec.c            |  4 +-
 drivers/gpu/drm/xe/xe_mmio.c            |  3 +-
 drivers/gpu/drm/xe/xe_query.c           |  3 +-
 drivers/gpu/drm/xe/xe_sync.c            |  4 +-
 drivers/gpu/drm/xe/xe_vm.c              | 21 ++++++--
 drivers/gpu/drm/xe/xe_vm_madvise.c      |  3 +-
 drivers/gpu/drm/xe/xe_wait_user_fence.c |  3 +-
 include/uapi/drm/xe_drm.h               | 64 +++++++++++++++++--------
 10 files changed, 95 insertions(+), 34 deletions(-)

-- 
2.40.1



More information about the Intel-xe mailing list