From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CFC9CCD11DD for ; Thu, 28 Mar 2024 21:17:07 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 71FAE10EBDC; Thu, 28 Mar 2024 21:17:07 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="GqXl5imX"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.9]) by gabe.freedesktop.org (Postfix) with ESMTPS id 2D90410EA58 for ; Thu, 28 Mar 2024 21:17:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1711660626; x=1743196626; h=date:message-id:from:to:cc:subject:in-reply-to: references:mime-version; bh=cbab/AiD+IEhyPvHeTkaf476MDxENxfqotoiu0D9qaU=; b=GqXl5imXOnL8QWssmOoFixSn6LWQhtyoHuVusccXs0jdnVD5O6PKqxt/ Rtbh2Yl7edmZWrnlH0Gu6gnhyLQjSAcm4khT/vEKqT55coRok9UthJwjg Vp3GR7gON6GGq124sm1YLwjmVQndfF/E4+bckavpGxEeUQd69gj27QY66 x2oAhGNPjgqbtKSVXEpiF630TFg6gtFFKJWoJOeFntUOrc7WFZ2+fhv7q 0G4ZI0TEGdc9MK+DCFrJIdAX6kh3ANxnnlawWkS581VRovBW61bHyUPMM 7TS3kIurj85UlnG84nAnYWvL40GqjHPjt1Qa3GZYU0fS3m3yPTaNDt+E/ g==; X-CSE-ConnectionGUID: fz/klmWoQR6SEysLaMsepg== X-CSE-MsgGUID: Ecq1BVA6RYa2Z0impRQeHA== X-IronPort-AV: E=McAfee;i="6600,9927,11027"; a="17567403" X-IronPort-AV: E=Sophos;i="6.07,162,1708416000"; d="scan'208";a="17567403" Received: from fmviesa005.fm.intel.com ([10.60.135.145]) by fmvoesa103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Mar 2024 14:17:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,162,1708416000"; d="scan'208";a="21242231" Received: from orsosgc001.jf.intel.com (HELO orsosgc001.intel.com) ([10.165.21.138]) by fmviesa005-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Mar 2024 14:17:05 -0700 Date: Thu, 28 Mar 2024 14:17:04 -0700 Message-ID: <857chmvyy7.wl-ashutosh.dixit@intel.com> From: "Dixit, Ashutosh" To: Francois Dugast Cc: , Kamil Konieczny , Mauro Carvalho Chehab , Lucas De Marchi , Rodrigo Vivi , Matt Roper Subject: Re: [PATCH i-g-t] drm-uapi: Align header with drm-xe-next In-Reply-To: <20240328173243.7-1-francois.dugast@intel.com> References: <20240328173243.7-1-francois.dugast@intel.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?ISO-8859-4?Q?Goj=F2?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/28.2 (x86_64-redhat-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-BeenThere: igt-dev@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development mailing list for IGT GPU Tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" On Thu, 28 Mar 2024 10:32:43 -0700, Francois Dugast wrote: > Hi Francois, > Currently xe_drm.h is based on drm-next with some additions from > drm-xe-next. > > However some uAPI updates brought by commit ca4607939 ("drm-uapi/xe: \ > Remove unused flags") were mistakenly reverted by commit aef5f4742 > ("drm-uapi: sync with drm-next f112b68f273f"). > > Ensure xe_drm.h is now aligned with kernel commit ("drm/xe/uapi: Add \ > IP version and stepping to GT list query"). Let me copy some people and see what they say and what we should be doing here. Previously we were making sure that IGT uapi files in include/drm-uapi/ were in sync with kernel drm-next branch. I am not sure what the reason for doing this was. When we were doing this, any definitions not in drm-next were put in lib/i915/i915_drm_local.h. So should we be doing this now even for Xe or not? Or we can just keep the IGT uapi headers in sync with drm-xe-next (or drm-tip)? If not, we would need to create a lib/xe/xe_drm_local.h and put stuff not in drm-next there. This patch syncs IGT with the uapi header in drm-xe-next. If this is ok, this is: Reviewed-by: Ashutosh Dixit > > Cc: Ashutosh Dixit > Signed-off-by: Francois Dugast > --- > include/drm-uapi/xe_drm.h | 19 ------------------- > 1 file changed, 19 deletions(-) > > diff --git a/include/drm-uapi/xe_drm.h b/include/drm-uapi/xe_drm.h > index 4353595a4..91ea92035 100644 > --- a/include/drm-uapi/xe_drm.h > +++ b/include/drm-uapi/xe_drm.h > @@ -871,10 +871,6 @@ struct drm_xe_vm_destroy { > * - %DRM_XE_VM_BIND_OP_PREFETCH > * > * and the @flags can be: > - * - %DRM_XE_VM_BIND_FLAG_READONLY > - * - %DRM_XE_VM_BIND_FLAG_IMMEDIATE - Valid on a faulting VM only, do the > - * MAP operation immediately rather than deferring the MAP to the page > - * fault handler. > * - %DRM_XE_VM_BIND_FLAG_NULL - When the NULL flag is set, the page > * tables are setup with a special bit which indicates writes are > * dropped and all reads return zero. In the future, the NULL flags > @@ -967,8 +963,6 @@ struct drm_xe_vm_bind_op { > /** @op: Bind operation to perform */ > __u32 op; > > -#define DRM_XE_VM_BIND_FLAG_READONLY (1 << 0) > -#define DRM_XE_VM_BIND_FLAG_IMMEDIATE (1 << 1) > #define DRM_XE_VM_BIND_FLAG_NULL (1 << 2) > #define DRM_XE_VM_BIND_FLAG_DUMPABLE (1 << 3) > /** @flags: Bind flags */ > @@ -1085,19 +1079,6 @@ struct drm_xe_exec_queue_create { > #define DRM_XE_EXEC_QUEUE_EXTENSION_SET_PROPERTY 0 > #define DRM_XE_EXEC_QUEUE_SET_PROPERTY_PRIORITY 0 > #define DRM_XE_EXEC_QUEUE_SET_PROPERTY_TIMESLICE 1 > -#define DRM_XE_EXEC_QUEUE_SET_PROPERTY_PREEMPTION_TIMEOUT 2 > -#define DRM_XE_EXEC_QUEUE_SET_PROPERTY_JOB_TIMEOUT 4 > -#define DRM_XE_EXEC_QUEUE_SET_PROPERTY_ACC_TRIGGER 5 > -#define DRM_XE_EXEC_QUEUE_SET_PROPERTY_ACC_NOTIFY 6 > -#define DRM_XE_EXEC_QUEUE_SET_PROPERTY_ACC_GRANULARITY 7 > -/* Monitor 128KB contiguous region with 4K sub-granularity */ > -#define DRM_XE_ACC_GRANULARITY_128K 0 > -/* Monitor 2MB contiguous region with 64KB sub-granularity */ > -#define DRM_XE_ACC_GRANULARITY_2M 1 > -/* Monitor 16MB contiguous region with 512KB sub-granularity */ > -#define DRM_XE_ACC_GRANULARITY_16M 2 > -/* Monitor 64MB contiguous region with 2M sub-granularity */ > -#define DRM_XE_ACC_GRANULARITY_64M 3 > > /** @extensions: Pointer to the first extension struct, if any */ > __u64 extensions; > -- > 2.34.1 >