On 26/01/17 09:46 AM, Sinclair Yeh wrote: > On Wed, Jan 25, 2017 at 10:49:33AM +0100, Christian König wrote: >> Am 25.01.2017 um 10:25 schrieb Thomas Hellstrom: >>> On 01/25/2017 09:21 AM, Michel Dänzer wrote: >>>> From: Michel Dänzer <michel.daen...@amd.com> >>>> >>>> The current caching state may not be tt_cached, even though the >>>> placement contains TTM_PL_FLAG_CACHED, because placement can contain >>>> multiple caching flags. Trying to swap out such a BO would trip up the >>>> >>>> BUG_ON(ttm->caching_state != tt_cached); >>>> >>>> in ttm_tt_swapout. >>>> >>>> Cc: sta...@vger.kernel.org >>>> Signed-off-by: Michel Dänzer <michel.daen...@amd.com> >>> Reviewed-by: Thomas Hellstrom <thellst...@vmware.com> >> >> Reviewed-by: Christian König <christian.koe...@amd.com>. > > Reviewed-by: Sinclair Yeh <s...@vmware.com>
Thanks for the reviews! Via which tree should we merge this? -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer
