linux-arch.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Howells <dhowells@redhat.com>
To: Russell King <rmk+lkml@arm.linux.org.uk>
Cc: dhowells@redhat.com,
	Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>,
	Andrew Morton <akpm@linux-foundation.org>,
	Nicolas Pitre <nico@cam.org>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Ingo Molnar <mingo@elte.hu>,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	linux-kernel@vger.kernel.org, Ralf Baechle <ralf@linux-mips.org>,
	benh@kernel.crashing.org, paulus@samba.org,
	David Miller <davem@davemloft.net>,
	Ingo Molnar <mingo@redhat.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	Steven Rostedt <rostedt@goodmis.org>,
	linux-arch@vger.kernel.org
Subject: Re: [RFC patch 08/18] cnt32_to_63 should use smp_rmb()
Date: Fri, 07 Nov 2008 23:41:55 +0000	[thread overview]
Message-ID: <9405.1226101315@redhat.com> (raw)
In-Reply-To: <20081107221813.GA18378@flint.arm.linux.org.uk>

Russell King <rmk+lkml@arm.linux.org.uk> wrote:

> Well, that's where it was - private to ARM.  Then David Howells came
> along and unilaterally - and without reference to anyone as far as I
> can see - moved it to include/linux.
> 
> Neither Nicolas, nor me had any idea that it was going to move into
> include/linux - the first we knew of it was when pulling the change
> from Linus' tree.
> 
> Look, if people in the kernel community can't or won't communicate
> with others (either through malice, purpose or accident), you can
> expect this kind of crap to happen.

Excuse me, Russell, but I sent Nicolas an email prior to doing so asking him
if he had any objections:

	To: Nicolas Pitre <nico@cam.org>
	cc: dhowells@redhat.com
	Subject: Moving asm-arm/cnt32_to_63.h to include/linux/
	Date: Thu, 31 Jul 2008 16:04:04 +0100

	Hi Nicolas,

	Mind if I move include/asm-arm/cnt32_to_63.h to include/linux/?

	I need to use it for MN10300.

	David

He didn't respond.  Not only that, but I copied Nicolas on the patch to make
the move and the patch to make MN10300 use it when I submitted it to Linus on
the 24th September, so it's not like he didn't have plenty of time.  He
certainly saw that because he joined in the discussion of the second patch.
Furthermore, he could've asked Linus to refuse the patch, or to revert it if
it had already gone in.

I suppose I should've cc'd the ARM list too... but why should it adversely
affect ARM?

David

  parent reply	other threads:[~2008-11-07 23:43 UTC|newest]

Thread overview: 124+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20081107052336.652868737@polymtl.ca>
2008-11-07  5:23 ` [RFC patch 01/18] get_cycles() : kconfig HAVE_GET_CYCLES Mathieu Desnoyers
2008-11-07  5:23 ` [RFC patch 02/18] get_cycles() : x86 HAVE_GET_CYCLES Mathieu Desnoyers
2008-11-07  5:23 ` [RFC patch 03/18] get_cycles() : sparc64 HAVE_GET_CYCLES Mathieu Desnoyers
2008-11-07  5:23 ` [RFC patch 04/18] get_cycles() : powerpc64 HAVE_GET_CYCLES Mathieu Desnoyers
2008-11-07 14:56   ` Josh Boyer
2008-11-07 18:14     ` Mathieu Desnoyers
2008-11-07  5:23 ` [RFC patch 05/18] get_cycles() : MIPS HAVE_GET_CYCLES_32 Mathieu Desnoyers
2008-11-07  5:23 ` [RFC patch 06/18] Trace clock generic Mathieu Desnoyers
2008-11-07  5:23 ` [RFC patch 07/18] Trace clock core Mathieu Desnoyers
2008-11-07  5:52   ` Andrew Morton
2008-11-07  6:16     ` Mathieu Desnoyers
2008-11-07  6:26       ` Andrew Morton
2008-11-07 16:12         ` Mathieu Desnoyers
2008-11-07 16:19           ` Andrew Morton
2008-11-07 18:16             ` Mathieu Desnoyers
2008-11-07  5:23 ` [RFC patch 08/18] cnt32_to_63 should use smp_rmb() Mathieu Desnoyers
2008-11-07  6:05   ` Andrew Morton
2008-11-07  8:12     ` Nicolas Pitre
2008-11-07  8:38       ` Andrew Morton
2008-11-07 11:20       ` David Howells
2008-11-07 15:01         ` Nicolas Pitre
2008-11-07 15:01           ` Nicolas Pitre
2008-11-07 15:50           ` Andrew Morton
2008-11-07 16:47             ` Nicolas Pitre
2008-11-07 17:21               ` Andrew Morton
2008-11-07 20:03                 ` Nicolas Pitre
2008-11-07 20:03                   ` Nicolas Pitre
2008-11-07 16:55             ` David Howells
2008-11-07 16:21           ` David Howells
2008-11-07 16:29             ` Andrew Morton
2008-11-07 17:10             ` David Howells
2008-11-07 17:26               ` Andrew Morton
2008-11-07 18:00                 ` Mathieu Desnoyers
2008-11-07 18:21                   ` Andrew Morton
2008-11-07 18:30                     ` Harvey Harrison
2008-11-07 18:42                       ` Mathieu Desnoyers
2008-11-07 18:33                     ` Mathieu Desnoyers
2008-11-07 18:36                     ` Linus Torvalds
2008-11-07 18:45                       ` Andrew Morton
2008-11-07 16:07         ` David Howells
2008-11-07 16:47         ` Mathieu Desnoyers
2008-11-07 20:11           ` Russell King
2008-11-07 20:11             ` Russell King
2008-11-07 21:36             ` Mathieu Desnoyers
2008-11-07 21:36               ` Mathieu Desnoyers
2008-11-07 22:18               ` Russell King
2008-11-07 22:18                 ` Russell King
2008-11-07 22:36                 ` Mathieu Desnoyers
2008-11-07 22:36                   ` Mathieu Desnoyers
2008-11-07 23:41               ` David Howells [this message]
2008-11-08  0:15                 ` Russell King
2008-11-08  0:15                   ` Russell King
2008-11-08 15:24                   ` Nicolas Pitre
2008-11-08 23:20                     ` [PATCH] clarify usage expectations for cnt32_to_63() Nicolas Pitre
2008-11-09  2:25                       ` Mathieu Desnoyers
2008-11-09  2:54                         ` Nicolas Pitre
2008-11-09  5:06                           ` Nicolas Pitre
2008-11-09  5:27                             ` [PATCH v2] " Nicolas Pitre
2008-11-09  6:48                               ` Mathieu Desnoyers
2008-11-09 13:34                                 ` Nicolas Pitre
2008-11-09 13:43                                   ` Russell King
2008-11-09 13:43                                     ` Russell King
2008-11-09 16:22                                   ` Mathieu Desnoyers
2008-11-10  4:20                                     ` Nicolas Pitre
2008-11-10  4:42                                       ` Andrew Morton
2008-11-10 21:34                                         ` Nicolas Pitre
2008-11-10 21:58                                           ` Andrew Morton
2008-11-10 23:15                                             ` Nicolas Pitre
2008-11-10 23:15                                               ` Nicolas Pitre
2008-11-10 23:22                                               ` Andrew Morton
2008-11-10 23:38                                                 ` Steven Rostedt
2008-11-11  0:26                                                 ` Nicolas Pitre
2008-11-11 18:28                                                   ` [PATCH] convert cnt32_to_63 to inline Mathieu Desnoyers
2008-11-11 19:13                                                     ` Russell King
2008-11-11 19:13                                                       ` Russell King
2008-11-11 20:11                                                       ` Mathieu Desnoyers
2008-11-11 20:11                                                         ` Mathieu Desnoyers
2008-11-11 21:51                                                         ` Russell King
2008-11-11 21:51                                                           ` Russell King
2008-11-12  3:48                                                           ` Mathieu Desnoyers
2008-11-12  3:48                                                             ` Mathieu Desnoyers
2008-11-11 21:00                                                     ` Nicolas Pitre
2008-11-11 21:13                                                       ` Russell King
2008-11-11 21:13                                                         ` Russell King
2008-11-11 22:31                                                   ` David Howells
2008-11-11 22:37                                                     ` Peter Zijlstra
2008-11-12  1:13                                                       ` Steven Rostedt
2008-11-08  0:45                 ` [RFC patch 08/18] cnt32_to_63 should use smp_rmb() David Howells
2008-11-07 17:04         ` David Howells
2008-11-07 17:17           ` Mathieu Desnoyers
2008-11-07 23:27           ` David Howells
2008-11-07 11:03     ` David Howells
2008-11-07 11:03       ` David Howells
2008-11-07 16:51       ` Mathieu Desnoyers
2008-11-07 20:18         ` Nicolas Pitre
2008-11-07 23:55         ` David Howells
2008-11-07 10:59   ` David Howells
2008-11-07  5:23 ` [RFC patch 09/18] Powerpc : Trace clock Mathieu Desnoyers
2008-11-07  5:23 ` [RFC patch 10/18] Sparc64 " Mathieu Desnoyers
2008-11-07  5:45   ` David Miller
2008-11-07 10:55 ` [RFC patch 08/18] cnt32_to_63 should use smp_rmb() David Howells
2008-11-07 17:09   ` Mathieu Desnoyers
2008-11-07 17:33     ` Steven Rostedt
2008-11-07 17:33       ` Steven Rostedt
2008-11-07 19:18       ` Mathieu Desnoyers
2008-11-07 19:32         ` Peter Zijlstra
2008-11-07 20:02           ` Mathieu Desnoyers
2008-11-07 20:45             ` Mathieu Desnoyers
2008-11-07 20:54               ` Paul E. McKenney
2008-11-07 21:04                 ` Steven Rostedt
2008-11-08  0:34                   ` Paul E. McKenney
2008-11-07 21:16                 ` Mathieu Desnoyers
2008-11-07 20:08     ` Steven Rostedt
2008-11-07 20:55       ` Paul E. McKenney
2008-11-07 21:27       ` Mathieu Desnoyers
2008-11-07 20:36     ` Nicolas Pitre
2008-11-07 20:55       ` Mathieu Desnoyers
2008-11-07 21:22         ` Nicolas Pitre
2008-11-07 23:50     ` David Howells
2008-11-08  0:55       ` Steven Rostedt
2008-11-09 11:51       ` David Howells
2008-11-09 14:31         ` Steven Rostedt
2008-11-09 14:31           ` Steven Rostedt
2008-11-09 16:18         ` Mathieu Desnoyers

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=9405.1226101315@redhat.com \
    --to=dhowells@redhat.com \
    --cc=a.p.zijlstra@chello.nl \
    --cc=akpm@linux-foundation.org \
    --cc=benh@kernel.crashing.org \
    --cc=davem@davemloft.net \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mathieu.desnoyers@polymtl.ca \
    --cc=mingo@elte.hu \
    --cc=mingo@redhat.com \
    --cc=nico@cam.org \
    --cc=paulus@samba.org \
    --cc=ralf@linux-mips.org \
    --cc=rmk+lkml@arm.linux.org.uk \
    --cc=rostedt@goodmis.org \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).