Date: Tue, 21 Mar 2023 15:20:12 +0100 From: Corvin =?ISO-8859-1?Q?K=F6hne?= <corvink@FreeBSD.org> To: Konstantin Belousov <kostikbel@gmail.com>, =?ISO-8859-1?Q?Jean-S=E9bastien_P=E9dron?= <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: <5d5698b1fdd6699d27a158c87cab868d0ea8046a.camel@FreeBSD.org> In-Reply-To: <ZBm47yFbwdLowSaY@kib.kiev.ua> References: <202303202047.32KKlrbx083372@gitrepo.freebsd.org> <ZBm47yFbwdLowSaY@kib.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
--=-S6/V0E4mTSdKG9YaCwN8 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, 2023-03-21 at 16:02 +0200, Konstantin Belousov wrote: > On Mon, Mar 20, 2023 at 08:47:53PM +0000, Jean-S=C3=A9bastien P=C3=A9dron > wrote: > > The branch main has been updated by dumbbell: > >=20 > > URL: > > https://cgit.FreeBSD.org/src/commit/?id=3Dd780c6a6abb5275c8c7219539553d= 51ac94a023e > >=20 > > commit d780c6a6abb5275c8c7219539553d51ac94a023e > > Author:=C2=A0=C2=A0=C2=A0=C2=A0 Jean-S=C3=A9bastien P=C3=A9dron <dumbbe= ll@FreeBSD.org> > > AuthorDate: 2023-03-13 14:28:22 +0000 > > Commit:=C2=A0=C2=A0=C2=A0=C2=A0 Jean-S=C3=A9bastien P=C3=A9dron <dumbbe= ll@FreeBSD.org> > > CommitDate: 2023-03-20 20:47:36 +0000 > >=20 > > =C2=A0=C2=A0=C2=A0 x86/pci_early_quirks: Support Intel 11th+ gen > > =C2=A0=C2=A0=C2=A0=20 > > =C2=A0=C2=A0=C2=A0 Newer Intel CPUs/iGPUs use a new method to determine= the base > > address of > > =C2=A0=C2=A0=C2=A0 the stolen memory. This code was ported from Linux. > > =C2=A0=C2=A0=C2=A0=20 > > =C2=A0=C2=A0=C2=A0 Reviewed by:=C2=A0=C2=A0=C2=A0 manu > > =C2=A0=C2=A0=C2=A0 Approved by:=C2=A0=C2=A0=C2=A0 manu > > =C2=A0=C2=A0=C2=A0 Differential Revision:=C2=A0 https://reviews.freebsd= .org/D39057 > > --- > > =C2=A0sys/x86/pci/pci_early_quirks.c | 26 ++++++++++++ > > =C2=A0sys/x86/pci/pci_early_quirks.h | 95 > > ++++++++++++++++++++++++++++++++++++++++++ > > =C2=A02 files changed, 121 insertions(+) > Is this stuff still needed or useful?=C2=A0 When it was added, the claim > was that > it would be soon used to reserve regions of physical memory that > buggy BIOSes > did not.=C2=A0 That never happen. >=20 > If the only current consumer of the stolen memory calculations are in > drm > kmods, wouldn't it be better to move the code there? I'm going to make use of intel_graphics_stolen_base and intel_graphics_stolen_size for GPU passthrough of bhyve vms. See https://reviews.freebsd.org/D26209 --=20 Kind regards, Corvin --=-S6/V0E4mTSdKG9YaCwN8 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEgvRSla3m2t/H2U9G2FTaVjFeAmoFAmQZvRwACgkQ2FTaVjFe AmqGzw/5ARYA6wcjtrUxY4uFmz49wYYS5X3N7zXqab4fUBzi7dO75E4AA6pEjPt1 JlZ/aF4Qu9pblLAgBt+qkoPlJv1OTi6Ui2Qtjdj6jvQRzH6EmGl1g7PfW/gHppUN ethJtuZslaiWK8pAxpclHgsPmzWHVe0f5qxUwS6ZT3Pu0CyB5jNSLy5/zBipzg+c 74Oc+JJX+zomXrVI/iV9XtqVBkiV+ZKBEFtLv3SsBlu7Af9kk4peKVBfA/O/sHL7 GCIWSN39oOlRHruwOkc/WZ4H/404cgBS8nYiIPByQX/Bhmeaotn0jpExtTL/y23t /q+C/x7FCnlLBxoYjohI0sNufA23Mu1Aws3vcyb1qTtlsWm5bfiF1MJ2mk1FxXpN ZqPGoeCn/rMh61pOisCljoKUGgvGmCiE3Xw2GvnJcdkPGdaqXnDURxmMvG1q0BQf HOe8QyjEuhgdY37WpWmLqfX0fWVwQsH2h6bWdsAppPMRqwAasNmo1HAyyXSleXwx KBZSZ7rXK0CRtQpGZF2fwKhwyfzzSrgMTdUDC7xwFVHhseIhQsEo6xSKxw9Q3aGe FcSb4f36MTaohmeK1FBHX5xov8LLOfFo3SkMJp4k6WLXELtD9Y3FIWeyei2uyznx Du0MLPgmGDY0cdbuBrpkwYNBzieaMvQs4ycyw3e3zSPb+HhYxLQ= =jkaB -----END PGP SIGNATURE----- --=-S6/V0E4mTSdKG9YaCwN8--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5d5698b1fdd6699d27a158c87cab868d0ea8046a.camel>