Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751489AbdGRJJ5 (ORCPT ); Tue, 18 Jul 2017 05:09:57 -0400 Received: from hqemgate14.nvidia.com ([216.228.121.143]:8566 "EHLO hqemgate14.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751334AbdGRJJz (ORCPT ); Tue, 18 Jul 2017 05:09:55 -0400 X-PGP-Universal: processed; by hqpgpgate102.nvidia.com on Tue, 18 Jul 2017 02:09:54 -0700 Subject: Re: [PATCH] gpu: host1x: Free the IOMMU domain when there is no device to attach To: Mikko Perttunen , Paul Kocialkowski , , , , Thierry Reding CC: Stephen Warren , Mikko Perttunen References: <20170710193305.5987-1-contact@paulk.fr> <09ecbf74-ac5d-a7f5-c6bd-53f4f84a0de5@kapsi.fi> From: Jon Hunter Message-ID: <8db74bfc-e442-dd32-4b24-1c33f3a82d92@nvidia.com> Date: Tue, 18 Jul 2017 10:09:45 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <09ecbf74-ac5d-a7f5-c6bd-53f4f84a0de5@kapsi.fi> X-Originating-IP: [10.21.132.162] X-ClientProxiedBy: UKMAIL101.nvidia.com (10.26.138.13) To UKMAIL101.nvidia.com (10.26.138.13) Content-Type: text/plain; charset="windows-1252" Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 818 Lines: 22 Hi Mikko, On 11/07/17 07:43, Mikko Perttunen wrote: > Thanks for the patch, didn't consider this case. I really need to get > together some system to automatically test on multiple platforms.. :) We already have the infrastructure in place to do this, however, at the moment we are just making sure the platforms boot. Something like this we should be able to catch, even if we don't explicitly test display/hdmi. I was thinking that we could parse the /dev and /sys file-systems to ensure that devices for a given platform are present after boot. I am not sure if there is a good way to do this, but if you have any thoughts, then I am willing to add more testing. What is interesting about this case, was that there were no specific error messages that indicated this had broken AFAICT. Cheers Jon -- nvpublic