From mboxrd@z Thu Jan  1 00:00:00 1970
From: Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [PATCH v6 6/8] vfio/pci: Intel graphics legacy mode assignment
Date: Wed, 18 May 2016 16:21:58 +0200
Message-ID: <1463581318.30045.66.camel@redhat.com>
References: <20160517201613.6175.45578.stgit@gimli.home>
	 <20160517201955.6175.97671.stgit@gimli.home>
Mime-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8BIT
Cc: qemu-devel@nongnu.org, allen.m.kay@intel.com, kvm@vger.kernel.org,
	"Zhao, Xinda" <xinda.zhao@intel.com>,
	Kevin Tian <kevin.tian@intel.com>
To: Alex Williamson <alex.williamson@redhat.com>
Return-path: <kvm-owner@vger.kernel.org>
Received: from mx1.redhat.com ([209.132.183.28]:55715 "EHLO mx1.redhat.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S932107AbcEROWB convert rfc822-to-8bit (ORCPT
	<rfc822;kvm@vger.kernel.org>); Wed, 18 May 2016 10:22:01 -0400
In-Reply-To: <20160517201955.6175.97671.stgit@gimli.home>
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

On Di, 2016-05-17 at 14:19 -0600, Alex Williamson wrote:
> +static int igd_gen(VFIOPCIDevice *vdev)
> +{
> +    if ((vdev->device_id & 0xfff) == 0xa84) {
> +        return 8; /* Broxton */
> +    }
> +
> +    switch (vdev->device_id & 0xff00) {
> +    /* Old, untested, unavailable, unknown */
> +    case 0x0000:
> +    case 0x2500:
> +    case 0x2700:
> +    case 0x2900:
> +    case 0x2a00:
> +    case 0x2e00:
> +    case 0x3500:
> +    case 0xa000:
> +        return -1;
> +    /* SandyBridge, IvyBridge, ValleyView, Haswell */
> +    case 0x0100:
> +    case 0x0400:
> +    case 0x0a00:
> +    case 0x0c00:
> +    case 0x0d00:
> +    case 0x0f00:
> +        return 6;
> +    /* BroadWell, CherryView, SkyLake, KabyLake */
> +    case 0x1600:
> +    case 0x1900:
> +    case 0x2200:
> +    case 0x5900:
> +        return 8;
> +    }
> +
> +    return 8; /* Assume newer is compatible */
> +}
> +

This link:
https://github.com/01org/Igvtg-qemu/commit/ea32e6769004d6eb98d2dbd859d81bf1885c6ad2#diff-9d4d99332b83a7de33cbeed489d60448R920

happened to land in my inbox these days.  It provides a bunch of
functions called is_$codename() and intel_gen_version().  Looks more
complete to me.  Maybe we should pick up them and find a place in the
qemu source tree where both vfio and intel-vgpu (and maybe xen device
assignment too) can share those functions?

cheers,
  Gerd