From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stable-owner@vger.kernel.org>
Received: from youngberry.canonical.com ([91.189.89.112]:35646 "EHLO
        youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S933386AbeAKM7n (ORCPT
        <rfc822;stable@vger.kernel.org>); Thu, 11 Jan 2018 07:59:43 -0500
Subject: Re: [Intel-gfx] [PATCH v2] x86/gpu: add CFL to early quirks
To: Rodrigo Vivi <rodrigo.vivi@intel.com>,
        Ingo Molnar <mingo@kernel.org>
Cc: David Airlie <airlied@linux.ie>, intel-gfx@lists.freedesktop.org,
        Lucas De Marchi <lucas.demarchi@intel.com>, x86@kernel.org,
        dri-devel@lists.freedesktop.org, "H . Peter Anvin" <hpa@zytor.com>,
        stable@vger.kernel.org, Thomas Gleixner <tglx@linutronix.de>
References: <20171213200425.2954-1-lucas.demarchi@intel.com>
 <20171215163925.harred3ei4ew6gca@gmail.com>
 <20171215211943.enktaerpipdpqykl@intel.com>
From: Timo Aaltonen <tjaalton@ubuntu.com>
Message-ID: <82487014-77f3-5434-e9cf-13ad228024fc@ubuntu.com>
Date: Thu, 11 Jan 2018 14:59:40 +0200
MIME-Version: 1.0
In-Reply-To: <20171215211943.enktaerpipdpqykl@intel.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: stable-owner@vger.kernel.org
List-ID: <stable.vger.kernel.org>

On 15.12.2017 23:19, Rodrigo Vivi wrote:
> On Fri, Dec 15, 2017 at 04:39:25PM +0000, Ingo Molnar wrote:
>>
>> * Lucas De Marchi <lucas.demarchi@intel.com> wrote:
>>
>>> CFL was missing from intel_early_ids[]. The PCI ID needs to be there to
>>> allow the memory region to be stolen, otherwise we could have RAM being
>>> arbitrarily overwritten if for example we keep using the UEFI framebuffer,
>>> depending on how BIOS has set up the e820 map.
>>>
>>> Fixes: b056f8f3d6b9 ("drm/i915/cfl: Add Coffee Lake PCI IDs for S Skus.")
>>> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>>> Cc: Anusha Srivatsa <anusha.srivatsa@intel.com>
>>> Cc: Jani Nikula <jani.nikula@linux.intel.com>
>>> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
>>> Cc: David Airlie <airlied@linux.ie>
>>> Cc: intel-gfx@lists.freedesktop.org
>>> Cc: dri-devel@lists.freedesktop.org
>>> Cc: Ingo Molnar <mingo@kernel.org>
>>> Cc: H. Peter Anvin <hpa@zytor.com>
>>> Cc: Thomas Gleixner <tglx@linutronix.de>
>>> Cc: x86@kernel.org
>>> Cc: <stable@vger.kernel.org> # v4.13+ 0890540e21cf drm/i915: add GT number to intel_device_info
>>> Cc: <stable@vger.kernel.org> # v4.13+ 41693fd52373 drm/i915/kbl: Change a KBL pci id to GT2 from GT1.5
>>> Cc: <stable@vger.kernel.org> # v4.13+
>>> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
>>> ---
>>>
>>> v2: improve commit message, add Fixes tag and CC stable
>>>
>>>  arch/x86/kernel/early-quirks.c | 1 +
>>>  include/drm/i915_pciids.h      | 6 ++++++
>>>  2 files changed, 7 insertions(+)
>>
>> Acked-by: Ingo Molnar <mingo@kernel.org>
> 
> Merged to drm-intel-next-queued.
> Thanks for the patches, suggestions and acks.

Shouldn't this go to v4.15 as well, because CFL is enabled there? I
don't see it in rc8.