[Xen-ia64-devel] RE: [Fedora-xen] fedora-xen-ia64 first pass

[Aron Griffis]

Akio Takebe wrote:  [Mon May 22 2006, 04:12:45AM EDT]
> This is a patch to turn on vp mode of xen.
> But I have compile error.
> 
> diff -r f7ff4d62d186 SPECS/xen.spec
> --- a/SPECS/xen.spec    Sat May 20 13:57:15 2006 -0400
> +++ b/SPECS/xen.spec    Tue May 23 11:11:14 2006 +0900
> @@ -23,6 +23,9 @@ Patch12: xen-3.0.1-tempdir.patch
>  # Patches to modify the default config of xend
>  Patch100: xen-config-dom0-minmem.patch
>  Patch101: xen-config-allow-http.patch
> +
> +# Patches to turn on vp mode of xen
> +Patch110: xen-vp_on.patch
>  
>  BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root
>  BuildRequires: transfig libidn-devel zlib-devel texi2html SDL-devel curl-devel
> @@ -67,6 +70,7 @@ or makes fun of you in front of your fri
>  # config patches
>  %patch100 -p1
>  %patch101 -p1
> +%patch110 -p1
>  
>  cp %{SOURCE2} .

Thanks, but I think this is making the change in the wrong place.  The
fedora-xen-ia64 repository is only the tools, not the hypervisor.
(This is a confusing misnomer on the part of Fedora.)

The change needs to be made instead in fedora-kernel-ia64, which
contains both the hypervisor and the kernel.  I think I fixed this in
cset 16.  Does it look right to you?

> @@ -150,8 +154,8 @@ rm -rf $RPM_BUILD_ROOT
>  %{_libdir}/%{name}
>  %{_libdir}/lib*
>  %{_mandir}/man?/*.?.gz
> -%dir /usr/lib/xen
> -%dir /usr/lib/xen/boot
> +#%dir /usr/lib/xen
> +#%dir /usr/lib/xen/boot
>  %ifarch i386 x86_64
>  /usr/lib/xen/boot/hvmloader
>  %endif

This part looks right.  I'm not sure why it was being specified twice
originally.  I just removed the lines instead of commenting them, see
fedora-xen-ia64 cset 3

> --- compile error is the below -----
> fedora-xen-ia64/BUILD/xen-unstable.hg/xen/arch/ia64/linux -I/root/fedora/fedora-xen-ia64/BUILD/xen-unstable.hg/xen/arch/ia64/linux-xen -DIA64 -DXEN -DLINUX_2_6 -DV_IOSAPIC_READY -ffixed-r13 -mfixed-range=f2-f5,f12-f127 -g -DCONFIG_XEN_IA64_DOM0_VP -g -D__XEN__ -c domain.c -o domain.o
> domain.c: In function ‘zap_domain_page_one’:
> domain.c:755: warning: implicit declaration of function ‘ptep_get_and_clear’
> domain.c:755: error: incompatible types in assignment
> domain.c: In function ‘assign_domain_page_replace’:
> domain.c:957: error: incompatible types in assignment
> make[5]: *** [domain.o] Error 1
> make[5]: Leaving directory `/root/fedora/fedora-xen-ia64/BUILD/xen-unstable.hg/xen/arch/ia64/xen'
> make[4]: *** [xen/built_in.o] Error 2
> make[4]: Leaving directory `/root/fedora/fedora-xen-ia64/BUILD/xen-unstable.hg/xen/arch/ia64'
> make[3]: *** [/root/fedora/fedora-xen-ia64/BUILD/xen-unstable.hg/xen/arch/ia64/built_in.o] Error 2
> make[3]: Leaving directory `/root/fedora/fedora-xen-ia64/BUILD/xen-unstable.hg/xen/arch/ia64'
> make[2]: *** [/root/fedora/fedora-xen-ia64/BUILD/xen-unstable.hg/xen/xen] Error 2
> make[2]: Leaving directory `/root/fedora/fedora-xen-ia64/BUILD/xen-unstable.hg/xen'
> make[1]: *** [install] Error 2
> make[1]: Leaving directory `/root/fedora/fedora-xen-ia64/BUILD/xen-unstable.hg/xen'
> make: *** [install-xen] Error 2

I think this comes from building the hypervisor in the wrong
repository.  Try fedora-kernel-ia64 instead (and sorry for the
confusion)

Regards,
Aron