From patchwork Wed Jul 2 16:36:51 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Reese, Armin C" X-Patchwork-Id: 4466861 Return-Path: X-Original-To: patchwork-intel-gfx@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 484109F36A for ; Wed, 2 Jul 2014 16:36:58 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 60B85201FB for ; Wed, 2 Jul 2014 16:36:57 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 7814620145 for ; Wed, 2 Jul 2014 16:36:56 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id E5CB56E648; Wed, 2 Jul 2014 09:36:55 -0700 (PDT) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTP id 3E3416E648 for ; Wed, 2 Jul 2014 09:36:54 -0700 (PDT) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP; 02 Jul 2014 09:31:09 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.01,589,1400050800"; d="scan'208";a="556495750" Received: from fmsmsx108.amr.corp.intel.com ([10.19.9.228]) by fmsmga001.fm.intel.com with ESMTP; 02 Jul 2014 09:36:53 -0700 Received: from fmsmsx117.amr.corp.intel.com (10.18.116.17) by FMSMSX108.amr.corp.intel.com (10.19.9.228) with Microsoft SMTP Server (TLS) id 14.3.123.3; Wed, 2 Jul 2014 09:36:52 -0700 Received: from fmsmsx107.amr.corp.intel.com ([169.254.9.166]) by fmsmsx117.amr.corp.intel.com ([169.254.3.56]) with mapi id 14.03.0123.003; Wed, 2 Jul 2014 09:36:53 -0700 From: "Reese, Armin C" To: "intel-gfx@lists.freedesktop.org" , "Vetter, Daniel" Thread-Topic: [PATCH] igt/gem_partial_pwrite_pread: Add set-cache subtest to validate JIRA VIZ-3721 Thread-Index: AQHPhoPteRlcuvL8EEyB/Mg0Gjrf4ZuNGY0A Date: Wed, 2 Jul 2014 16:36:51 +0000 Message-ID: <07EEC142D30DAC40BA4DD9C0F9EE5109405B7767@FMSMSX107.amr.corp.intel.com> References: <1402607917-12847-1-git-send-email-armin.c.reese@intel.com> In-Reply-To: <1402607917-12847-1-git-send-email-armin.c.reese@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.1.200.106] MIME-Version: 1.0 Cc: "Barnes, Jesse" Subject: Re: [Intel-gfx] [PATCH] igt/gem_partial_pwrite_pread: Add set-cache subtest to validate JIRA VIZ-3721 X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Hi Daniel, Is it possible to get this patch on the review board? I've been on vacation a couple of weeks and haven't been around to push it. Thanks, Armin -----Original Message----- From: Reese, Armin C Sent: Thursday, June 12, 2014 2:19 PM To: intel-gfx@lists.freedesktop.org Cc: Reese, Armin C Subject: [PATCH] igt/gem_partial_pwrite_pread: Add set-cache subtest to validate JIRA VIZ-3721 From: Armin Reese This subtest forces the driver down the code path in i915_gem_object_set_cache_level() which unbinds VMAs and triggers the kernel panic described in VIZ-3721. This test will pass if the bug fix is in place. Bugzilla: https://bugs.fredesktop.org/show_bug.cgi?id=76384 Signed-off-by: Armin Reese --- tests/gem_partial_pwrite_pread.c | 11 +++++++++++ 1 file changed, 11 insertions(+) -- 1.9.1 diff --git a/tests/gem_partial_pwrite_pread.c b/tests/gem_partial_pwrite_pread.c index 7945ba7..3e3ecc9 100644 --- a/tests/gem_partial_pwrite_pread.c +++ b/tests/gem_partial_pwrite_pread.c @@ -131,6 +131,14 @@ static void test_partial_reads(void) } +static void test_set_cache(void) +{ + igt_info("checking set-cache\n"); + + blt_bo_fill(staging_bo, scratch_bo, 0); + +} + static void test_partial_writes(void) { int i, j; @@ -244,6 +252,9 @@ static void do_tests(int cache_level, const char *suffix) gem_set_caching(fd, scratch_bo->handle, cache_level); } + igt_subtest_f("set-cache%s", suffix) + test_set_cache(); + igt_subtest_f("reads%s", suffix) test_partial_reads();