[PATCH] * grub-core/fs/ext2.c (grub_ext2_read_block): Factor out common code for indirect block handling.

2013-11-13 Thread Colin Watson
Saves 185 bytes on compressed image. --- ChangeLog | 7 grub-core/fs/ext2.c | 119 +--- 2 files changed, 45 insertions(+), 81 deletions(-) diff --git a/ChangeLog b/ChangeLog index 716f69c..4454fd8 100644 --- a/ChangeLog +++ b/Change

Re: 2 quick questions

2013-11-13 Thread Colin Watson
On Sat, Nov 09, 2013 at 08:42:56AM -0700, Gerard Butler wrote: > Is it possible to pass a command to a Linux based OS? So for instance > I select Ubuntu or Fedora and I tell it to run some script as soon as > it boots and I can put this comand in the menuentry. You can put whatever you like on the

Division clash between core and libgcc on ARM

2013-11-13 Thread Colin Watson
./configure --build=x86_64-linux-gnu --host=arm-linux-gnueabihf --target=arm-linux-gnueabihf --with-platform=uboot This fails with: arm-linux-gnueabihf-gcc -Wall -W -Wshadow -Wold-style-definition -Wpointer-arith -Wundef -Wextra -Wattributes -Wchar-subscripts -Wcomment -Wdeprecated-declara

Re: [PATCH] remove dependency on /boot/config-* in grub.d/20_linux_xen

2013-11-13 Thread Konrad Rzeszutek Wilk
On Sun, Nov 10, 2013 at 09:40:29PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 15.07.2013 20:00, Konrad Rzeszutek Wilk wrote: > > Hey, > > > > There is a discussion on the linux-kernel mailing list in which the > > Linus states that "if you depend on any config file, you're broken >

Re: Division clash between core and libgcc on ARM

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 14:42, Colin Watson wrote: > ./configure --build=x86_64-linux-gnu --host=arm-linux-gnueabihf > --target=arm-linux-gnueabihf --with-platform=uboot > > This fails with: > > arm-linux-gnueabihf-gcc -Wall -W -Wshadow -Wold-style-definition > -Wpointer-arith -Wundef -Wextra -Wattri

Re: Division clash between core and libgcc on ARM

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 14:42, Colin Watson wrote: > How does this look? > > * grub-core/kern/misc.c: Don't redirect divisions in the > GRUB_UTIL case. You also need similar adjustment to misc.h signature.asc Description: OpenPGP digital signature _

Re: [RFT] Use pair of shifts instead of and in ARM code

2013-11-13 Thread Leif Lindholm
On Wed, Nov 13, 2013 at 01:00:17AM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > 1ff and 3fff can't be easily loaded on ARM. This patch replaces load+and > with pair of shift to keep only wanted bits. Actually, on ARMv7 we could just use MOVW instead: > diff --git a/grub-core/kern/arm/c

Re: Division clash between core and libgcc on ARM

2013-11-13 Thread Colin Watson
On Wed, Nov 13, 2013 at 03:08:35PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 13.11.2013 14:42, Colin Watson wrote: > > * grub-core/kern/misc.c: Don't redirect divisions in the > > GRUB_UTIL case. > > You also need similar adjustment to misc.h Good catch, thanks. C

arm-uboot: misc.S:56: Error: r13 not allowed here -- `sub sp,fp,#4'

2013-11-13 Thread Colin Watson
After my previous fix, arm-uboot still fails to build with: arm-linux-gnueabihf-gcc -DHAVE_CONFIG_H -I. -I../../../grub-core -I.. -Wall -W -I../../../include -I../include -DGRUB_MACHINE_UBOOT=1 -DGRUB_MACHINE_ARM=1 -DGRUB_MACHINE=ARM_UBOOT -DGRUB_TARGET_CPU_ARM=1 -nostdinc -isystem /usr/lib

Re: [RFT] Use pair of shifts instead of and in ARM code

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 15:16, Leif Lindholm wrote: > On Wed, Nov 13, 2013 at 01:00:17AM +0100, Vladimir 'φ-coder/phcoder' > Serbinenko wrote: >> 1ff and 3fff can't be easily loaded on ARM. This patch replaces load+and >> with pair of shift to keep only wanted bits. > > Actually, on ARMv7 we could just us

Re: arm-uboot: misc.S:56: Error: r13 not allowed here -- `sub sp, fp, #4'

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 15:49, Colin Watson wrote: > After my previous fix, arm-uboot still fails to build with: > > arm-linux-gnueabihf-gcc -DHAVE_CONFIG_H -I. -I../../../grub-core -I.. > -Wall -W -I../../../include -I../include -DGRUB_MACHINE_UBOOT=1 > -DGRUB_MACHINE_ARM=1 -DGRUB_MACHINE=ARM_UBOOT -

Re: arm-uboot: misc.S:56: Error: r13 not allowed here -- `sub sp,fp,#4'

2013-11-13 Thread Leif Lindholm
On Wed, Nov 13, 2013 at 02:49:19PM +, Colin Watson wrote: > After my previous fix, arm-uboot still fails to build with: > > arm-linux-gnueabihf-gcc -DHAVE_CONFIG_H -I. -I../../../grub-core -I.. > -Wall -W -I../../../include -I../include -DGRUB_MACHINE_UBOOT=1 > -DGRUB_MACHINE_ARM=1 -DGRU

[PATCH] Remove the need for thumb-interwork

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
The only place where we mix arm and thumb is hand-written asm parts. Attached patch adds explicit interworkers to asm and let's compile the rest without interworking. diff --git a/conf/Makefile.common b/conf/Makefile.common index 0dc26db..d0d52f0 100644 --- a/conf/Makefile.common +++ b/conf/Makefil

Re: arm-uboot: misc.S:56: Error: r13 not allowed here -- `sub sp, fp, #4'

2013-11-13 Thread Leif Lindholm
On Wed, Nov 13, 2013 at 04:15:53PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 13.11.2013 15:49, Colin Watson wrote: > > After my previous fix, arm-uboot still fails to build with: > > > > arm-linux-gnueabihf-gcc -DHAVE_CONFIG_H -I. -I../../../grub-core -I.. > > -Wall -W -I../../

ctz[sd]i2

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Hello, I've seen that you had problems with ctzdi2. Attached patch may help diff --git a/configure.ac b/configure.ac index 4ad7584..0182b42 100644 --- a/configure.ac +++ b/configure.ac @@ -829,7 +848,7 @@ CFLAGS="$TARGET_CFLAGS -Wl,--defsym,abort=main" fi # Check for libgcc symbols -AC_CHECK_FU

Re: ctz[sd]i2

2013-11-13 Thread Leif Lindholm
Hi Vladimir, On Wed, Nov 13, 2013 at 04:41:46PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > Hello, I've seen that you had problems with ctzdi2. Attached patch may help Thank you. I've had problems similar to the ones I used to have with __clear_cache on 32-bit ARM, in that the linker

Re: arm-uboot: misc.S:56: Error: r13 not allowed here -- `sub sp, fp, #4'

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 16:36, Leif Lindholm wrote: > On Wed, Nov 13, 2013 at 04:15:53PM +0100, Vladimir 'φ-coder/phcoder' > Serbinenko wrote: >> On 13.11.2013 15:49, Colin Watson wrote: >>> After my previous fix, arm-uboot still fails to build with: >>> >>> arm-linux-gnueabihf-gcc -DHAVE_CONFIG_H -I. -I.

Re: arm-uboot: misc.S:56: Error: r13 not allowed here -- `sub sp, fp, #4'

2013-11-13 Thread Leif Lindholm
On Wed, Nov 13, 2013 at 04:45:00PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > >>> misc.S: Assembler messages: > >>> misc.S:56: Error: r13 not allowed here -- `sub sp,fp,#4' > >>> make[3]: *** [kern/arm/kernel_exec-misc.o] Error 1 > >>> > >>> I don't think SP can be used that way i

Re: [Xen-devel] pvgrub2 is merged

2013-11-13 Thread Ian Campbell
On Sat, 2013-11-09 at 21:52 +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > Hello, all. pvgrub2 has just became part of upstream grub as ports i386-xen > and x86_64-xen. > http://git.savannah.gnu.org/cgit/grub.git I was just talking to some folks here and we thought this might make an inter

.gitignore additions?

2013-11-13 Thread Leif Lindholm
Hi there, Since the migration to git, I end up with a lot of untracked files after a build. The below gets rid of them: diff --git a/.gitignore b/.gitignore index e7251ff..1985a6e 100644 --- a/.gitignore +++ b/.gitignore @@ -127,6 +127,7 @@ mod-*.c missing netboot_test *.o +*.a ohci_test par

Re: .gitignore additions?

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 18:41, Leif Lindholm wrote: > Hi there, > > Since the migration to git, I end up with a lot of untracked files after > a build. The below gets rid of them: > Do you already have commit rights? If not, please register on savannah. If you do, go ahead. > diff --git a/.gitignore b/.git

Re: [PATCH] * grub-core/fs/ext2.c (grub_ext2_read_block): Factor out common code for indirect block handling.

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 12:33, Colin Watson wrote: > Saves 185 bytes on compressed image. Go ahead. > --- > ChangeLog | 7 > grub-core/fs/ext2.c | 119 > +--- > 2 files changed, 45 insertions(+), 81 deletions(-) > > diff --git a/ChangeLog b/

Re: [Xen-devel] pvgrub2 is merged

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 17:36, Ian Campbell wrote: > On Sat, 2013-11-09 at 21:52 +0100, Vladimir 'φ-coder/phcoder' Serbinenko > wrote: >> Hello, all. pvgrub2 has just became part of upstream grub as ports i386-xen >> and x86_64-xen. >> http://git.savannah.gnu.org/cgit/grub.git > > I was just talking to som

Re: [Xen-devel] pvgrub2 is merged

2013-11-13 Thread M A Young
On Sun, 10 Nov 2013, Andrey Borzenkov wrote: В Sat, 09 Nov 2013 21:52:20 +0100 Vladimir 'φ-coder/phcoder' Serbinenko пишет: Hello, all. pvgrub2 has just became part of upstream grub as ports i386-xen and x86_64-xen. http://git.savannah.gnu.org/cgit/grub.git Documentation on its usage is m

Re: [Xen-devel] pvgrub2 is merged

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 20:06, M A Young wrote: > > > On Sun, 10 Nov 2013, Andrey Borzenkov wrote: > >> В Sat, 09 Nov 2013 21:52:20 +0100 >> Vladimir 'φ-coder/phcoder' Serbinenko пишет: >> >>> Hello, all. pvgrub2 has just became part of upstream grub as ports >>> i386-xen and x86_64-xen. >>> http://git.s

Re: arm-uboot: misc.S:56: Error: r13 not allowed here -- `sub sp, fp, #4'

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2013 17:05, Leif Lindholm wrote: > On Wed, Nov 13, 2013 at 04:45:00PM +0100, Vladimir 'φ-coder/phcoder' > Serbinenko wrote: > misc.S: Assembler messages: > misc.S:56: Error: r13 not allowed here -- `sub sp,fp,#4' > make[3]: *** [kern/arm/kernel_exec-misc.o] Error 1 >

[PATCH] Compile asm-files to thumb2

2013-11-13 Thread Vladimir 'φ-coder/phcoder' Serbinenko
diff --git a/grub-core/kern/arm/cache.S b/grub-core/kern/arm/cache.S index 8522d24..2ad774d 100644 --- a/grub-core/kern/arm/cache.S +++ b/grub-core/kern/arm/cache.S @@ -21,7 +21,12 @@ .file "cache.S" .text .syntax unified +#if !defined (__thumb2__) || !defined (ARMV7)