Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 21 Mar 2023 16:02:23 +0200
From:      Konstantin Belousov <kostikbel@gmail.com>
To:        =?utf-8?Q?Jean-S=C3=A9bastien_P=C3=A9dron?= <dumbbell@freebsd.org>
Cc:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: d780c6a6abb5 - main - x86/pci_early_quirks: Support Intel 11th+ gen
Message-ID:  <ZBm47yFbwdLowSaY@kib.kiev.ua>
In-Reply-To: <202303202047.32KKlrbx083372@gitrepo.freebsd.org>
References:  <202303202047.32KKlrbx083372@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Mar 20, 2023 at 08:47:53PM +0000, Jean-Sébastien Pédron wrote:
> The branch main has been updated by dumbbell:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=d780c6a6abb5275c8c7219539553d51ac94a023e
> 
> commit d780c6a6abb5275c8c7219539553d51ac94a023e
> Author:     Jean-Sébastien Pédron <dumbbell@FreeBSD.org>
> AuthorDate: 2023-03-13 14:28:22 +0000
> Commit:     Jean-Sébastien Pédron <dumbbell@FreeBSD.org>
> CommitDate: 2023-03-20 20:47:36 +0000
> 
>     x86/pci_early_quirks: Support Intel 11th+ gen
>     
>     Newer Intel CPUs/iGPUs use a new method to determine the base address of
>     the stolen memory. This code was ported from Linux.
>     
>     Reviewed by:    manu
>     Approved by:    manu
>     Differential Revision:  https://reviews.freebsd.org/D39057
> ---
>  sys/x86/pci/pci_early_quirks.c | 26 ++++++++++++
>  sys/x86/pci/pci_early_quirks.h | 95 ++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 121 insertions(+)
Is this stuff still needed or useful?  When it was added, the claim was that
it would be soon used to reserve regions of physical memory that buggy BIOSes
did not.  That never happen.

If the only current consumer of the stolen memory calculations are in drm
kmods, wouldn't it be better to move the code there?



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?ZBm47yFbwdLowSaY>