From: Andi Shyti <andi.shyti@linux.intel.com>
To: "Zhang, Carl" <carl.zhang@intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>,
"Gu, Lihao" <lihao.gu@intel.com>,
Andi Shyti <andi.shyti@linux.intel.com>,
"Justen, Jordan L" <jordan.l.justen@intel.com>,
Intel GFX <intel-gfx@lists.freedesktop.org>,
DRI Devel <dri-devel@lists.freedesktop.org>,
Chris Wilson <chris@chris-wilson.co.uk>,
"Yang, Fei" <fei.yang@intel.com>,
"Roper, Matthew D" <matthew.d.roper@intel.com>
Subject: Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation
Date: Fri, 9 Jun 2023 11:59:56 +0200 [thread overview]
Message-ID: <ZIL4HNonFakDqJtB@ashyti-mobl2.lan> (raw)
In-Reply-To: <PH0PR11MB55793C2F3B66887186CF68198753A@PH0PR11MB5579.namprd11.prod.outlook.com>
Hi Carl,
> > > besides this, ask a dumb question.
> > > How we retrieve the pat_index from a shared resource though dma_buf fd?
> > > maybe we need to know whether it could be CPU cached if we want map it.
> > > Of course, looks there are no real usage to access it though CPU.
> > > Just use it directly without any pat related options ?
> >
> > I am not understanding. Do you want to ask the PAT table to the driver? Are
> > you referring to the CPU PAT index?
> >
> > In any case, if I understood correctly, you don't necessarily always need to
> > set the PAT options and the cache options will fall into the default values.
> >
> > Please let me know if I haven't answered the question.
> >
>
> If mesa create a resource , then use DRM_IOCTL_PRIME_HANDLE_TO_FD convert it to a dma fd.
> Then share it to media, media use DRM_IOCTL_PRIME_FD_TO_HANDLE convert it to a gem bo.
> But media does not know the PAT index , because mesa create it and set it.
> So, if media want to call DRM_IOCTL_I915_GEM_MMAP_OFFSET, media does not know whether it could be WB.
That's a good point. To be honest I am not really sure how this
is handled.
Fei, Jordan? Do you have suggestion here?
Andi
next prev parent reply other threads:[~2023-06-09 10:00 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-06 10:00 [PATCH v17 0/1] drm/i915: Allow user to set cache at BO creation Andi Shyti
2023-06-06 10:00 ` [PATCH v17 1/1] " Andi Shyti
2023-06-06 10:10 ` Tvrtko Ursulin
2023-06-06 10:18 ` Andi Shyti
2023-06-06 11:12 ` Joonas Lahtinen
2023-06-06 11:14 ` Andi Shyti
2023-06-07 3:40 ` Zhang, Carl
2023-06-07 5:11 ` Andi Shyti
2023-06-07 5:21 ` Zhang, Carl
2023-06-09 9:59 ` Andi Shyti [this message]
2023-06-09 15:12 ` Yang, Fei
2023-06-11 14:43 ` Zhang, Carl
2023-06-07 15:01 ` Andi Shyti
2023-06-07 15:46 ` Andi Shyti
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZIL4HNonFakDqJtB@ashyti-mobl2.lan \
--to=andi.shyti@linux.intel.com \
--cc=carl.zhang@intel.com \
--cc=chris@chris-wilson.co.uk \
--cc=dri-devel@lists.freedesktop.org \
--cc=fei.yang@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jordan.l.justen@intel.com \
--cc=lihao.gu@intel.com \
--cc=matthew.d.roper@intel.com \
--cc=tvrtko.ursulin@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).