Hi David et al.
After applying the patches I just sent to you
and a third for hpt366 (sent to Bart) my sparc64
build is section mismatch clean.
I have not tried different configs neither
allyesconfig etc.
Most of the remaining patches are queued by akpm
and I hope they hit mainlin before next me
.init.text:__alloc_bootmem()
alloc_bootmem() is only used during early init and for any subsequent
call to kernel_map_range() the program logic avoid the call.
So annotate kernel_map_range() with __ref to tell modpost to
ignore the reference to a __init function.
Signed-off-by: Sam Ravnborg <[EM
a() is used only during early init and for cpu
hotplug. So annotating them all __cpuinit solves the
section mismatch and should be correct.
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
Cc: David S. Miller <[EMAIL PROTECTED]>
---
Please review before you apply.
I am not 100% sure if
On Wed, Feb 20, 2008 at 10:28:39PM -0800, David Miller wrote:
> From: Sam Ravnborg <[EMAIL PROTECTED]>
> Date: Wed, 20 Feb 2008 22:50:20 +0100
>
> > Full diff of the included changes below.
> >
> > If you can use this in part or in toal they have my:
> &
() references mdesc_fill_in_cpu_data() via dr_cpu_configure()
and that looks bogus.
Full diff of the included changes below.
If you can use this in part or in toal they have my:
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
Sam
diff --git a/arch/sparc64/kernel/hvtramp.S b/arch/s
On Wed, Feb 13, 2008 at 12:41:20AM -0800, David Miller wrote:
>
> As I mentioned to a few ppc folks at LCA08 I plan to use
> the LMB code from powerpc as a basis for NUMA support on
> sparc64.
>
> There are two changes.
>
> 1) Move arch/powerpc/mm/lmb.c to lib/lmb.c, put the main
>interface
a .S file?
Note that for example contains
section definitions for use in .S files.
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
diff --git a/arch/sparc64/lib/GENbzero.S b/arch/sparc64/lib/GENbzero.S
index f9c71d6..6a4f956 100644
--- a/arch/sparc64/lib/GENbzero.S
+++ b/arch/s
>
> I was more concerned with the larger question that contexts like this,
> assignment of __init function pointers to dynamic data structure
> members, is not checked.
>
> The checker has no way to make sure that pbm->scan_bus() is only
> made in __init contexts, even though we can plainly see t
On Mon, Jan 21, 2008 at 05:24:58PM -0800, David Miller wrote:
> From: Sam Ravnborg <[EMAIL PROTECTED]>
> Date: Mon, 21 Jan 2008 21:22:10 +0100
>
> > Fix following Section mismatch warning in sparc64:
> >
> > WARNING: arch/sparc64/kernel/built-in.o(.text+0x13dec):
_devexit_p()
around the function pointers.
Unfortunately modpost did not catch this since modpost do not warn about
references from .data to .devexit from variables named *_driver.
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
diff --git a/drivers/input/misc/sparcspkr.c b/drivers/input/
G: arch/sparc64/kernel/built-in.o(.text+0x23868): Section mismatch:
reference to .devinit.text:vio_dev_release (between 'vio_create_one' and
'vio_add')
The pci_* were all missing __init annotations.
For the vio.c case it was a function with a wrong annotation which was removed.
Hi Steven.
On Thu, Jan 03, 2008 at 02:16:10AM -0500, Steven Rostedt wrote:
> If CONFIG_MCOUNT is selected and /proc/sys/kernel/mcount_enabled is set to a
> non-zero value the mcount routine will be called everytime we enter a kernel
> function that is not marked with the "notrace" attribute.
>
>
> diff --git a/kernel/Makefile b/kernel/Makefile
> index dfa9695..749825a 100644
> --- a/kernel/Makefile
> +++ b/kernel/Makefile
> @@ -80,3 +80,11 @@ quiet_cmd_ikconfiggz = IKCFG $@
> targets += config_data.h
> $(obj)/config_data.h: $(obj)/config_data.gz FORCE
> $(call if_changed,ikconfig
Hi Jeff.
On Wed, Oct 24, 2007 at 11:22:40AM -0400, Jeff Dike wrote:
> On Thu, Oct 11, 2007 at 05:54:46PM -0500, Rob Landley wrote:
> > Guess what? I still need this patch to build the final 2.6.23, months
> > later.
> >
> > I know it may not be the right fix, but the build breaks for me without
builds)
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
build tested only
Stumbled over these when looking for other things.
Sam
diff --git a/arch/sparc64/Makefile b/arch/sparc64/Makefile
index 6c92a42..01159cb 100644
--- a/arch/sparc64/Makefile
+++ b/arch/sparc64/Makefile
@@
reflect if ld added alignment.
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
buildtested only. But there should be no functional changes.
Will do this for all archs (over time).
Sam
diff --git a/arch/sparc64/kernel/vmlinux.lds.S
b/arch/sparc64/kernel/vmlinux.lds.S
index b
Make vmlinux.lds almost readable.
When going through the file fixed the following:
- Use PAGE_SIZE as replacement for hardcoded 4096
- Moves label definitions inside {} to avoid ld alignment
that may be added between label and section
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
On Thu, Jul 26, 2007 at 02:15:43PM +0200, Takashi Iwai wrote:
> At Thu, 26 Jul 2007 15:02:51 +0300 (EEST),
> Meelis Roos wrote:
> >
> > > > ERROR: "dma_free_coherent" [sound/core/snd-page-alloc.ko] undefined!
> > > > ERROR: "dma_alloc_coherent" [sound/core/snd-page-alloc.ko] undefined!
> > >
> >
Wrong list address in first try.
- Forwarded message from Sam Ravnborg <[EMAIL PROTECTED]> -
Subject: [PATCH 7/7] sparc64: fix section mismatch warning in mdesc.c
Sender: [EMAIL PROTECTED]
From: Sam Ravnborg <[EMAIL PROTECTED]>
Date: Fri, 20 Jul 2007 23:31:30 +0200
User
Wrong list address in first try.
- Forwarded message from Sam Ravnborg <[EMAIL PROTECTED]> -
Subject: [PATCH 6/7] sparc64: fix section mismatch warning in pci_sunv4
Sender: [EMAIL PROTECTED]
From: Sam Ravnborg <[EMAIL PROTECTED]>
Date: Fri, 20 Jul 2007 23:30:44 +0200
User
On Mon, Jun 04, 2007 at 11:58:28PM -0700, David Miller wrote:
> From: Sam Ravnborg <[EMAIL PROTECTED]>
> Date: Sat, 2 Jun 2007 22:32:06 +0200
>
> > I do not know if it is OK to use .init.text??
> > I would assume that CONFIG_HOTPLUG_CPU had impact here.
>
> Th
On Sat, Jun 02, 2007 at 02:02:20PM -0700, David Miller wrote:
> From: Sam Ravnborg <[EMAIL PROTECTED]>
> Date: Sat, 2 Jun 2007 22:32:06 +0200
>
> > The good news is that there is a nice pattern here. All sections that
> > have the references end in _patch.
> >
Hi David.
As per discussion in other mail I have looked into the section
mismatch warnings coming from trampoline.S
If the section is changed to .init.text it gives the following warnings:
WARNING: arch/sparc64/kernel/built-in.o(.sun4v_1insn_patch+0x8): Section
mismatch: reference to .init.text
On Mon, May 28, 2007 at 11:00:29PM -0700, David Miller wrote:
> From: Sam Ravnborg <[EMAIL PROTECTED]>
> Date: Mon, 28 May 2007 20:53:36 +0200
>
> > First off. I tried building a sparc64 kernel from -linus using
> > defconfig and got no section mismatch warning
>
> On a related topic, I've been wondering how to make instruction
> patching sections work with initdata without warning. I have
> a bunch of assembler patches in the main kernel image that go
> something like this:
>
> 661: ldx [%g7 + TRAP_PER_CPU_TSB_HUGE], %g5
> nop
>
+ sparclinux.
David asked me to mark them "Signed-off-by" but
my understanding are that Signed-off-by: document the path of a patch
so they are marked "Acked-by:".
Here is shortlog + diffstat.
Sam Ravnborg (5):
sparc64: ident vmlinux.lds.S
asm-generic: add RO_DATA
On Thu, May 24, 2007 at 01:45:12PM -0700, David Miller wrote:
> From: Sam Ravnborg <[EMAIL PROTECTED]>
> Date: Thu, 24 May 2007 22:09:34 +0200
>
> > The other archs shall have a similar clean-up over time.
> > sparc64 was just a good place to start because David had
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
arch/sparc64/kernel/vmlinux.lds.S |9 +
1 files changed, 1 insertions(+), 8 deletions(-)
diff --git a/arch/sparc64/kernel/vmlinux.lds.S
b/arch/sparc64/kernel/vmlinux.lds.S
index 8559d07..2d6fb93 100644
--- a/arch/sparc64/
Add the definition of initramfs to vmlinux.lds.h
This simplifies arch lds script a bit.
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
Is these 8 lines really worth it.
I think yes - but no less than this.
Sam
include/asm-generic/vmlinux.lds.h | 13 +
1
sparc64 had hardcoded pagesize to 8192 for alignment.
But optimal alignment is PAGE_SIZE so replace harcoded values
with the PAGE_SIZE constant.
Note: PAGE_SIZE differ with configuration so this is a change
in functionality - see page.h
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
RODATA has a hardcoded alignment of 4096 but architectures
may which to decide their optimal alignmnet.
Introduce RO_DATA that takes an alignment parameter.
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
include/asm-generic/vmlinux.lds.h |9 ++---
1 files changed, 6 insertions
This introduce a style similar to C code.
No functional changes.
Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]>
---
arch/sparc64/kernel/vmlinux.lds.S | 239 ++---
1 files changed, 142 insertions(+), 97 deletions(-)
diff --git a/arch/sparc64/kernel/vmlinux
Grumble.. why is this list not following usual naming scheme...
Please include all recepients listed below in replies.
Sam
To: David Miller <[EMAIL PROTECTED]>, [EMAIL PROTECTED],
[EMAIL PROTECTED], LKML <[EMAIL PROTECTED]>
With basis in sparc64 I did a small cleanup / consolidat
> Hi everybody,
> recently a change in the glibc elf.h header has been introduced causing
> modpost to spawn tons of warnings (like the one below) building the kernel on
> sparc:
>
Applied.
You need to reread SubmittingPatches:
> diff -urNad --exclude=CVS --exclude=.svn ./scripts/mod/modpost.c
On Wed, Mar 23, 2005 at 10:17:48AM -0800, David S. Miller wrote:
> On Wed, 23 Mar 2005 15:53:26 +0200 (EET)
> Meelis Roos <[EMAIL PROTECTED]> wrote:
>
> > floppy module in 2.6.12-rc1 has an unresolved symbol floppy_hardint.
>
> Yes, this is known not to work. The floppy driver can only
> work no
On Thu, Mar 03, 2005 at 11:31:27PM -0600, Tom 'spot' Callaway wrote:
>
> The end result is that the undefined CONFIG_ symbol errors are not
> thrown when the config utilities are run. The errors are not fatal, but
> they do cause confusion when users are building kernels for
> sparc/sparc64.
Anoth
36 matches
Mail list logo