All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] parisc: remove gratuitous cpu_online_map declaration.
@ 2008-12-03 23:42 Rusty Russell
  2008-12-09  3:54 ` Kyle McMartin
  0 siblings, 1 reply; 5+ messages in thread
From: Rusty Russell @ 2008-12-03 23:42 UTC (permalink / raw)
  To: Grant Grundler; +Cc: linux-kernel, Stephen Rothwell

This is defined in linux/cpumask.h (included in this file already),
and this will break when the definition changes (in a following patch).

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
---
 arch/parisc/include/asm/smp.h |    2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/parisc/include/asm/smp.h b/arch/parisc/include/asm/smp.h
--- a/arch/parisc/include/asm/smp.h
+++ b/arch/parisc/include/asm/smp.h
@@ -15,8 +15,6 @@
 #include <linux/threads.h>	/* for NR_CPUS */
 #include <linux/cpumask.h>
 typedef unsigned long address_t;
-
-extern cpumask_t cpu_online_map;
 
 
 /*



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] parisc: remove gratuitous cpu_online_map declaration.
  2008-12-03 23:42 [PATCH] parisc: remove gratuitous cpu_online_map declaration Rusty Russell
@ 2008-12-09  3:54 ` Kyle McMartin
  2008-12-09 22:06   ` Rusty Russell
  0 siblings, 1 reply; 5+ messages in thread
From: Kyle McMartin @ 2008-12-09  3:54 UTC (permalink / raw)
  To: Rusty Russell
  Cc: Grant Grundler, linux-kernel, Stephen Rothwell, linux-parisc

On Thu, Dec 04, 2008 at 10:12:41AM +1030, Rusty Russell wrote:
> This is defined in linux/cpumask.h (included in this file already),
> and this will break when the definition changes (in a following patch).
> 
> Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

Accepted, though it would have been nice if it had either hit all the
maintainers, or hit the list too...

cheers, Kyle

> ---
>  arch/parisc/include/asm/smp.h |    2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/arch/parisc/include/asm/smp.h b/arch/parisc/include/asm/smp.h
> --- a/arch/parisc/include/asm/smp.h
> +++ b/arch/parisc/include/asm/smp.h
> @@ -15,8 +15,6 @@
>  #include <linux/threads.h>	/* for NR_CPUS */
>  #include <linux/cpumask.h>
>  typedef unsigned long address_t;
> -
> -extern cpumask_t cpu_online_map;
>  
>  
>  /*
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] parisc: remove gratuitous cpu_online_map declaration.
  2008-12-09  3:54 ` Kyle McMartin
@ 2008-12-09 22:06   ` Rusty Russell
  2008-12-09 22:44     ` Grant Grundler
  0 siblings, 1 reply; 5+ messages in thread
From: Rusty Russell @ 2008-12-09 22:06 UTC (permalink / raw)
  To: Kyle McMartin, Stephen Rothwell; +Cc: Grant Grundler, linux-parisc

On Tuesday 09 December 2008 14:24:07 Kyle McMartin wrote:
> On Thu, Dec 04, 2008 at 10:12:41AM +1030, Rusty Russell wrote:
> > This is defined in linux/cpumask.h (included in this file already),
> > and this will break when the definition changes (in a following patch).
> > 
> > Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
> 
> Accepted, though it would have been nice if it had either hit all the
> maintainers, or hit the list too...

OK, I picked one maintainer at semi-random.   I've added you and Willy
to my annoy-maintainers script.

Thanks,
Rusty.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] parisc: remove gratuitous cpu_online_map declaration.
  2008-12-09 22:06   ` Rusty Russell
@ 2008-12-09 22:44     ` Grant Grundler
  2008-12-09 23:11       ` Helge Deller
  0 siblings, 1 reply; 5+ messages in thread
From: Grant Grundler @ 2008-12-09 22:44 UTC (permalink / raw)
  To: Rusty Russell
  Cc: Kyle McMartin, Stephen Rothwell, Grant Grundler, linux-parisc

On Wed, Dec 10, 2008 at 08:36:19AM +1030, Rusty Russell wrote:
> On Tuesday 09 December 2008 14:24:07 Kyle McMartin wrote:
> > On Thu, Dec 04, 2008 at 10:12:41AM +1030, Rusty Russell wrote:
> > > This is defined in linux/cpumask.h (included in this file already),
> > > and this will break when the definition changes (in a following patch).
> > > 
> > > Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
> > 
> > Accepted, though it would have been nice if it had either hit all the
> > maintainers, or hit the list too...
> 
> OK, I picked one maintainer at semi-random.   I've added you and Willy
> to my annoy-maintainers script.

Rusty,
willy and I should be dropped as parisc-linux maintainers.
(We can review parisc patches, just not listed as maintainers.)

Please add Helge Deller <deller@gmx.net> (IIRC).

Patch to MAINTAINERS was submitted just a week ago or so.

cheers,
grant

> 
> Thanks,
> Rusty.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-parisc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] parisc: remove gratuitous cpu_online_map declaration.
  2008-12-09 22:44     ` Grant Grundler
@ 2008-12-09 23:11       ` Helge Deller
  0 siblings, 0 replies; 5+ messages in thread
From: Helge Deller @ 2008-12-09 23:11 UTC (permalink / raw)
  To: Rusty Russell
  Cc: Grant Grundler, Kyle McMartin, Stephen Rothwell, linux-parisc

Grant Grundler wrote:
> On Wed, Dec 10, 2008 at 08:36:19AM +1030, Rusty Russell wrote:
>> On Tuesday 09 December 2008 14:24:07 Kyle McMartin wrote:
>>> On Thu, Dec 04, 2008 at 10:12:41AM +1030, Rusty Russell wrote:
>>>> This is defined in linux/cpumask.h (included in this file already),
>>>> and this will break when the definition changes (in a following patch).
>>>>
>>>> Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
>>> Accepted, though it would have been nice if it had either hit all the
>>> maintainers, or hit the list too...
>> OK, I picked one maintainer at semi-random.   I've added you and Willy
>> to my annoy-maintainers script.
> 
> Rusty,
> willy and I should be dropped as parisc-linux maintainers.
> (We can review parisc patches, just not listed as maintainers.)
> 
> Please add Helge Deller <deller@gmx.net> (IIRC).

Rusty,

Please use: Helge Deller <deller@gmx.de>

Thanks,
Helge

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2008-12-09 23:11 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-12-03 23:42 [PATCH] parisc: remove gratuitous cpu_online_map declaration Rusty Russell
2008-12-09  3:54 ` Kyle McMartin
2008-12-09 22:06   ` Rusty Russell
2008-12-09 22:44     ` Grant Grundler
2008-12-09 23:11       ` Helge Deller

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.