[Qemu-devel] [PATCH RFC] PowerPC: Added uapi directory into linux-header
This is corrently done for powerpc. Signed-off-by: Bharat Bhushan --- configure |1 + scripts/update-linux-headers.sh |5 + 2 files changed, 6 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 780b19a..bdc2d5e 100755 --- a/configure +++ b/configure @@ -3701,6 +3701,7 @@ if test "$linux" = "yes" ; then # For non-KVM architectures we will not have asm headers if [ -e "$source_path/linux-headers/asm-$linux_arch" ]; then symlink "$source_path/linux-headers/asm-$linux_arch" linux-headers/asm + symlink "$source_path/linux-headers/uapi/asm-$linux_arch" linux-headers/uapi/asm fi fi diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh index 4c7b566..9f6bf25 100755 --- a/scripts/update-linux-headers.sh +++ b/scripts/update-linux-headers.sh @@ -48,6 +48,11 @@ for arch in $ARCHLIST; do rm -rf "$output/linux-headers/asm-$arch" mkdir -p "$output/linux-headers/asm-$arch" +if [ $arch = powerpc ]; then + rm -rf $output/linux-headers/uapi/asm-$arch/* +cp "$linux/arch/$arch/include/uapi/asm/epapr_hcalls.h" "$output/linux-headers/uapi/asm-$arch/" +fi + for header in kvm.h kvm_para.h; do cp "$tmpdir/include/asm/$header" "$output/linux-headers/asm-$arch" done -- 1.7.0.4
Re: [Qemu-devel] [PATCH RFC] PowerPC: Added uapi directory into linux-header
On 14.12.2012, at 12:04, Bharat Bhushan wrote: > This is corrently done for powerpc. > > Signed-off-by: Bharat Bhushan Jan, could you please check if this is correct? > --- > configure |1 + > scripts/update-linux-headers.sh |5 + > 2 files changed, 6 insertions(+), 0 deletions(-) > > diff --git a/configure b/configure > index 780b19a..bdc2d5e 100755 > --- a/configure > +++ b/configure > @@ -3701,6 +3701,7 @@ if test "$linux" = "yes" ; then > # For non-KVM architectures we will not have asm headers > if [ -e "$source_path/linux-headers/asm-$linux_arch" ]; then > symlink "$source_path/linux-headers/asm-$linux_arch" linux-headers/asm > + symlink "$source_path/linux-headers/uapi/asm-$linux_arch" > linux-headers/uapi/asm > fi > fi > > diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh > index 4c7b566..9f6bf25 100755 > --- a/scripts/update-linux-headers.sh > +++ b/scripts/update-linux-headers.sh > @@ -48,6 +48,11 @@ for arch in $ARCHLIST; do > > rm -rf "$output/linux-headers/asm-$arch" > mkdir -p "$output/linux-headers/asm-$arch" > +if [ $arch = powerpc ]; then This looks bogus. There shouldn't be any powerpc specifics anywhere in this file. Alex > + rm -rf $output/linux-headers/uapi/asm-$arch/* > +cp "$linux/arch/$arch/include/uapi/asm/epapr_hcalls.h" > "$output/linux-headers/uapi/asm-$arch/" > +fi > + > for header in kvm.h kvm_para.h; do > cp "$tmpdir/include/asm/$header" "$output/linux-headers/asm-$arch" > done > -- > 1.7.0.4 > >
Re: [Qemu-devel] [PATCH RFC] PowerPC: Added uapi directory into linux-header
On 14 December 2012 11:04, Bharat Bhushan wrote: > This is corrently done for powerpc. This commit message is a bit cryptic. > Signed-off-by: Bharat Bhushan > --- > configure |1 + > scripts/update-linux-headers.sh |5 + > 2 files changed, 6 insertions(+), 0 deletions(-) > > diff --git a/configure b/configure > index 780b19a..bdc2d5e 100755 > --- a/configure > +++ b/configure > @@ -3701,6 +3701,7 @@ if test "$linux" = "yes" ; then > # For non-KVM architectures we will not have asm headers > if [ -e "$source_path/linux-headers/asm-$linux_arch" ]; then >symlink "$source_path/linux-headers/asm-$linux_arch" linux-headers/asm > + symlink "$source_path/linux-headers/uapi/asm-$linux_arch" > linux-headers/uapi/asm > fi This creates a broken symlink if you run it on an arch that doesn't have a uapi directory: cam-vm-266:precise:qemu$ file linux-headers/uapi/asm linux-headers/uapi/asm: broken symbolic link to `/home/petmay01/linaro/qemu-from-laptop/qemu/linux-headers/uapi/asm-x86' Shouldn't it go in its own "if [ -e "$source_path/linux-headers/uapi/asm-$linux_arch" ]" guard ? > fi > > diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh > index 4c7b566..9f6bf25 100755 > --- a/scripts/update-linux-headers.sh > +++ b/scripts/update-linux-headers.sh > @@ -48,6 +48,11 @@ for arch in $ARCHLIST; do > > rm -rf "$output/linux-headers/asm-$arch" > mkdir -p "$output/linux-headers/asm-$arch" > +if [ $arch = powerpc ]; then > + rm -rf $output/linux-headers/uapi/asm-$arch/* > +cp "$linux/arch/$arch/include/uapi/asm/epapr_hcalls.h" > "$output/linux-headers/uapi/asm-$arch/" > +fi (1) This doesn't seem to be doing the mkdir of "$output/linux-headers/uapi/asm-$arch/" anywhere. (2) The setup of the uapi/ subdirectory shouldn't be inside an "if [ $arch = powerpc ]" guard, so it can work on all archs with a uapi/. You want to rm-and-mkdir the uapi/ directory if $linux/arch/$arch/include/uapi/ exists. (3) it would be neater to put the "copy this ppc specific header file" code at the bottom of the for loop next to the existing "copy this x86 specific header file" code. -- PMM
Re: [Qemu-devel] [PATCH RFC] PowerPC: Added uapi directory into linux-header
> -Original Message- > From: Alexander Graf [mailto:ag...@suse.de] > Sent: Friday, December 14, 2012 5:06 PM > To: Bhushan Bharat-R65777 > Cc: qemu-devel qemu-devel; qemu-...@nongnu.org List; Bhushan Bharat-R65777; > Jan > Kiszka > Subject: Re: [PATCH RFC] PowerPC: Added uapi directory into linux-header > > > On 14.12.2012, at 12:04, Bharat Bhushan wrote: > > > This is corrently done for powerpc. > > > > Signed-off-by: Bharat Bhushan > > Jan, could you please check if this is correct? > > > --- > > configure |1 + > > scripts/update-linux-headers.sh |5 + > > 2 files changed, 6 insertions(+), 0 deletions(-) > > > > diff --git a/configure b/configure > > index 780b19a..bdc2d5e 100755 > > --- a/configure > > +++ b/configure > > @@ -3701,6 +3701,7 @@ if test "$linux" = "yes" ; then > > # For non-KVM architectures we will not have asm headers > > if [ -e "$source_path/linux-headers/asm-$linux_arch" ]; then > > symlink "$source_path/linux-headers/asm-$linux_arch" > > linux-headers/asm > > + symlink "$source_path/linux-headers/uapi/asm-$linux_arch" > > + linux-headers/uapi/asm > > fi > > fi > > > > diff --git a/scripts/update-linux-headers.sh > > b/scripts/update-linux-headers.sh index 4c7b566..9f6bf25 100755 > > --- a/scripts/update-linux-headers.sh > > +++ b/scripts/update-linux-headers.sh > > @@ -48,6 +48,11 @@ for arch in $ARCHLIST; do > > > > rm -rf "$output/linux-headers/asm-$arch" > > mkdir -p "$output/linux-headers/asm-$arch" > > +if [ $arch = powerpc ]; then > > This looks bogus. There shouldn't be any powerpc specifics anywhere in this > file. This file have x86 specific also, why ? -Bharat > > > Alex > > > + rm -rf $output/linux-headers/uapi/asm-$arch/* > > +cp "$linux/arch/$arch/include/uapi/asm/epapr_hcalls.h" > "$output/linux-headers/uapi/asm-$arch/" > > +fi > > + > > for header in kvm.h kvm_para.h; do > > cp "$tmpdir/include/asm/$header" "$output/linux-headers/asm-$arch" > > done > > -- > > 1.7.0.4 > > > > >
