* Fix s390 build failure.
@ 2006-02-10 20:04 Dave Jones
2006-02-10 21:14 ` Linus Torvalds
0 siblings, 1 reply; 6+ messages in thread
From: Dave Jones @ 2006-02-10 20:04 UTC (permalink / raw)
To: Linux Kernel; +Cc: torvalds
arch/s390/kernel/compat_signal.c:199: error: conflicting types for 'do_sigaction'
include/linux/sched.h:1115: error: previous declaration of 'do_sigaction' was here
Signed-off-by: Dave Jones <davej@redhat.com>
--- linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c~ 2006-02-10 12:47:57.000000000 -0500
+++ linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c 2006-02-10 12:48:05.000000000 -0500
@@ -196,7 +196,7 @@ sys32_sigaction(int sig, const struct ol
}
int
-do_sigaction(int sig, const struct k_sigaction *act, struct k_sigaction *oact);
+do_sigaction(int sig, struct k_sigaction *act, struct k_sigaction *oact);
asmlinkage long
sys32_rt_sigaction(int sig, const struct sigaction32 __user *act,
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Fix s390 build failure.
2006-02-10 20:04 Fix s390 build failure Dave Jones
@ 2006-02-10 21:14 ` Linus Torvalds
2006-02-10 21:27 ` Dave Jones
0 siblings, 1 reply; 6+ messages in thread
From: Linus Torvalds @ 2006-02-10 21:14 UTC (permalink / raw)
To: Dave Jones; +Cc: Linux Kernel
On Fri, 10 Feb 2006, Dave Jones wrote:
>
> arch/s390/kernel/compat_signal.c:199: error: conflicting types for 'do_sigaction'
> include/linux/sched.h:1115: error: previous declaration of 'do_sigaction' was here
>
> Signed-off-by: Dave Jones <davej@redhat.com>
>
> --- linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c~ 2006-02-10 12:47:57.000000000 -0500
> +++ linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c 2006-02-10 12:48:05.000000000 -0500
> @@ -196,7 +196,7 @@ sys32_sigaction(int sig, const struct ol
> }
>
> int
> -do_sigaction(int sig, const struct k_sigaction *act, struct k_sigaction *oact);
> +do_sigaction(int sig, struct k_sigaction *act, struct k_sigaction *oact);
Umm. Shouldn't we just _remove_ the incorrect and unnecessary
extra declaration?
Linus
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Fix s390 build failure.
2006-02-10 21:14 ` Linus Torvalds
@ 2006-02-10 21:27 ` Dave Jones
2006-02-11 9:46 ` Heiko Carstens
0 siblings, 1 reply; 6+ messages in thread
From: Dave Jones @ 2006-02-10 21:27 UTC (permalink / raw)
To: Linus Torvalds; +Cc: Linux Kernel
arch/s390/kernel/compat_signal.c:199: error: conflicting types for 'do_sigaction'
include/linux/sched.h:1115: error: previous declaration of 'do_sigaction' was here
Signed-off-by: Dave Jones <davej@redhat.com>
--- linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c~ 2006-02-10 12:47:57.000000000 -0500
+++ linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c 2006-02-10 12:48:05.000000000 -0500
@@ -196,7 +196,4 @@ sys32_sigaction(int sig, const struct ol
}
-int
-do_sigaction(int sig, const struct k_sigaction *act, struct k_sigaction *oact);
-
asmlinkage long
sys32_rt_sigaction(int sig, const struct sigaction32 __user *act,
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Fix s390 build failure.
2006-02-10 21:27 ` Dave Jones
@ 2006-02-11 9:46 ` Heiko Carstens
2006-02-11 9:51 ` Andrew Morton
0 siblings, 1 reply; 6+ messages in thread
From: Heiko Carstens @ 2006-02-11 9:46 UTC (permalink / raw)
To: Dave Jones; +Cc: Linus Torvalds, Linux Kernel, Andrew Morton
On Fri, Feb 10, 2006 at 04:27:11PM -0500, Dave Jones wrote:
>
> arch/s390/kernel/compat_signal.c:199: error: conflicting types for 'do_sigaction'
> include/linux/sched.h:1115: error: previous declaration of 'do_sigaction' was here
>
> Signed-off-by: Dave Jones <davej@redhat.com>
>
> --- linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c~ 2006-02-10 12:47:57.000000000 -0500
> +++ linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c 2006-02-10 12:48:05.000000000 -0500
> @@ -196,7 +196,4 @@ sys32_sigaction(int sig, const struct ol
> }
>
> -int
> -do_sigaction(int sig, const struct k_sigaction *act, struct k_sigaction *oact);
> -
> asmlinkage long
> sys32_rt_sigaction(int sig, const struct sigaction32 __user *act,
> -
Hmm.. I sent the same patch earlier to Andrew already. To whom should I send
simple compile fixes? I thought I'm supposed to send all patches to Andrew
first?!
Anyway, the patch s390-compat-signal-compile-fix.patch can be removed from
the -mm tree.
Thanks,
Heiko
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Fix s390 build failure.
2006-02-11 9:46 ` Heiko Carstens
@ 2006-02-11 9:51 ` Andrew Morton
2006-02-11 9:59 ` Heiko Carstens
0 siblings, 1 reply; 6+ messages in thread
From: Andrew Morton @ 2006-02-11 9:51 UTC (permalink / raw)
To: Heiko Carstens; +Cc: davej, torvalds, linux-kernel
Heiko Carstens <heiko.carstens@de.ibm.com> wrote:
>
> On Fri, Feb 10, 2006 at 04:27:11PM -0500, Dave Jones wrote:
> >
> > arch/s390/kernel/compat_signal.c:199: error: conflicting types for 'do_sigaction'
> > include/linux/sched.h:1115: error: previous declaration of 'do_sigaction' was here
> >
> > Signed-off-by: Dave Jones <davej@redhat.com>
> >
> > --- linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c~ 2006-02-10 12:47:57.000000000 -0500
> > +++ linux-2.6.15.noarch/arch/s390/kernel/compat_signal.c 2006-02-10 12:48:05.000000000 -0500
> > @@ -196,7 +196,4 @@ sys32_sigaction(int sig, const struct ol
> > }
> >
> > -int
> > -do_sigaction(int sig, const struct k_sigaction *act, struct k_sigaction *oact);
> > -
> > asmlinkage long
> > sys32_rt_sigaction(int sig, const struct sigaction32 __user *act,
> > -
>
> Hmm.. I sent the same patch earlier to Andrew already. To whom should I send
> simple compile fixes?
You can copy Linus - sometimes he applies things ;) If not, I'll catch it.
> Anyway, the patch s390-compat-signal-compile-fix.patch can be removed from
> the -mm tree.
That's fine - I'll autodrop it next time I sync with Linus.
I've been a bit sluggish with the s390 patches while waiting for you and
Christoph to sort things out. I'm thinking of holding off on
s390-add-missing-validation-for-dasd-discipline-specific-ioctls.patch and
s390-cleanup-of-dasd-eer-module.patch.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Fix s390 build failure.
2006-02-11 9:51 ` Andrew Morton
@ 2006-02-11 9:59 ` Heiko Carstens
0 siblings, 0 replies; 6+ messages in thread
From: Heiko Carstens @ 2006-02-11 9:59 UTC (permalink / raw)
To: Andrew Morton; +Cc: davej, torvalds, linux-kernel
> I've been a bit sluggish with the s390 patches while waiting for you and
> Christoph to sort things out. I'm thinking of holding off on
> s390-add-missing-validation-for-dasd-discipline-specific-ioctls.patch and
> s390-cleanup-of-dasd-eer-module.patch.
Just don't send them for now. I hope the dasd guys here and Christoph will
come to a conclusion on how to go on with this soon.
Thanks,
Heiko
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2006-02-11 9:59 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-02-10 20:04 Fix s390 build failure Dave Jones
2006-02-10 21:14 ` Linus Torvalds
2006-02-10 21:27 ` Dave Jones
2006-02-11 9:46 ` Heiko Carstens
2006-02-11 9:51 ` Andrew Morton
2006-02-11 9:59 ` Heiko Carstens
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox