Date: Fri, 22 Jan 2016 09:19:22 +0100 From: =?UTF-8?Q?Roger_Pau_Monn=c3=a9?= <royger@FreeBSD.org> To: Devin Teske <dteske@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org, Baptiste Daroussin <bapt@freebsd.org> Subject: Re: svn commit: r294417 - in stable/10/sys/boot: common fdt forth i386/libi386 i386/loader Message-ID: <56A1E60A.6080808@FreeBSD.org> In-Reply-To: <16A6EF29-CA34-43BC-8587-51C7B71BBE72@freebsd.org> References: <201601201323.u0KDN2s0057920@repo.freebsd.org> <16A6EF29-CA34-43BC-8587-51C7B71BBE72@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
El 21/01/16 a les 21.38, Devin Teske ha escrit: > >> On Jan 20, 2016, at 5:23 AM, Roger Pau Monné <royger@FreeBSD.org> wrote: >> >> Author: royger >> Date: Wed Jan 20 13:23:02 2016 >> New Revision: 294417 >> URL: https://svnweb.freebsd.org/changeset/base/294417 >> >> Log: >> MFC r277215, r277291, r277418, r280953 and r280954: >> >> loader: implement multiboot support for Xen Dom0 >> >> Note that only the subset of the multiboot specification needed in order to >> boot a Xen Dom0 is implemented. >> >> Sponsored by: Citrix Systems R&D >> > > > Two issues with this merge. > > First, this port now needs to be re-educated: > https://svnweb.freebsd.org/ports/head/emulators/xen-kernel/Makefile?view=markup <https://svnweb.freebsd.org/ports/head/emulators/xen-kernel/Makefile?view=markup> > > 46 <https://svnweb.freebsd.org/ports/head/emulators/xen-kernel/Makefile?view=markup#l46> .if ${OSVERSION} < 1100055 > 47 <https://svnweb.freebsd.org/ports/head/emulators/xen-kernel/Makefile?view=markup#l47> IGNORE= Only supported on recent FreeBSD 11 > 48 <https://svnweb.freebsd.org/ports/head/emulators/xen-kernel/Makefile?view=markup#l48> .endif > > Looking at your MFC, it looks like the OSVERSION check should allow some version of 10. > > Unfortunately (second issue), this patch doesn't touch stable/10/sys/sys/param.h so the port has no clear way of accurately making sure you're on a 10.x release that supports Xen. Hm, no. This is a backport of the loader changes only, in order to make it possible to boot a HEAD kernel plus Xen on a 10 world (and to make it easier to MFC other loader changes to 10). The stable/10 kernel is still not able to run as Xen Dom0, so the port is correct. That's why I haven't added 'Relnotes: yes' to the commit message :). Roger.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?56A1E60A.6080808>