From: Kenneth Graunke <kenneth@whitecape.org>
To: Rodrigo Vivi <rodrigo.vivi@gmail.com>
Cc: mesa-dev@lists.freedesktop.org, intel-gfx@lists.freedesktop.org
Subject: Re: [Mesa-dev] [PATCH 2/2 mesa] i965: Adding more reserved PCI IDs for Haswell.
Date: Mon, 13 May 2013 17:06:10 -0700 [thread overview]
Message-ID: <51917FF2.4080208@whitecape.org> (raw)
In-Reply-To: <1368478419-22402-2-git-send-email-rodrigo.vivi@gmail.com>
On 05/13/2013 01:53 PM, Rodrigo Vivi wrote:
> At DDX commit Chris mentioned the tendency we have of finding out more
> PCI IDs only when users report. So Let's add all new reserved Haswell IDs.
>
> References: http://bugs.freedesktop.org/show_bug.cgi?id=63701
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
Paulo sent me an email that suggested that some of the 0x___E PCI IDs
might actually have a different EU count than the others. But I have no
idea whether that's true or not...
If it is, though, then we'd likely just hang the GPU on such
systems...which is probably not any better than, say, falling back to
llvmpipe.
I'm not sure what the right thing to do is here.
> ---
> include/pci_ids/i965_pci_ids.h | 24 +++++++++++++
> src/mesa/drivers/dri/intel/intel_chipset.h | 54 ++++++++++++++++++++++++++++--
> src/mesa/drivers/dri/intel/intel_context.c | 26 ++++++++++++++
> 3 files changed, 101 insertions(+), 3 deletions(-)
>
> diff --git a/include/pci_ids/i965_pci_ids.h b/include/pci_ids/i965_pci_ids.h
> index 3e9765c..808eb4e 100644
> --- a/include/pci_ids/i965_pci_ids.h
> +++ b/include/pci_ids/i965_pci_ids.h
> @@ -35,6 +35,12 @@ CHIPSET(0x0426, HASWELL_M_GT3, hsw_gt3)
> CHIPSET(0x040A, HASWELL_S_GT1, hsw_gt1)
> CHIPSET(0x041A, HASWELL_S_GT2, hsw_gt2)
> CHIPSET(0x042A, HASWELL_S_GT3, hsw_gt3)
> +CHIPSET(0x040B, HASWELL_B_GT1, hsw_gt1)
> +CHIPSET(0x041B, HASWELL_B_GT2, hsw_gt2)
> +CHIPSET(0x042B, HASWELL_B_GT3, hsw_gt3)
> +CHIPSET(0x040E, HASWELL_E_GT1, hsw_gt1)
> +CHIPSET(0x041E, HASWELL_E_GT2, hsw_gt2)
> +CHIPSET(0x042E, HASWELL_E_GT3, hsw_gt3)
> CHIPSET(0x0C02, HASWELL_SDV_GT1, hsw_gt1)
> CHIPSET(0x0C12, HASWELL_SDV_GT2, hsw_gt2)
> CHIPSET(0x0C22, HASWELL_SDV_GT3, hsw_gt3)
> @@ -44,6 +50,12 @@ CHIPSET(0x0C26, HASWELL_SDV_M_GT3, hsw_gt3)
> CHIPSET(0x0C0A, HASWELL_SDV_S_GT1, hsw_gt1)
> CHIPSET(0x0C1A, HASWELL_SDV_S_GT2, hsw_gt2)
> CHIPSET(0x0C2A, HASWELL_SDV_S_GT3, hsw_gt3)
> +CHIPSET(0x0C0B, HASWELL_SDV_B_GT1, hsw_gt1)
> +CHIPSET(0x0C1B, HASWELL_SDV_B_GT2, hsw_gt2)
> +CHIPSET(0x0C2B, HASWELL_SDV_B_GT3, hsw_gt3)
> +CHIPSET(0x0C0E, HASWELL_SDV_E_GT1, hsw_gt1)
> +CHIPSET(0x0C1E, HASWELL_SDV_E_GT2, hsw_gt2)
> +CHIPSET(0x0C2E, HASWELL_SDV_E_GT3, hsw_gt3)
> CHIPSET(0x0A02, HASWELL_ULT_GT1, hsw_gt1)
> CHIPSET(0x0A12, HASWELL_ULT_GT2, hsw_gt2)
> CHIPSET(0x0A22, HASWELL_ULT_GT3, hsw_gt3)
> @@ -53,6 +65,12 @@ CHIPSET(0x0A26, HASWELL_ULT_M_GT3, hsw_gt3)
> CHIPSET(0x0A0A, HASWELL_ULT_S_GT1, hsw_gt1)
> CHIPSET(0x0A1A, HASWELL_ULT_S_GT2, hsw_gt2)
> CHIPSET(0x0A2A, HASWELL_ULT_S_GT3, hsw_gt3)
> +CHIPSET(0x0A0B, HASWELL_ULT_B_GT1, hsw_gt1)
> +CHIPSET(0x0A1B, HASWELL_ULT_B_GT2, hsw_gt2)
> +CHIPSET(0x0A2B, HASWELL_ULT_B_GT3, hsw_gt3)
> +CHIPSET(0x0A0E, HASWELL_ULT_E_GT1, hsw_gt1)
> +CHIPSET(0x0A1E, HASWELL_ULT_E_GT2, hsw_gt2)
> +CHIPSET(0x0A2E, HASWELL_ULT_E_GT3, hsw_gt3)
> CHIPSET(0x0D02, HASWELL_CRW_GT1, hsw_gt1)
> CHIPSET(0x0D12, HASWELL_CRW_GT2, hsw_gt2)
> CHIPSET(0x0D22, HASWELL_CRW_GT3, hsw_gt3)
> @@ -62,6 +80,12 @@ CHIPSET(0x0D26, HASWELL_CRW_M_GT3, hsw_gt3)
> CHIPSET(0x0D0A, HASWELL_CRW_S_GT1, hsw_gt1)
> CHIPSET(0x0D1A, HASWELL_CRW_S_GT2, hsw_gt2)
> CHIPSET(0x0D2A, HASWELL_CRW_S_GT3, hsw_gt3)
> +CHIPSET(0x0D0B, HASWELL_CRW_B_GT1, hsw_gt1)
> +CHIPSET(0x0D1B, HASWELL_CRW_B_GT2, hsw_gt2)
> +CHIPSET(0x0D2B, HASWELL_CRW_B_GT3, hsw_gt3)
> +CHIPSET(0x0D0E, HASWELL_CRW_E_GT1, hsw_gt1)
> +CHIPSET(0x0D1E, HASWELL_CRW_E_GT2, hsw_gt2)
> +CHIPSET(0x0D2E, HASWELL_CRW_E_GT3, hsw_gt3)
> CHIPSET(0x0F31, BAYTRAIL_M_1, byt)
> CHIPSET(0x0F32, BAYTRAIL_M_2, byt)
> CHIPSET(0x0F33, BAYTRAIL_M_3, byt)
> diff --git a/src/mesa/drivers/dri/intel/intel_chipset.h b/src/mesa/drivers/dri/intel/intel_chipset.h
> index ee735bb..1e98cf4 100644
> --- a/src/mesa/drivers/dri/intel/intel_chipset.h
> +++ b/src/mesa/drivers/dri/intel/intel_chipset.h
> @@ -102,6 +102,12 @@
> #define PCI_CHIP_HASWELL_S_GT1 0x040A /* Server */
> #define PCI_CHIP_HASWELL_S_GT2 0x041A
> #define PCI_CHIP_HASWELL_S_GT3 0x042A
> +#define PCI_CHIP_HASWELL_B_GT1 0x040B /* Reserved */
> +#define PCI_CHIP_HASWELL_B_GT2 0x041B
> +#define PCI_CHIP_HASWELL_B_GT3 0x042B
> +#define PCI_CHIP_HASWELL_E_GT1 0x040E /* Reserved */
> +#define PCI_CHIP_HASWELL_E_GT2 0x041E
> +#define PCI_CHIP_HASWELL_E_GT3 0x042E
> #define PCI_CHIP_HASWELL_SDV_GT1 0x0C02 /* Desktop */
> #define PCI_CHIP_HASWELL_SDV_GT2 0x0C12
> #define PCI_CHIP_HASWELL_SDV_GT3 0x0C22
> @@ -111,6 +117,12 @@
> #define PCI_CHIP_HASWELL_SDV_S_GT1 0x0C0A /* Server */
> #define PCI_CHIP_HASWELL_SDV_S_GT2 0x0C1A
> #define PCI_CHIP_HASWELL_SDV_S_GT3 0x0C2A
> +#define PCI_CHIP_HASWELL_SDV_B_GT1 0x0C0B /* Reserved */
> +#define PCI_CHIP_HASWELL_SDV_B_GT2 0x0C1B
> +#define PCI_CHIP_HASWELL_SDV_B_GT3 0x0C2B
> +#define PCI_CHIP_HASWELL_SDV_E_GT1 0x0C0E /* Reserved */
> +#define PCI_CHIP_HASWELL_SDV_E_GT2 0x0C1E
> +#define PCI_CHIP_HASWELL_SDV_E_GT3 0x0C2E
> #define PCI_CHIP_HASWELL_ULT_GT1 0x0A02 /* Desktop */
> #define PCI_CHIP_HASWELL_ULT_GT2 0x0A12
> #define PCI_CHIP_HASWELL_ULT_GT3 0x0A22
> @@ -120,6 +132,12 @@
> #define PCI_CHIP_HASWELL_ULT_S_GT1 0x0A0A /* Server */
> #define PCI_CHIP_HASWELL_ULT_S_GT2 0x0A1A
> #define PCI_CHIP_HASWELL_ULT_S_GT3 0x0A2A
> +#define PCI_CHIP_HASWELL_ULT_B_GT1 0x0A0B /* Reserved */
> +#define PCI_CHIP_HASWELL_ULT_B_GT2 0x0A1B
> +#define PCI_CHIP_HASWELL_ULT_B_GT3 0x0A2B
> +#define PCI_CHIP_HASWELL_ULT_E_GT1 0x0A0E /* Reserved */
> +#define PCI_CHIP_HASWELL_ULT_E_GT2 0x0A1E
> +#define PCI_CHIP_HASWELL_ULT_E_GT3 0x0A2E
> #define PCI_CHIP_HASWELL_CRW_GT1 0x0D02 /* Desktop */
> #define PCI_CHIP_HASWELL_CRW_GT2 0x0D12
> #define PCI_CHIP_HASWELL_CRW_GT3 0x0D22
> @@ -129,6 +147,12 @@
> #define PCI_CHIP_HASWELL_CRW_S_GT1 0x0D0A /* Server */
> #define PCI_CHIP_HASWELL_CRW_S_GT2 0x0D1A
> #define PCI_CHIP_HASWELL_CRW_S_GT3 0x0D2A
> +#define PCI_CHIP_HASWELL_CRW_B_GT1 0x0D0B /* Reserved */
> +#define PCI_CHIP_HASWELL_CRW_B_GT2 0x0D1B
> +#define PCI_CHIP_HASWELL_CRW_B_GT3 0x0D2B
> +#define PCI_CHIP_HASWELL_CRW_E_GT1 0x0D0E /* Reserved */
> +#define PCI_CHIP_HASWELL_CRW_E_GT2 0x0D1E
> +#define PCI_CHIP_HASWELL_CRW_E_GT3 0x0D2E
>
> #define IS_MOBILE(devid) (devid == PCI_CHIP_I855_GM || \
> devid == PCI_CHIP_I915_GM || \
> @@ -209,39 +233,63 @@
> #define IS_HSW_GT1(devid) (devid == PCI_CHIP_HASWELL_GT1 || \
> devid == PCI_CHIP_HASWELL_M_GT1 || \
> devid == PCI_CHIP_HASWELL_S_GT1 || \
> + devid == PCI_CHIP_HASWELL_B_GT1 || \
> + devid == PCI_CHIP_HASWELL_E_GT1 || \
> devid == PCI_CHIP_HASWELL_SDV_GT1 || \
> devid == PCI_CHIP_HASWELL_SDV_M_GT1 || \
> devid == PCI_CHIP_HASWELL_SDV_S_GT1 || \
> + devid == PCI_CHIP_HASWELL_SDV_B_GT1 || \
> + devid == PCI_CHIP_HASWELL_SDV_E_GT1 || \
> devid == PCI_CHIP_HASWELL_ULT_GT1 || \
> devid == PCI_CHIP_HASWELL_ULT_M_GT1 || \
> devid == PCI_CHIP_HASWELL_ULT_S_GT1 || \
> + devid == PCI_CHIP_HASWELL_ULT_B_GT1 || \
> + devid == PCI_CHIP_HASWELL_ULT_E_GT1 || \
> devid == PCI_CHIP_HASWELL_CRW_GT1 || \
> devid == PCI_CHIP_HASWELL_CRW_M_GT1 || \
> - devid == PCI_CHIP_HASWELL_CRW_S_GT1)
> + devid == PCI_CHIP_HASWELL_CRW_S_GT1 || \
> + devid == PCI_CHIP_HASWELL_CRW_B_GT1 || \
> + devid == PCI_CHIP_HASWELL_CRW_E_GT1)
> #define IS_HSW_GT2(devid) (devid == PCI_CHIP_HASWELL_GT2 || \
> devid == PCI_CHIP_HASWELL_M_GT2 || \
> devid == PCI_CHIP_HASWELL_S_GT2 || \
> + devid == PCI_CHIP_HASWELL_B_GT2 || \
> + devid == PCI_CHIP_HASWELL_E_GT2 || \
> devid == PCI_CHIP_HASWELL_SDV_GT2 || \
> devid == PCI_CHIP_HASWELL_SDV_M_GT2 || \
> devid == PCI_CHIP_HASWELL_SDV_S_GT2 || \
> + devid == PCI_CHIP_HASWELL_SDV_B_GT2 || \
> + devid == PCI_CHIP_HASWELL_SDV_E_GT2 || \
> devid == PCI_CHIP_HASWELL_ULT_GT2 || \
> devid == PCI_CHIP_HASWELL_ULT_M_GT2 || \
> devid == PCI_CHIP_HASWELL_ULT_S_GT2 || \
> + devid == PCI_CHIP_HASWELL_ULT_B_GT2 || \
> + devid == PCI_CHIP_HASWELL_ULT_E_GT2 || \
> devid == PCI_CHIP_HASWELL_CRW_GT2 || \
> devid == PCI_CHIP_HASWELL_CRW_M_GT2 || \
> - devid == PCI_CHIP_HASWELL_CRW_S_GT2)
> + devid == PCI_CHIP_HASWELL_CRW_S_GT2 || \
> + devid == PCI_CHIP_HASWELL_CRW_B_GT2 || \
> + devid == PCI_CHIP_HASWELL_CRW_E_GT2)
> #define IS_HSW_GT3(devid) (devid == PCI_CHIP_HASWELL_GT3 || \
> devid == PCI_CHIP_HASWELL_M_GT3 || \
> devid == PCI_CHIP_HASWELL_S_GT3 || \
> + devid == PCI_CHIP_HASWELL_B_GT3 || \
> + devid == PCI_CHIP_HASWELL_E_GT3 || \
> devid == PCI_CHIP_HASWELL_SDV_GT3 || \
> devid == PCI_CHIP_HASWELL_SDV_M_GT3 || \
> devid == PCI_CHIP_HASWELL_SDV_S_GT3 || \
> + devid == PCI_CHIP_HASWELL_SDV_B_GT3 || \
> + devid == PCI_CHIP_HASWELL_SDV_E_GT3 || \
> devid == PCI_CHIP_HASWELL_ULT_GT3 || \
> devid == PCI_CHIP_HASWELL_ULT_M_GT3 || \
> devid == PCI_CHIP_HASWELL_ULT_S_GT3 || \
> + devid == PCI_CHIP_HASWELL_ULT_B_GT3 || \
> + devid == PCI_CHIP_HASWELL_ULT_E_GT3 || \
> devid == PCI_CHIP_HASWELL_CRW_GT3 || \
> devid == PCI_CHIP_HASWELL_CRW_M_GT3 || \
> - devid == PCI_CHIP_HASWELL_CRW_S_GT3)
> + devid == PCI_CHIP_HASWELL_CRW_S_GT3 || \
> + devid == PCI_CHIP_HASWELL_CRW_B_GT3 || \
> + devid == PCI_CHIP_HASWELL_CRW_E_GT3)
>
> #define IS_HASWELL(devid) (IS_HSW_GT1(devid) || \
> IS_HSW_GT2(devid) || \
> diff --git a/src/mesa/drivers/dri/intel/intel_context.c b/src/mesa/drivers/dri/intel/intel_context.c
> index 88cc247..4eb6cdd 100644
> --- a/src/mesa/drivers/dri/intel/intel_context.c
> +++ b/src/mesa/drivers/dri/intel/intel_context.c
> @@ -235,6 +235,32 @@ intelGetString(struct gl_context * ctx, GLenum name)
> case PCI_CHIP_HASWELL_CRW_S_GT3:
> chipset = "Intel(R) Haswell Server";
> break;
> + case PCI_CHIP_HASWELL_B_GT1:
> + case PCI_CHIP_HASWELL_B_GT2:
> + case PCI_CHIP_HASWELL_B_GT3:
> + case PCI_CHIP_HASWELL_SDV_B_GT1:
> + case PCI_CHIP_HASWELL_SDV_B_GT2:
> + case PCI_CHIP_HASWELL_SDV_B_GT3:
> + case PCI_CHIP_HASWELL_ULT_B_GT1:
> + case PCI_CHIP_HASWELL_ULT_B_GT2:
> + case PCI_CHIP_HASWELL_ULT_B_GT3:
> + case PCI_CHIP_HASWELL_CRW_B_GT1:
> + case PCI_CHIP_HASWELL_CRW_B_GT2:
> + case PCI_CHIP_HASWELL_CRW_B_GT3:
> + case PCI_CHIP_HASWELL_E_GT1:
> + case PCI_CHIP_HASWELL_E_GT2:
> + case PCI_CHIP_HASWELL_E_GT3:
> + case PCI_CHIP_HASWELL_SDV_E_GT1:
> + case PCI_CHIP_HASWELL_SDV_E_GT2:
> + case PCI_CHIP_HASWELL_SDV_E_GT3:
> + case PCI_CHIP_HASWELL_ULT_E_GT1:
> + case PCI_CHIP_HASWELL_ULT_E_GT2:
> + case PCI_CHIP_HASWELL_ULT_E_GT3:
> + case PCI_CHIP_HASWELL_CRW_E_GT1:
> + case PCI_CHIP_HASWELL_CRW_E_GT2:
> + case PCI_CHIP_HASWELL_CRW_E_GT3:
> + chipset = "Intel(R) Haswell";
> + break;
> default:
> chipset = "Unknown Intel Chipset";
> break;
>
next prev parent reply other threads:[~2013-05-14 0:06 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-13 20:53 [PATCH 1/2 mesa] i965: Add missing Haswell GT3 Desktop to IS_HSW_GT3 check Rodrigo Vivi
2013-05-13 20:53 ` [PATCH 2/2 mesa] i965: Adding more reserved PCI IDs for Haswell Rodrigo Vivi
2013-05-14 0:06 ` Kenneth Graunke [this message]
2013-05-14 0:06 ` [Mesa-dev] [PATCH 1/2 mesa] i965: Add missing Haswell GT3 Desktop to IS_HSW_GT3 check Kenneth Graunke
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51917FF2.4080208@whitecape.org \
--to=kenneth@whitecape.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=mesa-dev@lists.freedesktop.org \
--cc=rodrigo.vivi@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox