From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by gabe.freedesktop.org (Postfix) with ESMTPS id B840B6E872 for ; Tue, 30 Nov 2021 23:27:27 +0000 (UTC) Date: Tue, 30 Nov 2021 15:27:14 -0800 Message-ID: <878rx5z0kt.wl-ashutosh.dixit@intel.com> From: "Dixit, Ashutosh" In-Reply-To: <20211128070823.1396652-3-priyanka.dandamudi@intel.com> References: <20211128070823.1396652-1-priyanka.dandamudi@intel.com> <20211128070823.1396652-3-priyanka.dandamudi@intel.com> MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII Subject: Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/i915_hangman: Explicitly test per engine reset vs full GPU reset List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: priyanka.dandamudi@intel.com Cc: igt-dev@lists.freedesktop.org List-ID: On Sat, 27 Nov 2021 23:08:23 -0800, wrote: > > From: John Harrison > > Although the hangman test was ensuring that *some* reset functionality > was enabled, it did not differentiate what kind. So add in the > infrastructure required to choose between per engine reset or full GPU > reset and update the test to use it. Hangman will now check that it > can capture error logs when both forms of reset occur. It will also > ensure that the 'does per engine reset work' tests will only run if > per engine reset is avaiable. This is insufficient description for what is included here. This patch should be split into multiple patches and resubmitted.