[PATCH 1/4] drm/ttm: add TTM_PL_FLAG_TEMPORARY flag v3

[Christian König]

Hi Nirmoy,

can you give that patch set a testing round and an review?

The memory stress test with low VRAM on APUs should be sufficient and 
you should have that setup still around.

Thanks in advance,
Christian.

Am 07.06.21 um 14:58 schrieb Yu, Lang:
>
> [AMD Official Use Only]
>
>
> Thanks，please let Nirmoy do it.
>
> Regards，
> Lang
>
> 获取 Outlook for iOS <https://aka.ms/o0ukef>
> ------------------------------------------------------------------------
> *发件人:* Christian König <ckoenig.leichtzumerken at gmail.com>
> *发送时间:* Monday, June 7, 2021 8:41:49 PM
> *收件人:* Thomas Hellström (Intel) <thomas_os at shipmail.org>; Yu, Lang 
> <Lang.Yu at amd.com>
> *抄送:* dri-devel at lists.freedesktop.org <dri-devel at lists.freedesktop.org>
> *主题:* Re: [PATCH 1/4] drm/ttm: add TTM_PL_FLAG_TEMPORARY flag v3
> Thanks!
>
> Lang can anybody from your team give as an reviewed-by/test-by on the
> amdgpu patches? I just want another pair of eyes looking over it.
>
> If nobody else has time I will ask Nirmoy for a quick testing round.
>
> Thanks,
> Christian.
>
> Am 07.06.21 um 14:38 schrieb Thomas Hellström (Intel):
> > Sure. LGTM,
> >
> > Reviewed-by: Thomas Hellström <thomas.hellstrom at linux.intel.com>
> >
> >
> > On 6/7/21 2:36 PM, Christian König wrote:
> >> Thomas any comments on this?
> >>
> >> Is the purpose of this now clear enough?
> >>
> >> Thanks,
> >> Christian.
> >>
> >> Am 01.06.21 um 14:25 schrieb Christian König:
> >>> From: Lang Yu <Lang.Yu at amd.com>
> >>>
> >>> Sometimes drivers need to use bounce buffers to evict BOs. While
> >>> those reside
> >>> in some domain they are not necessarily suitable for CS.
> >>>
> >>> Add a flag so that drivers can note that a bounce buffers needs to be
> >>> reallocated during validation.
> >>>
> >>> v2: add detailed comments
> >>> v3 (chk): merge commits and rework commit message
> >>>
> >>> Suggested-by: Christian König <christian.koenig at amd.com>
> >>> Signed-off-by: Lang Yu <Lang.Yu at amd.com>
> >>> Signed-off-by: Christian König <christian.koenig at amd.com>
> >>> ---
> >>>   drivers/gpu/drm/ttm/ttm_bo.c    | 3 +++
> >>>   include/drm/ttm/ttm_placement.h | 7 +++++--
> >>>   2 files changed, 8 insertions(+), 2 deletions(-)
> >>>
> >>> diff --git a/drivers/gpu/drm/ttm/ttm_bo.c
> >>> b/drivers/gpu/drm/ttm/ttm_bo.c
> >>> index 51a94fd63bd7..6b393502198e 100644
> >>> --- a/drivers/gpu/drm/ttm/ttm_bo.c
> >>> +++ b/drivers/gpu/drm/ttm/ttm_bo.c
> >>> @@ -929,6 +929,9 @@ static bool ttm_bo_places_compat(const struct
> >>> ttm_place *places,
> >>>   {
> >>>       unsigned i;
> >>>   +    if (mem->placement & TTM_PL_FLAG_TEMPORARY)
> >>> +        return false;
> >>> +
> >>>       for (i = 0; i < num_placement; i++) {
> >>>           const struct ttm_place *heap = &places[i];
> >>>   diff --git a/include/drm/ttm/ttm_placement.h
> >>> b/include/drm/ttm/ttm_placement.h
> >>> index aa6ba4d0cf78..8995c9e4ec1b 100644
> >>> --- a/include/drm/ttm/ttm_placement.h
> >>> +++ b/include/drm/ttm/ttm_placement.h
> >>> @@ -47,8 +47,11 @@
> >>>    * top of the memory area, instead of the bottom.
> >>>    */
> >>>   -#define TTM_PL_FLAG_CONTIGUOUS  (1 << 19)
> >>> -#define TTM_PL_FLAG_TOPDOWN     (1 << 22)
> >>> +#define TTM_PL_FLAG_CONTIGUOUS  (1 << 0)
> >>> +#define TTM_PL_FLAG_TOPDOWN     (1 << 1)
> >>> +
> >>> +/* For multihop handling */
> >>> +#define TTM_PL_FLAG_TEMPORARY   (1 << 2)
> >>>     /**
> >>>    * struct ttm_place
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20210607/2dadcf78/attachment.htm>