From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ben Widawsky <ben@bwidawsk.net>
Subject: Re: [PATCH 10/16] intel_l3_parity: Assert all GEN7+
 support
Date: Tue, 17 Sep 2013 16:52:59 -0700
Message-ID: <20130917235259.GD6112@bwidawsk.net>
References: <1379050122-12774-1-git-send-email-benjamin.widawsky@intel.com>
	<1379050122-12774-11-git-send-email-benjamin.widawsky@intel.com>
	<45EA1CA55A8B924B8A73C0520E7232CB62FE874B@FMSMSX107.amr.corp.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org>
Received: from mail.bwidawsk.net (bwidawsk.net [166.78.191.112])
	by gabe.freedesktop.org (Postfix) with ESMTP id 6CC64E738F
	for <intel-gfx@lists.freedesktop.org>;
	Tue, 17 Sep 2013 16:53:04 -0700 (PDT)
Content-Disposition: inline
In-Reply-To: <45EA1CA55A8B924B8A73C0520E7232CB62FE874B@FMSMSX107.amr.corp.intel.com>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org
Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org
To: "Bell, Bryan J" <bryan.j.bell@intel.com>
Cc: "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org>, "Venkatesh, Vishnu" <vishnu.venkatesh@intel.com>, "Widawsky,
	Benjamin" <benjamin.widawsky@intel.com>
List-Id: intel-gfx@lists.freedesktop.org

On Mon, Sep 16, 2013 at 06:18:28PM +0000, Bell, Bryan J wrote:
> L3 dynamic parity is not supported on VLV. Please add the check for VLV. 
> 
> I can send you the email thread, if needed. 
> 
> --Thanks
> Bryan

More importantly, we need to fix this in the kernel too. Thanks for
catching this.

> 
> -----Original Message-----
> From: Ben Widawsky [mailto:benjamin.widawsky@intel.com] 
> Sent: Thursday, September 12, 2013 10:29 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Venkatesh, Vishnu; Bell, Bryan J; Widawsky, Benjamin; Ben Widawsky
> Subject: [PATCH 10/16] intel_l3_parity: Assert all GEN7+ support
> 
> Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
> ---
>  tools/intel_l3_parity.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/intel_l3_parity.c b/tools/intel_l3_parity.c index 970dcd6..a3d268b 100644
> --- a/tools/intel_l3_parity.c
> +++ b/tools/intel_l3_parity.c
> @@ -120,7 +120,7 @@ int main(int argc, char *argv[])
>  	assert(ret != -1);
>  
>  	fd = open(path, O_RDWR);
> -	if (fd == -1 && IS_IVYBRIDGE(devid)) {
> +	if (fd == -1 && intel_gen(devid) > 6) {
>  		perror("Opening sysfs");
>  		exit(EXIT_FAILURE);
>  	} else if (fd == -1)
> --
> 1.8.4
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ben Widawsky, Intel Open Source Technology Center