Here is a series of patches that fix the most obvious sparc bugs and
get the grub2 tree building again.
I have code for boot/ that will implement the sparc boot blocks and
things of that nature, but that will come later.
First things first, let's get it to actually build. :)
__
This isn't %100 correct, but it gets the tree building.
2009-03-03 David S. Miller
* loader/ieee1275/multiboot2.c (grub_mb2_arch_boot): Handle
sparc like powerpc.
---
loader/ieee1275/multiboot2.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/loader/ieee1275/
2009-03-03 David S. Miller
* include/grub/sparc64/ieee1275/loader.h: New file.
* include/grub/sparc64/ieee1275/memory.h: Likewise.
* include/grub/sparc64/kernel.h: Likewise.
* loader/sparc64/ieee1275/linux.c: Likewise.
* loader/sparc64/ieee1275/linux_nor
We don't have that code model, and such a code model is unnecessary
on sparc64.
2009-03-03 David S. Miller
* configure.ac: Don't try to use -mcmodel=large on sparc.
* configure: Rebuilt.
---
configure|9 -
configure.ac |9 -
2 files changed, 16 ins
The sparc64 cache flush implementation was wrong in so many
ways.
When you "save" you have to allocate at least a full register
save area, which would be 128 bytes on sparc64. But we
actually don't need a register window here, this is a leaf
function and there are enough global and out registers
This corrects the sparc64 setjmp implementation.
We need to store the return address register, the
stack pointer, and frame pointer into the jump buf.
And on longjmp we need restore those registers, flush the register
window state, and pull in the top-most register window.
2009-03-03 David S.
This replaces the C implementation with a proper assembler
one. Also add missing grub_prefix[] declaration to
kernel.h
2009-03-03 David S. Miller
* kern/sparc64/ieee1275/init.c: Delete, replace with...
* kern/sparc64/ieee1275/crt0.S: assembler implementation.
* includ
From: Bean
Date: Wed, 4 Mar 2009 13:58:50 +0800
> On Wed, Mar 4, 2009 at 10:15 AM, David Miller wrote:
> >
> > We don't have that code model, and such a code model is unnecessary
> > on sparc64.
>
> Hi,
>
> Oh, thanks for pointing out. In fact, mcmodel=l
Many of these warnings are for two reasons:
1) grub_ssize_t is != 'int' on sparc64, and many pieces of code
try to pass an int pointer in for the final arg of several
ieee1275 "get property" functions which take a grub_ssize_t
pointer.
2) The third arg to grub_ieee1275_get_integer_prope
This facility will be used on sparc because the bootblocks
I will generate will be in A.OUT, so we need to pass that
to objcopy instead of 'binary'.
2009-03-04 David S. Miller
* genmk.rb: Allow rmk file to specify pkglib_IMAGES object
file format in IMG_FMT.
* conf/i38
2009-03-04 David S. Miller
* include/grub/sparc64/ieee1275/boot.h: New file.
* boot/sparc/ieee1275/boot.S: Likewise.
* boot/sparc/ieee1275/diskboot.S: Likewise.
* conf/sparc64-ieee1275.rmk: Build sparc boot blocks.
* conf/sparc64-ieee1275.mk: Regenerate.
In order for the boot block images for sparc to work properly,
we have to strip them out before doing the objcopy.
This is harmless on i386-pc and some other future grub
targets might need this too, so do it universally.
2009-03-04 David S. Miller
* genmk.rb: Strip object files built
From: David Miller
Date: Wed, 04 Mar 2009 01:07:35 -0800 (PST)
>
> 2009-03-04 David S. Miller
>
> * include/grub/sparc64/ieee1275/boot.h: New file.
> * boot/sparc/ieee1275/boot.S: Likewise.
> * boot/sparc/ieee1275/diskboot.S: Likewise.
> * conf
From: Vesa Jääskeläinen
Date: Wed, 04 Mar 2009 12:59:37 +0200
> David Miller wrote:
>
> > + stx %g1, [%sp + 2047 + (14 * 8)]
> > +retl
> > +restore %o1, 0, %o0
>
> Please check your white spaces. As we can see here those three commands
> are
From: Vesa Jääskeläinen
Date: Wed, 04 Mar 2009 13:34:46 +0200
> David Miller wrote:
> > diff --git a/include/grub/sparc64/setjmp.h b/include/grub/sparc64/setjmp.h
> > index 12d8e01..183a820 100644
> > --- a/include/grub/sparc64/setjmp.h
> > +++ b/include/grub/sparc64/
From: Vesa Jääskeläinen
Date: Wed, 04 Mar 2009 13:40:20 +0200
> David Miller wrote:
> > This replaces the C implementation with a proper assembler
> > one. Also add missing grub_prefix[] declaration to
> > kernel.h
> >
> > 2009-03-03 David S. Miller
>
From: Vesa Jääskeläinen
Date: Wed, 04 Mar 2009 13:44:16 +0200
> David Miller wrote:
> > 2009-03-03 David S. Miller
> >
> > * include/grub/sparc64/ieee1275/loader.h: New file.
> > * include/grub/sparc64/ieee1275/memory.h: Likewise.
> > * includ
From: Vesa Jääskeläinen
Date: Wed, 04 Mar 2009 13:50:37 +0200
> David Miller wrote:
> > There is no way anyone tried to build the sparc target in
> > years :-) This updates sparc64-ieee1275.rmk with modern
> > times and regenerates sparc64-ieee1275.mk as well as
> >
From: Vesa Jääskeläinen
Date: Wed, 04 Mar 2009 13:56:08 +0200
> David Miller wrote:
> > Many of these warnings are for two reasons:
> >
> > 1) grub_ssize_t is != 'int' on sparc64, and many pieces of code
> >try to pass an int pointer in for the fina
From: Vesa Jääskeläinen
Date: Wed, 04 Mar 2009 13:59:43 +0200
> David Miller wrote:
> > In order for the boot block images for sparc to work properly,
> > we have to strip them out before doing the objcopy.
> >
> > This is harmless on i386-pc and some other future
From: Javier Martín
Date: Wed, 04 Mar 2009 16:58:22 +0100
> As an example of what this entails, the failure to think with the future
> in mind has extraordinarily hampered the 32->64 bit transition: many
> Windows apps (among them the Vorbis codecs!) had be nearly rewritten
> because they used "u
From: Robert Millan
Date: Wed, 4 Mar 2009 22:26:51 +0100
> On Tue, Mar 03, 2009 at 06:15:08PM -0800, David Miller wrote:
> >
> > Here is a series of patches that fix the most obvious sparc bugs and
> > get the grub2 tree building again.
> >
> > I have code
2009-03-03 David S. Miller
* normal/sparc64/setjmp.S: Fix setjmp implementation.
* include/grub/sparc64/setjmp.h: Update grub_jmp_buf[].
---
include/grub/sparc64/setjmp.h |7 ---
normal/sparc64/setjmp.S | 20 ++--
2 files changed, 18 insertions(
2009-03-03 David S. Miller
* kern/sparc64/cache.S: Fix grub_arch_sync_caches implementation.
---
kern/sparc64/cache.S | 24 ++--
1 files changed, 10 insertions(+), 14 deletions(-)
diff --git a/kern/sparc64/cache.S b/kern/sparc64/cache.S
index 2ebb693..dac3f93 1006
2009-03-03 David S. Miller
* kern/sparc64/ieee1275/init.c: Delete, replace with...
* kern/sparc64/ieee1275/crt0.S: assembler implementation.
* include/grub/sparc64/ieee1275/kernel.h: Declare grub_prefix[].
---
include/grub/sparc64/ieee1275/kernel.h |6 +-
kern/sparc6
This is a respin of the sparc64 grub patches I posted
with changes based upon feedback (whitespace, use common
code, type police, don't include generated files, etc.)
It's also made against current top of trunk (there were
some sparc64-ieee1275.rmk changes in the past 24 hours
so this was necessa
2009-03-03 David S. Miller
* include/grub/sparc64/ieee1275/loader.h: New file.
* include/grub/sparc64/ieee1275/memory.h: Likewise.
* include/grub/sparc64/kernel.h: Likewise.
* loader/sparc64/ieee1275/linux.c: Likewise.
---
include/grub/sparc64/ieee1275/loader.h |
2009-03-03 David S. Miller
* loader/ieee1275/multiboot2.c (grub_mb2_arch_boot): Handle
sparc like powerpc.
---
loader/ieee1275/multiboot2.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/loader/ieee1275/multiboot2.c b/loader/ieee1275/multiboot2.c
index
2009-03-04 David S. Miller
* kern/ieee1275/cmain.c (grub_ieee1275_init): Third arg to
grub_ieee1275_get_integer_property is a grub_uint32_t pointer.
* kern/ieee1275/init.c (grub_machine_init): Fix type of
'actual'.
* kern/ieee1275/openfw.c (grub_children_i
2009-03-04 David S. Miller
* genmk.rb: Allow rmk file to specify pkglib_IMAGES object
file format in IMG_FMT.
* conf/i386-pc.rmk (IMG_FMT): Define.
---
conf/i386-pc.rmk |1 +
genmk.rb |2 +-
2 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/c
2009-03-03 David S. Miller
* conf/sparc64-ieee1275.rmk: Update.
---
conf/sparc64-ieee1275.rmk | 302 +---
1 files changed, 89 insertions(+), 213 deletions(-)
diff --git a/conf/sparc64-ieee1275.rmk b/conf/sparc64-ieee1275.rmk
index 1658a66..5be3e
2009-03-04 David S. Miller
* genmk.rb: Strip object files built for boot images.
---
genmk.rb |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/genmk.rb b/genmk.rb
index fe022cd..83ef639 100644
--- a/genmk.rb
+++ b/genmk.rb
@@ -61,6 +61,7 @@ MOSTLYCLEANFILES += #{
2009-03-04 David S. Miller
* include/grub/sparc64/ieee1275/boot.h: New file.
* boot/sparc/ieee1275/boot.S: Likewise.
* boot/sparc/ieee1275/diskboot.S: Likewise.
* conf/sparc64-ieee1275.rmk: Build sparc boot blocks.
* conf/sparc64-ieee1275.mk: Regenerate.
-
From: Isaac Dupree
Date: Thu, 5 Mar 2009 08:04:07 -0500
> David Miller wrote:
> > "unsigned long" is always safe because it is going to be
> > the largest natural word size on the machine, at least
> > as large as a pointer will be.
>
> er... C standard doe
From: Robert Millan
Date: Fri, 6 Mar 2009 19:55:53 +0100
> On Thu, Mar 05, 2009 at 08:04:07AM -0500, Isaac Dupree wrote:
> > > We've been using "unsigned long" for storing virtual addresses in the
> > > Linux kernel for 10+ years and it works just fine. :-)
> >
> > and Linux kernel uses GCC com
From: Robert Millan
Date: Fri, 6 Mar 2009 20:01:10 +0100
> In the future, please send followup patches in the same thread. It helps
> keeping track of things, which is quite hard for us to keep up with already..
Sure no problem.
What's going to happen with these patches?
I am able to boot int
From: Pavel Roskin
Date: Fri, 06 Mar 2009 14:03:57 -0500
> On Fri, 2009-03-06 at 19:55 +0100, Robert Millan wrote:
> > On Thu, Mar 05, 2009 at 08:04:07AM -0500, Isaac Dupree wrote:
> > > > We've been using "unsigned long" for storing virtual addresses in the
> > > > Linux kernel for 10+ years and
From: Robert Millan
Date: Sat, 7 Mar 2009 11:37:36 +0100
> On Fri, Mar 06, 2009 at 04:38:47PM -0800, David Miller wrote:
> > I am able to boot into the GRUB kernel prompt on my test system,
> > I have a lot of new code written, I'm _EXTREMELY_ motivated
> > and want to
From: Robert Millan
Date: Sat, 7 Mar 2009 11:38:38 +0100
> On Sat, Mar 07, 2009 at 12:56:43AM -0500, Pavel Roskin wrote:
> > Quoting David Miller :
> >
> >>> I was considering making grub_size_t long and grub_ssize_t unsigned
> >>> long. I remember
From: David Miller
Date: Sat, 07 Mar 2009 06:26:35 -0800 (PST)
> FWIW, everything other than actually booting a Linux kernel
> is functional in my local tree.
Sparc64 Linux kernel booting now works in my tree too, even compressed
kernel images and initrd are fully functional.
I&#
From: Kevin Lacquement
Date: Mon, 09 Mar 2009 12:03:14 -0700
> If we do that, though, we'll still need to create wrapper headers
> for the headers that flex #includes. Also, does grub depend on gcc,
> or will it work with other C compilers? I'd like to avoid adding a
> dependency on a specific
From: phcoder
Date: Thu, 12 Mar 2009 09:23:34 +0100
> Index: include/grub/elf.h
> ===
> --- include/grub/elf.h(revision 2036)
> +++ include/grub/elf.h(working copy)
> @@ -77,7 +77,7 @@
>Elf32_Half e_shentsize;
From: phcoder
Date: Thu, 12 Mar 2009 14:43:51 +0100
> Actually what I was doing now was discussing. If we don't discuss we
> may everyone create our own fork. I previously had problems because
> some of the structures in headers didn't have proper alignment
> attribute. My problem was that grub2
From: Robert Millan
Date: Fri, 13 Mar 2009 20:44:47 +0100
> When we have more than one video backend, and a clear idea on which parts are
> generic and which are backend-specific, we could readjust. For now this makes
> VBE happy.
I intend to write an ofvideo.c driver, so we'll have that
opport
From: phcoder
Date: Fri, 13 Mar 2009 21:41:42 +0100
> Actually our segment table is also our table for transforming
> between virtual and physical address. I don't see why entry point
> would be defined against virtual address of lowest physical segement
I would suggest simply looping over the p
One issue I need to resolve before I can send finalized
patches out for sparc is about device naming.
Currently the PowerPC ieee1275 support allows using both device
aliases and full openfirmware device path names with the usual GRUB
partition specification concatenated at the end. For the most
From: phcoder
Date: Sun, 15 Mar 2009 00:44:15 +0100
> For me it looks like it's disk driver-specific (ofdisk) and not
> platform-specific. Perhaps quoting or escaping could offer a better
> solution?
Specifying partitions using "/path/to/device:partition" syntax is an
attribute of openfirmware.
From: Robert Millan
Date: Sun, 15 Mar 2009 16:45:31 +0100
> It's not an absolute must that device names are unique. You can still
> identify partitions by their filesystem UUID or label, and in fact this
> is what our default scripts (grub-mkconfig) do anyway.
Things like UUID and labels are no
From: Vesa Jääskeläinen
Date: Sun, 15 Mar 2009 11:22:21 +0200
> David Miller wrote:
> > The problem is two fold:
> >
> > 1) "," characters can appear anywhere in an openfirmware path
> >name. For example my workstations disk is:
> >
> >
From: Robert Millan
Date: Wed, 18 Mar 2009 11:18:46 +0100
> On Sun, Mar 15, 2009 at 03:41:16PM -0700, David Miller wrote:
> > From: Robert Millan
> > Date: Sun, 15 Mar 2009 16:45:31 +0100
> >
> > > It's not an absolute must that device names are unique. You
From: David Miller
Date: Wed, 18 Mar 2009 13:55:22 -0700 (PDT)
> From: Robert Millan
> Date: Wed, 18 Mar 2009 11:18:46 +0100
>
> > On Sun, Mar 15, 2009 at 03:41:16PM -0700, David Miller wrote:
> > > From: Robert Millan
> > > Date: Sun, 15 Mar 2009 16:45:31
From: phcoder
Date: Sun, 22 Mar 2009 01:41:52 +0100
> Device name in boot block will anyway be written in it's complete
> raw OFW form, no matter how the rest of the grub2 accessing the
> disks. Using UUIDs in this place would be nice. On IRC you said that
> we have 120 bytes for the name. UUID i
From: phcoder
Date: Sun, 22 Mar 2009 01:48:14 +0100
> phcoder wrote:
> > Device name in boot block will anyway be written in it's complete raw OFW
> > form, no matter how the rest of the grub2 accessing the disks. Using UUIDs
> > in this place would be nice. On IRC you said that we have 120 byt
From: phcoder
Date: Sun, 22 Mar 2009 23:51:14 +0100
> You could do it the way similar to how it's done on i386-pc:
> jmp use_hardcoded
>
> use_hardcoded:
> And then the grub-setup tool nops out the first jump if stage2
> resides on a disk different than where bootblock is
Sure we could do that.
From: Robert Millan
Date: Sun, 22 Mar 2009 13:22:34 +0100
> On Sun, Mar 22, 2009 at 01:41:52AM +0100, phcoder wrote:
> > I don't see why
> > sth like
> > (/p...@1e\,60/p...@0/p...@9/p...@0/s...@1/d...@0,1)
> > is a problem.
>
> FWIW, I like this much better than replacing our comma separat
The FSF has received my copyright assignment paperwork so I'm starting
to feed fresh copies of my sparc64 changes.
Here, we fix the setjmp assembler implementation for sparc64 and also
we fix all of the grub_setjmp declarations to have the proper
"returns_twice" attribute.
For normal "setjmp" GC
From: Robert Millan
Date: Sat, 28 Mar 2009 14:42:34 +0100
> On Sat, Mar 28, 2009 at 02:36:00PM +0800, Bean wrote:
> > Hi,
> >
> > Currently, the grub-emu is in every rmk files, but I don't see the
> > difference between grub-emu in i386-pc, i386-coreboot or i386-efi. In
> > fact, grub-emu runs i
From: "Yoshinori K. Okuji"
Date: Sun, 29 Mar 2009 20:29:26 +0900
> So if nobody else can, I would like to get it straight again myself,
> although I am pretty busy (as I have a startup company, can you
> imagine how tough it is?).
Everybody is too busy to give this project the attention and time
From: "Yoshinori K. Okuji"
Date: Sun, 29 Mar 2009 21:55:34 +0900
> On Sunday 29 March 2009 20:40:17 David Miller wrote:
> > From: "Yoshinori K. Okuji"
> > Date: Sun, 29 Mar 2009 20:29:26 +0900
> >
> > > So if nobody else can, I would like to
From: Vesa Jääskeläinen
Date: Sun, 29 Mar 2009 16:23:11 +0300
> Yoshinori K. Okuji wrote:
> > On Sunday 29 March 2009 20:40:17 David Miller wrote:
> >> Everybody is too busy to give this project the attention and time it
> >> deserves to be maintained properly.
> &
From: phcoder
Date: Wed, 01 Apr 2009 09:43:01 +0200
> Move from todo on wiki is done except UltraSparc.
> David Miller: these todos seem to be very outdated by your work. If some of
> them or new tasks are applicable could you add them to task tracker?
> # Install on disk
> # Gr
From: "Yoshinori K. Okuji"
Date: Sat, 28 Mar 2009 23:34:25 +0900
> On Saturday 28 March 2009 22:39:48 Robert Millan wrote:
> > Unless there's someone else in the list who is knowledgeable about sparc
> > assembly and is going to review this part of David's patch, I'm going to
> > assume it's fine
From: "Yoshinori K. Okuji"
Date: Wed, 1 Apr 2009 22:59:03 +0900
> David, I can give you a permission, if you tell me your account on Savannah.
It is simply "davem"
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/g
From: Robert Millan
Date: Wed, 1 Apr 2009 15:01:11 +0200
> On Wed, Apr 01, 2009 at 01:17:40AM -0700, David Miller wrote:
> > From: "Yoshinori K. Okuji"
> > Date: Sat, 28 Mar 2009 23:34:25 +0900
> >
> > > On Saturday 28 March 2009 22:39:48 Robert Mill
1) Window save was done wrongly, but no matter we don't even
need to allocate a register window for such a simple routine,
there are enough registers available to do this
2) 'flush' operates on at least 8 bytes at a time, as defined
by V9, so 8 byte align the extremities of the region an
2009-04-01 David S. Miller
* kern/sparc64/dl.c (grub_arch_dl_relocate_symbols): Add
support for R_SPARC_OLO10 relocations. Fix compile warning for
R_SPARC_WDISP30 case.
---
kern/sparc64/dl.c | 12 +---
1 files changed, 9 insertions(+), 3 deletions(-)
diff -
From: David Miller
Date: Wed, 01 Apr 2009 17:14:51 -0700 (PDT)
>
> 1) Window save was done wrongly, but no matter we don't even
>need to allocate a register window for such a simple routine,
>there are enough registers available to do this
>
> 2) 'flush'
From: David Miller
Date: Wed, 01 Apr 2009 17:16:08 -0700 (PDT)
> 2009-04-01 David S. Miller
>
> * kern/sparc64/dl.c (grub_arch_dl_relocate_symbols): Add
> support for R_SPARC_OLO10 relocations. Fix compile warning for
> R_SPARC_WDIS
From: Pavel Roskin
Date: Tue, 07 Apr 2009 18:06:15 -0400
> On Tue, 2009-04-07 at 14:59 -0700, David Miller wrote:
>> From: David Miller
>> Date: Wed, 01 Apr 2009 17:16:08 -0700 (PDT)
>>
>> > 2009-04-01 David S. Miller
>> >
>> >* kern/s
This fixes the use of various fundamental types involved in
ieee1275 calls.
In ieee1275, the array of arguments and return values passed
to the ieee1275 call handler is defined as an array of "cells".
No ifs, ands, or buts. They are all cells and must be of the
size of a cell for a given platfor
The 'actual' argument to various ieee1275 functions is of type
"grub_ssize_t" but several spots were erroneously using "int".
This happens to work on powerpc but does not on sparc64.
2009-04-07 David S. Miller
* kern/ieee1275/init.c (grub_machine_init): Make 'actual' grub_ssize_t.
From: David Miller
Date: Tue, 07 Apr 2009 17:03:26 -0700 (PDT)
> 2009-04-07 David S. Miller
>
> * disk/ieee1275/nand.c (grub_nand_open): All ieee1275 call arg
> slots are of type grub_ieee1275_cell_t.
> (grub_nand_read): Likewise.
> * kern/i
From: David Miller
Date: Tue, 07 Apr 2009 17:03:27 -0700 (PDT)
> The 'actual' argument to various ieee1275 functions is of type
> "grub_ssize_t" but several spots were erroneously using "int".
>
> This happens to work on powerpc but does not on
From: Manoel Rebelo Abranches
Date: Thu, 09 Apr 2009 18:42:08 -0300
> At first it seems ok. I'll test it in the weekend ok?
Thanks! Let me know how it goes.
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-d
From: Manoel Rebelo Abranches
Date: Thu, 09 Apr 2009 20:00:43 -0300
> This patch is ok. it shouldn't be int.
Committed, thanks.
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
The grub printf code tries to treat 'long' and 'int' the same,
that doesn't work on (most) 64-bit platforms.
2009-04-11 David S. Miller
* kern/misc.c (grub_ltoa): New function.
(grub_vsprintf): Use it to format 'long' integers.
---
kern/misc.c | 35 +
This patch allows a platform to specify the image file output
format, it also adds the --strip-unneeded option to the objcopy
invocation as well.
I've left the i386-pc.mk portion of this patch out of the posting,
but rest assured I will check that part in once this gets an ACK :-)
2009-04-11 Da
The ieee1275 ofdisk driver doesn't use a unique value for
disk->id so it's really easy to get disk corruption. I was
able to see such corruption by simply booting grub from one
disk and booting a Linux kernel from another, both of which
were on the same disk controller.
The solution implemented
Newer versions of gcc emit references to libgcc internal
functions __bswapsi2 and __bswapdi2 when building ufs and
some other filesystem modules on sparc64.
Unfortunately older versions of gcc lack these symbols in
libgcc so we can't export them universally, we have to check
for it using configur
This adds the sparc64 boot block implementation for GRUB.
There might end up being minor changes to this in the end,
depending upon how we end up handling openfirmware path names.
But by in large this is the final way the code will look.
I've documented how these boot blocks work on several
occa
From: "Yoshinori K. Okuji"
Date: Sat, 11 Apr 2009 19:23:23 +0900
> On Saturday 11 April 2009 17:08:31 David Miller wrote:
>> This patch allows a platform to specify the image file output
>> format, it also adds the --strip-unneeded option to the objcopy
>> invocat
From: "Yoshinori K. Okuji"
Date: Sat, 11 Apr 2009 19:19:49 +0900
> On Saturday 11 April 2009 17:08:29 David Miller wrote:
>> The grub printf code tries to treat 'long' and 'int' the same,
>> that doesn't work on (most) 64-bit platforms.
>
>
From: Pavel Roskin
Date: Sat, 11 Apr 2009 23:53:33 -0400
> On Sat, 2009-04-11 at 01:27 -0700, David Miller wrote:
>> Newer versions of gcc emit references to libgcc internal
>> functions __bswapsi2 and __bswapdi2 when building ufs and
>> some other filesyst
From: Pavel Roskin
Date: Sun, 12 Apr 2009 02:29:15 -0400
> On Sat, 2009-04-11 at 01:08 -0700, David Miller wrote:
>> The ieee1275 ofdisk driver doesn't use a unique value for
>> disk->id so it's really easy to get disk corruption. I was
>> able to see such corr
From: Pavel Roskin
Date: Sun, 12 Apr 2009 02:39:22 -0400
> On Sat, 2009-04-11 at 01:08 -0700, David Miller wrote:
>> The grub printf code tries to treat 'long' and 'int' the same,
>> that doesn't work on (most) 64-bit platforms.
>
>> + n
From: phcoder
Date: Sat, 11 Apr 2009 21:10:26 +0200
> I do not agree with having separate grub_itoa/grub_ltoa/grub_lltoa in
> kernel where the size is critical. We should have only lltoa and use
> it in all contexts.
I look forward to your patch implementing this improvement :-)
_
From: phcoder
Date: Sun, 12 Apr 2009 12:44:10 +0200
> David Miller wrote:
>> From: Pavel Roskin
>> Date: Sun, 12 Apr 2009 02:29:15 -0400
>>
>>> On Sat, 2009-04-11 at 01:08 -0700, David Miller wrote:
>> I think fixing disk cache corruption is more important t
From: Pavel Roskin
Date: Sun, 12 Apr 2009 17:01:07 -0400
> It's not just about distribution properties. My impression is that your
> code misuses hashes as indices in a table. Let's make the hash 32-bit.
> That would make it harder to misuse.
They aren't indices in the traditional sense, they
From: Pavel Roskin
Date: Sun, 12 Apr 2009 17:19:42 -0400
> On Sun, 2009-04-12 at 00:58 -0700, David Miller wrote:
>
>> > Actually, changes to configure.ac affect other users as well. Users of
>> > other architectures would have to run those checks.
>>
>>
From: Pavel Roskin
Date: Sun, 12 Apr 2009 17:24:49 -0400
> On Sat, 2009-04-11 at 08:29 -0700, Colin D Bennett wrote:
>
>> If we could build with -Werror, then it wouldn't be so hard to find the
>> warnings since the build would abort...
>
> It's also possible to redirect stderr to a file so th
From: Pavel Roskin
Date: Mon, 13 Apr 2009 01:06:49 -0400
> On Sun, 2009-04-12 at 18:05 -0700, David Miller wrote:
>
>> > By the way, the checks for those functions should be done with the
>> > target compiler, that is after the second occurrence of CC="$TA
From: David Miller
Date: Sat, 11 Apr 2009 04:46:44 -0700 (PDT)
> From: "Yoshinori K. Okuji"
> Date: Sat, 11 Apr 2009 19:23:23 +0900
>
>> On Saturday 11 April 2009 17:08:31 David Miller wrote:
>>> This patch allows a platform to specify the image file output
&g
From: phcoder
Date: Mon, 13 Apr 2009 03:09:20 +0200
> David Miller wrote:
>> Ummm, that's essentially what my code does, except that the "index"
>> is the address of the cached path entry itself.
> Sorry I totally misread your code. Now it's fine
Thanks for
make_device_name() uses an arbitrary value for the device name
length, putting an arbitrary limit on such strings.
We can do better by calculating the length of this string precisely.
2009-04-12 David S. Miller
* util/hostdisk.c (make_device_name): Do not make any assumptions
Should be self-explanatory, these tools will be used on sparc64 just
like they are on i386-pc
2009-04-12 David S. Miller
* conf/common.rmk (grub_probe_SOURCES): Add Sun partition module.
(grub_fstest_SOURCES): Likewise.
---
conf/common.rmk |5 +++--
1 files changed, 3 ins
Openfirmware block device drivers do not allow for multiple concurrent
opens of the same device. It simply does not work.
Fortunately grub doesn't actually need to do this, it just happens
to do so. Here is the fix for the case of module dependency handling.
The other case that hits this probl
I've commited these changes.
2009-04-12 David S. Miller
* kern/sparc64/ieee1275/init.c: Delete, replace with...
* kern/sparc64/ieee1275/crt0.S: assembler implementation.
* include/grub/sparc64/ieee1275/kernel.h: Declare grub_prefix[].
(GRUB_MOD_ALIGN, GRUB_MOD_GA
This adds the ieee1275 interfaces sparc64's openboot has for
allocating physical memory, creating mappings, etc.
We can't reuse any of the existing similar calls used by powerpc since
these use two cells for passing physical addresses (even though only
one of those cell slots is used, this is leg
This adds the sparc64 loader for Linux kernels.
Committed.
2009-04-13 David S. Miller
* include/grub/sparc64/ieee1275/loader.h: New file.
* include/grub/sparc64/ieee1275/memory.h: Likewise.
* include/grub/sparc64/kernel.h: Likewise.
* loader/sparc64/ieee1275/lin
From: Chip Panarchy
Date: Mon, 13 Apr 2009 17:40:22 +1000
> Hello
>
> When you mention 'Sun Partition' what file-system are you specifying?
>
> ZFS?
> UFS?
EXT3, but the sun "partition" support has to be there for grub tools
to even get at the filesystems at all.
1 - 100 of 196 matches
Mail list logo
