<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
<p class="MsoPlainText ContentPasted0" style="margin:0in;font-size:11pt;font-family:Calibri, sans-serif">
Sounds weird to have a platform restriction on uAPI though. UMD not using this extension is not a problem, is it?<o:p class="ContentPasted0"> </o:p></p>
<p class="MsoPlainText" style="margin:0in;font-size:11pt;font-family:Calibri, sans-serif">
<o:p class="ContentPasted0"> </o:p></p>
</div>
<div id="appendonsend"></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
</div>
<div id="divRplyFwdMsg" dir="ltr" class="elementToProof"><font face="Calibri, sans-serif" style="font-size: 11pt; color: rgb(0, 0, 0);"><b>From:</b> Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com><br>
</font></div>
<div dir="ltr"><font face="Calibri, sans-serif" style="font-size: 11pt; color: rgb(0, 0, 0);"><b>Sent:</b> Thursday, May 25, 2023 1:33 AM<br>
<b>To:</b> Yang, Fei <fei.yang@intel.com>; intel-gfx@lists.freedesktop.org <intel-gfx@lists.freedesktop.org><br>
<b>Cc:</b> dri-devel@lists.freedesktop.org <dri-devel@lists.freedesktop.org>; Vivi, Rodrigo <rodrigo.vivi@intel.com><br>
<b>Subject:</b> Re: [Intel-gfx] [PATCH v12 0/1] drm/i915: Allow user to set cache at BO creation</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt">
<div class="PlainText"><br>
On 24/05/2023 21:02, fei.yang@intel.com wrote:<br>
> From: Fei Yang <fei.yang@intel.com><br>
> <br>
> This series introduce a new extension for GEM_CREATE,<br>
> 1. end support for set caching ioctl [PATCH 1/2]<br>
> 2. add set_pat extension for gem_create [PATCH 2/2]<br>
> <br>
> v2: drop one patch that was merged separately<br>
> commit 341ad0e8e254 ("drm/i915/mtl: Add PTE encode function")<br>
> v3: rebased on <a href="https://patchwork.freedesktop.org/series/117082/" data-auth="NotApplicable">
https://patchwork.freedesktop.org/series/117082/</a><br>
> v4: fix missing unlock introduced in v3, and<br>
> solve a rebase conflict<br>
> v5: replace obj->cache_level with pat_set_by_user,<br>
> fix i915_cache_level_str() for legacy platforms.<br>
> v6: rebased on <a href="https://patchwork.freedesktop.org/series/117480/" data-auth="NotApplicable">
https://patchwork.freedesktop.org/series/117480/</a><br>
> v7: rebased on <a href="https://patchwork.freedesktop.org/series/117528/" data-auth="NotApplicable">
https://patchwork.freedesktop.org/series/117528/</a><br>
> v8: dropped the two dependent patches that has been merged<br>
> separately. Add IGT link and Tested-by (MESA).<br>
> v9: addressing comments (Andi)<br>
> v10: acked-by and tested-by MESA<br>
> v11: drop "end support for set caching ioctl" (merged)<br>
> remove tools/include/uapi/drm/i915_drm.h<br>
> v12: drop Bspec reference in comment. add to commit message instead<br>
> <br>
> Fei Yang (1):<br>
> drm/i915: Allow user to set cache at BO creation<br>
> <br>
> drivers/gpu/drm/i915/gem/i915_gem_create.c | 36 +++++++++++++++++++<br>
> drivers/gpu/drm/i915/gem/i915_gem_object.c | 6 ++++<br>
> include/uapi/drm/i915_drm.h | 41 ++++++++++++++++++++++<br>
> 3 files changed, 83 insertions(+)<br>
> <br>
<br>
Do you also have a Test-with: run against the new IGT somewhere?<br>
<br>
Regards,<br>
<br>
Tvrtko<br>
</div>
</span></font></div>
</body>
</html>