Skip site navigation (1)Skip section navigation (2)
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>