From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754430AbdA0IlO (ORCPT ); Fri, 27 Jan 2017 03:41:14 -0500 Received: from mail-sn1nam01on0047.outbound.protection.outlook.com ([104.47.32.47]:35648 "EHLO NAM01-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754384AbdA0Ika (ORCPT ); Fri, 27 Jan 2017 03:40:30 -0500 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Christian.Koenig@amd.com; Subject: Re: [PATCH] drm/ttm: Make sure BOs being swapped out are cacheable To: Thomas Hellstrom , =?UTF-8?Q?Michel_D=c3=a4nzer?= , Sinclair Yeh , , References: <20170125082131.25544-1-michel@daenzer.net> <56e0a49e-89b6-4542-e905-57f0dc2ae79c@amd.com> <20170126004632.GA3457@vmware.com> <9af4b5b5-a8a5-8908-f59f-6cdc8efe0570@daenzer.net> <5778f881-ce81-1c58-f2d4-02175c3d2615@vmware.com> <20170127073040.l2ofthrhsbteahpx@phenom.ffwll.local> From: =?UTF-8?Q?Christian_K=c3=b6nig?= Message-ID: <5e8d77b2-8b71-6542-4ef5-c7a5d16f2455@amd.com> Date: Fri, 27 Jan 2017 09:22:47 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <20170127073040.l2ofthrhsbteahpx@phenom.ffwll.local> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [2a02:908:1251:7981:5d6a:1040:c41c:5a5] X-ClientProxiedBy: VI1PR07CA0092.eurprd07.prod.outlook.com (10.165.229.146) To CY4PR12MB1301.namprd12.prod.outlook.com (10.168.168.138) X-MS-Office365-Filtering-Correlation-Id: 26573c62-33cc-425b-d880-08d4468db63a X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(48565401076);SRVR:CY4PR12MB1301; X-Microsoft-Exchange-Diagnostics: 1;CY4PR12MB1301;3:Djlb180YhS65xDe07GhRAqSeag56cvmgHNvexxltHuxM3w06jRrhLV38l0sE3yl1nBg7KkHd3zlbxEWoq0j5m9WpXCajG97x3GDOTHASIs89OoAO/HtGpEOWIocNI0ILAHhGcsMau0FJ2nstprFMkakKK+fHWRhH14T1H8hUqTsC02AA0nMB86LQjGPJsQkLpp6fxrUYHCDW6gAWtXUnySLjo3SDCs7ce/oG3WrKmnB04N/nbOYdaSgTD4sMEZRobQF0OXBFyqwUtUWHvt0VE09ihPNcWYoOjnZT7dkmNg0= X-Microsoft-Exchange-Diagnostics: 1;CY4PR12MB1301;25:z9qR3yhNpt+Fzi/a6AK7LYJAUwf3v1Y3JCBNMZEqSSi1YNm1Wv19kn2FgFKCl+OvhBXUUGWbRhDLpEzfhMg7i8ZzjZJ5nrSVP5vcAGytltLLDj9y/L249WmIcK8IMetFcQnMIxg0/XUFBKqBzeNge80HttwPFUegKRtrpuXMgbFnBYfplVn9PjQni1rXhYwQo8JtiAZK7Tk25MmjW4JjVVCnvAJYMLsWgAEWr/coHVkYmU9lEsPxIntMOb+qV7u5pJeDMM1lpC8kSG6Jy9WWdyC6sD9hJygbnKnlNIOozaLbER3tFm+m8qXS9gjnclI7DG3Vs1nRrEZUL1cf5XJ7A1ZYq9fFxrq8Pg1M7Vv/zUhvZfBmX4IHsrSRyQ10PYoTQy3US0g/DHyyCirKRFlhUxc031EZMd8P5MOp7YPWbt4fl7AbLKU/OhpkURwngRVIRUkUsWzrnp1fDyU/FpoupPPkc4LQdZJmKTD5U/Ce1K5xR01RXUXng3hAv7cdNXx9eQNPD77Yr9re38srMzNVqH7emqylUU9e6HWUiY9SZ64Vd10uaz2R+KcylBor06aqkcKo7JyeAvv1vqU+kImWGlFUsu2o0SWw1fd9tQpbPDqg40MYw31783AfNUG2ihCKYlu1OGsE42p7qUQ3dbkh1MyKjxROFjcRS8Ups982+YC9hpaznSF/+kzlJchhbo9W X-Microsoft-Exchange-Diagnostics: 1;CY4PR12MB1301;31:SFu9PhFMlKuoxdPww+BZv+emL9RyL75nII6dhLTQMVcgOyQK1ZuelrCTZadCM4BAALDZ2/50ZyG5rLhT7GhX5AT0xFIcsqqP+qAP9k1KNmumflxEXL92nVqSPeHS9qufm/b/zAkE2RCRmnBAKrRdC2T5kyrEU7WUs5LyvcqVcloMhcLq6tbtHdLpHBXmV3+dG1UUlv4cUw6ib+4zoWyXH+QUqoJi2VTYQEbhQwOQY0KK/eRDO+zvNsxtKJWjjLnpphHggVm/Bh8yF2+sRIxfnkjSwDAUipjpOzSjrZTj1e0=;20:x1EYry9oyWYCoxXiJ1gCh+383//uA1CVResJtTjKRqq/rD24oDb0YK7G9tQcXrYGlzn+e16mi6drlLQXkjtzNRIuHTMHpSrQ9Sg50qlpLjiiH4Ldp31jmHLNoAYdEJWabe17lUOA4C6vquxSaT1c6+95xZyOBOulrVKpvi0kOiVv0PFkqe5g1u2/3iG973G/SL5SthUNob1u+nwDJ7p1dhmImvNOp3LzDDavjfcO49rIMOgttq86fWxBaXmm5i9iwEha+UoxE9YBN6hDlrXyPaCUZBbxvghAfl/B4fjTsoQ7QUN6xJkSC+HWohGVDiqg8+D2wlHMOg1EeY9vGBzXnCvVPnOhyNNeXWLMfkKqrZbAN+/8huOutge+LfZKMsBmhvWWtok7TAOXoY/MqcmEPsD2n0S4Iw6fNPVWjHcrn1MGddh5hmnsK/jzk2Xg++/s+jgYOuhA02Y9LsyLxRdXbyUbjjbF/iWQwQYT6TgwyllXRJ3zkwliO9Cib09N/6g/ X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(61668805478150)(9452136761055)(767451399110); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001)(6055026)(6041248)(20161123558021)(20161123555025)(20161123562025)(20161123564025)(20161123560025)(6072148);SRVR:CY4PR12MB1301;BCL:0;PCL:0;RULEID:;SRVR:CY4PR12MB1301; X-Microsoft-Exchange-Diagnostics: 1;CY4PR12MB1301;4:c7b+eQSp2kzpPI6NXbZBqFS7gMO7iUsPASqFCmSy4yC8BIYSDhGfaPww3UoNaxQHDPSDpFb7E9Qk7qXzXqDLD3s/56S2yUQHuRq3N0hWO27ark93i4MIWD3qBfhelapdRhzyjGzkDsskZgPYovyOoYQtGxYjpk1sT0qrQX714Nb34vS5j6wdsS/2RIQPXU904rdkxJu1XuYnJpTfk7nDehXyXOtuXZFhUAgCZooTLJfsSpCTvJe8wQHZJydoUhd60VBuwYxGPgA3fS1ECil1fxPcUl5hE8HfpJjNKhJBfRK3weeuYSwJ4+MhVL8HZuo/snqyzOM3NP9r0TfXQeJh5qP1I4NmT2IIi1ASwuzsyLw+M8QfsotlqtmnJc59y+iMwKPytg1YrgeR81IAFgbVzUv8bVYBwl7YP2UFda3uBjln3QIQij6R8NukgYZ3vsKMlgwEYslztAPuifWhfD5basFmGufYXsM8KbTsGfZfiGapMKcnlkW0wQ5MJpFLMEdE9pS7tm2/8MdumZMMzNN3OfLiJk3dyUIg7/fdL0SDJ7+Js1t+XfrgcizAMUJRLaVBS3Wq4/20KH2sHN0GD/MF4PjZyHVi/Pa3F9YaLYdASaewtpFGf7vQH64k/Pxrv+ZJMGG8xOSvafOEmnYArPAwZAli1bk7LPFRACyflUGDW1zcLYv0UW68W4mvJ/nfRIFbHho1uTXN3E0XMpGLDUj1tnRjjDQny3S7cbfF7HCxv7I= X-Forefront-PRVS: 0200DDA8BE X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(7916002)(39840400002)(39860400002)(39450400003)(39850400002)(39410400002)(51914003)(377454003)(24454002)(199003)(189002)(81166006)(33646002)(36756003)(65806001)(47776003)(92566002)(65956001)(107886002)(68736007)(97736004)(189998001)(5001770100001)(25786008)(8676002)(50466002)(4001350100001)(83506001)(6486002)(53936002)(81156014)(105586002)(7736002)(64126003)(93886004)(106356001)(2906002)(86362001)(23746002)(305945005)(5660300001)(2870700001)(101416001)(42186005)(2201001)(65826007)(31696002)(6666003)(6116002)(54356999)(50986999)(38730400001)(31686004)(2950100002)(1706002)(76176999)(229853002);DIR:OUT;SFP:1101;SCL:1;SRVR:CY4PR12MB1301;H:[IPv6:2a02:908:1251:7981:5d6a:1040:c41c:5a5];FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;CY4PR12MB1301;23:7NXVxk/0SRSguRZ+wh3H5OQBa9sKmyogDj9Fs?= =?Windows-1252?Q?9rakR2G5Bp9+RzyEvLASGlcbzlIFa5Ocj0C3TmypsCrcFuzJ/Fs5vj/W?= =?Windows-1252?Q?TqCL2ytS3a8sDDkgqoqkswUxRcli5ce1GoPsxK/xWDlHGppXxF/A7h8o?= =?Windows-1252?Q?dwswvQsJ+5zFxqS8PW2rX3SYECGP8E7ymm1xujM1bu5yhb6KtUbHM5UK?= =?Windows-1252?Q?YcS8GGX2lOqk8SVD1F8wllnv6UDWcJ+zhar0YkRePfjqFg0P9gg8Rh1Z?= =?Windows-1252?Q?KcuVJap5PNs1ArpIIeoSv4Qz0hKjRa2lFBlRm23RgrwPYUWaZPPzvtGB?= =?Windows-1252?Q?UoqoMYvswQcIrPkNycQk653zaCZKH1bJxwNblkEv3A+XSFJEp+nedqcv?= =?Windows-1252?Q?CBHkap7DeqUW2sajvMkCNM19S9gScB3bZ413XRqsFwCW2b/YS9uRLVP0?= =?Windows-1252?Q?akshZQwsvgrxlpbDQEjnycP16NplfztyTC0R7L/CWHjgjFXt2Nsv96J5?= =?Windows-1252?Q?gq5/+W4LcrGSDeJt52YArgBPc0uyfYS9ABnZFWn0JNnvPmMdZM65MFXf?= =?Windows-1252?Q?BYHGq6VKA0C++hZZ0P4h0rB4wtMNWnc9OU7ye2lxK1TCNZDZooh/ouB2?= =?Windows-1252?Q?xrHZinupe52KlBY6yBMgi2Y92Ml9SOM7x/D6/yNcTRQO/5xhQBbURIgB?= =?Windows-1252?Q?AhkSj2y5UC7W89Gu71TPWfGxIwK5/N+K8p05Q6J7cgjECNmotRh4+dDU?= =?Windows-1252?Q?2uZHYeMI/OLbgmupQl3ewmU72D1rnaIrRQ2odqGMn4lrEjT7qVxnnKd1?= =?Windows-1252?Q?+BuDuYOVqH00RrvN0Y3UOYoHYXvTye0nqJ5gBOG1WCnT46iEP8Qo68AJ?= =?Windows-1252?Q?YwzQ9A6WbyYzOPDj0m0rOv8+O7A7QP3O3CMcK8Iq0sruO7pvTkvzBVTP?= =?Windows-1252?Q?gm7QAGzMeirBNJdXMSTjZSnPCerv0s9Hw8Ime25Zc3qrg++Z8G+khO73?= =?Windows-1252?Q?W74loqhGxcj1wY0thvhGun4dTuOB5whRnmupc3vTUKuSzTaxAkg26XEF?= =?Windows-1252?Q?337C69Msvd+91VkKa4zbbF3VCht1nthyqb01cPrJpitUunx191n75UHa?= =?Windows-1252?Q?GxnNRkVbRcfdv9xqENdRumYjdHMjyZSC6XZrzlIEk4DjqOAN1NU5ynU8?= =?Windows-1252?Q?U7Wtsd0JxzrKF9WcsrSCgVFFizd4cHGdvbB16oGnLKesIX8Xbe1OqOOj?= =?Windows-1252?Q?Yq+lgYCAk3+7lfwBmYAR7b1GDGqgNMZW85blv8EJws5o1rlw+ddp14j5?= =?Windows-1252?Q?CFhQMMgBR8ukG95a1xHG73A9H7Vprq0fF5NN4Fwthvh6bf0Q5uTfurYL?= =?Windows-1252?Q?qJe72lx0dE5V2eTuEkETn0jP3YqKg/69AX1yYlPN2CmhfX6HShXZBnsM?= =?Windows-1252?Q?ECUH4XnxAEB4otQTKxlrNJZRhqENQXsFI1Ow4n/6+fUIFaGOcCmKbQpx?= =?Windows-1252?Q?qoYZoNtX+kywy8fiQloPKCvFbgWt7vpsznFT0bXCFX2TnaT7D7FTJ0Q9?= =?Windows-1252?Q?WB+UgjO55KI3GI=3D?= X-Microsoft-Exchange-Diagnostics: 1;CY4PR12MB1301;6:mTfck4nQgLHWX5PzGCEIYxqVNUs2dxeFGcY3XNDldiFX/YskL+8gBrpCLL4AVznxIJsxv2ZCDMAnvL4Db+q1AInP4AQ4YW25l58VzZXer7dH6tAX8ZI1ptzVUxwZVot4b/XTdAd2AFAkiluilwYdk4z4DujUxVcT/9/D2KqROW3i0H/duiKzljG4uHAEK66bH/QnjaJNcBXCW+snxD4mSEftHs5b6QTFsTLUwZwovM3rCTjXPWl0+7+f+quRgNbjud4or67S3SdgsKxI2RRCIW5e8HGgzVtzHryBXuiKg3Z2QJzYTkxsfq0atwWzpcpZTf9sjOZ9R6SCsZ/3w8fvU7VSlO02x5ckRm2YU0mBjdnolUviUS/wivTjhNVCw5Mk3tP7HTbsG5ZrTjP6b/oRGS7eSxV8dUj+HwhnXJM5jJpHpZXVXWUYmMsrrTEb8ChWA3Gfjqi52av/MjeQta4R4g==;5:kyUoktx9djiENK68IJlA2P/t8bsB66vgR3F2Rq046+EoKm3V5tCR16FLCgtuUQPo+X7JY7rXzCaW1qKuwY7ycbEXnDGJIl5FSJLtP3i/Pc8981VYYqw/rRwLENxJV9crk5E1hPypC0Bs52T5lAnBVw==;24:mLxMp5NIAKEkFza8nYYVg37NIMZViUQAybGHwKlCZplpu9spbzEVfmRgF65j9mUMrb4CDqm6OpU9DewK5faFKIBbBFmAD2Q9fUB4PjohAB8= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;CY4PR12MB1301;7:/MOUR+cEbOVQ9SrqERrWA6llERAdeoUlVaVB31nOyxU/1xb8RcN56w8+FBXg7huh57NV2KUhPU88MekS98HnfMVJgBTVwTwdLOFNjUkmItcLI9a87997+O0n7dXre/yQjQjuCE75jKIYcNTh/mdr+UB/ExQq9Xj4HBzw9OhoAl90K0EEpp/YUW4Wd0FGBKMEOudcMmQKn4VOQ6P9/1yvu0MW2e9vv+J/HZ/iB+fWDchMZYBAQz9frJpqmbuQxtzC+6QnPW8Cz/JZJ5ibr/eqYQGyEkCV3rjt15HpdGjnXcs6eACipMXQxL9kCWvGDDdqkvJiJI4NhQB9wDvcu/n8GAOfQfvUC9EPrkCWDhHdiHSBEGQp5kshb0eNLpn+Yro7boLYRUVYVgZ6WnkCIeZOgH5dsOJZE7KgZjfhIWOsc0/Jsv/z6m7DfWcwRkIDnr7YCuo4VSRpppm6LPZ32gV0jg==;20:WinlbYyAYGY0hGf6lmGZQdTIDIGYJJXVG5rFyHFRbcB+AFOBR6kr/3Vt4HWvMucu0bb0mIIT1BAXTygfG9UPiOcL6OmnK5pO/FfK5u/PZObCm1T5gCz2P7gPGn3n89aNeAM/VIs7/CSuI2pfXVmVQYI66TCA9w/XzGFAb4ruz3QkVUl0cUAtoucaH3jmrlEt0zJ2VzJyI6PRMwIPqthZ5uFcfxNnillGCaL1TceFAyO5Bd20AIAI12mggIYUefAq X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Jan 2017 08:23:01.3653 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR12MB1301 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am 27.01.2017 um 08:30 schrieb Daniel Vetter: > On Fri, Jan 27, 2017 at 07:23:58AM +0100, Thomas Hellstrom wrote: >> On 01/27/2017 03:29 AM, Michel Dänzer wrote: >>> 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 >>>>>>> >>>>>>> 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: stable@vger.kernel.org >>>>>>> Signed-off-by: Michel Dänzer >>>>>> Reviewed-by: Thomas Hellstrom >>>>> Reviewed-by: Christian König . >>>> Reviewed-by: Sinclair Yeh >>> Thanks for the reviews! Via which tree should we merge this? >>> >>> >> I don't maintain a TTM tree any longer. Let's check with Daniel if he >> can merge it through drm-misc. > I'm trying very hard not to get volunteered for ttm maintainer :-) Yeah, ok I volunteer. Wanted to take that over for a while anyway. > Nominally Alex&Christian have drm-misc commit rights, but they haven't > used them yet. But I think merging through drm-misc would make sense, > there's regular pull request trains for both -next and -fixes. Completely agree on merging it through drm-misc. Going to give my push rights a try today. > Or merge through the amd tree with Dave's ack, but I'd really like to get amd folks > into the drm-misc group ... I've got a few more already reviewed TTM changes which are currently waiting to be pushed upstream where amdgpu has dependencies on. Going to sync with Alex so he sends his pull requests with those changes to Dave after the merge of the depending changes. If you want to object that just merging through the AMD tree would be simpler, I agree but I actually want to do this exercise at least once :) Regards, Christian. > -Daniel