All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Christian König" <christian.koenig@amd.com>
To: Dave Airlie <airlied@gmail.com>
Cc: "Michel Dänzer" <michel@daenzer.net>,
	dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH] drm/ttm: revert "make TT creation purely optional v3"
Date: Wed, 12 Aug 2020 10:44:08 +0200	[thread overview]
Message-ID: <af875ff5-eb21-c7d2-23f8-50be3975ba7d@amd.com> (raw)
In-Reply-To: <CAPM=9tyUBMXMWbPbXLsNV7j10svYWRHnsYrNOeksQm_EPp-0VA@mail.gmail.com>

Am 11.08.20 um 21:34 schrieb Dave Airlie:
> On Wed, 12 Aug 2020 at 03:11, Christian König
> <ckoenig.leichtzumerken@gmail.com> wrote:
>> Am 11.08.20 um 18:42 schrieb Michel Dänzer:
>>> On 2020-08-11 2:53 p.m., Christian König wrote:
>>>> Am 11.08.20 um 14:52 schrieb Christian König:
>>>>> Am 11.08.20 um 11:24 schrieb Christian König:
>>>>>> This reverts commit 2ddef17678bc2ea1d20517dd2b4ed4aa967ffa8b.
>>>>>>
>>>>>> As it turned out VMWGFX needs a much wider audit to fix this.
>>>>>>
>>>>>> Signed-off-by: Christian König <christian.koenig@amd.com>
>>>>> Dare to give me an rb for this? I already tested on amdgpu and it
>>>>> should be fixing the VMWGFX problems we currently have.
>>>> Ups, that was your old AMD address :)
>>>>
>>>> So once more with feeling,
>>> :)
>>>
>>> However, I'm afraid I'm not deep enough into TTM code anymore to give
>>> you an R-b, given it's not a straight revert (presumably due to other
>>> changes in the meantime).
>> Yeah, indeed had to keep Daves changes intact.
>>
>> Dave can you take a look then?
> Yup just came in after my "stop working Dave" time.
>
> Did you have an amdgpu patch that worked on top of this? does that
> need reverting or did it not land yet?

The amdgpu patch which I already pushed works with or without this. So 
no change needed here.

>
> Otherwise,
>
> Reviewed-by: Dave Airlie <airlied@redhat.com>

Thanks,
Christian.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

      parent reply	other threads:[~2020-08-12  8:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-11  9:24 [PATCH] drm/ttm: revert "make TT creation purely optional v3" Christian König
2020-08-11 12:52 ` Christian König
2020-08-11 12:53   ` Christian König
2020-08-11 16:42     ` Michel Dänzer
2020-08-11 17:11       ` Christian König
2020-08-11 19:34         ` Dave Airlie
2020-08-12  3:02           ` Dave Airlie
2020-08-12  9:08             ` Christian König
2020-08-12  8:44           ` Christian König [this message]

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=af875ff5-eb21-c7d2-23f8-50be3975ba7d@amd.com \
    --to=christian.koenig@amd.com \
    --cc=airlied@gmail.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=michel@daenzer.net \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.