From: Tobias Klauser <tklauser@distanz.ch>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Ley Foon Tan <lftan@altera.com>,
Linux-Arch <linux-arch@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
Chung-Lin Tang <cltang@codesourcery.com>,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH v3 01/29] asm-generic: add generic futex for !CONFIG_SMP
Date: Fri, 26 Sep 2014 15:17:46 +0200 [thread overview]
Message-ID: <20140926131746.GA3279@distanz.ch> (raw)
In-Reply-To: <3608975.MxU44B73lk@wuerfel>
On 2014-09-24 at 13:10:00 +0200, Arnd Bergmann <arnd@arndb.de> wrote:
> On Wednesday 24 September 2014 18:57:54 Ley Foon Tan wrote:
> > On Wed, Sep 24, 2014 at 6:40 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> > > On Wednesday 24 September 2014 18:18:50 Ley Foon Tan wrote:
> > >> On Tue, Sep 23, 2014 at 6:40 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> > >> > On Tuesday 23 September 2014 18:20:08 LF.Tan wrote:
> > >> >> Hi Arnd
> > >> >>
> > >> >> Are you okay with this generic futex in asm-generic?
> > >> >>
> > >> >
> > >> > Yes, this looks good. Remind me again who need this, would it make
> > >> > sense to merge this through an architecture-specific tree for
> > >> > whichever architecture can benefit from it?
> > >> What is the common practice for this?
> > >> Do you mean merge with nios2 or other arch, eg m68k?
> > >
> > > I mean do it as part of the nios2 patches.
> > I am okay with this.
> > Do I need to get ACK for all patches in order for nios2 merge into mainline?
> > Thanks.
> >
>
> Please add my 'Acked-by: Arnd Bergmann <arnd@arndb.de>' to this patch
> once Thomas is ok with it. I think we had concluded already that it
> was correct when we talked about it last time, but he doesn't seem
> convinced yet so I want to wait for his reply.
>
> You definitely need an Ack for all patches touching code outside of
> arch/nios2, from the respective maintainers. For the nios2 code,
> having as many acks as you can helps, and I should probably take
> a last look so I can ack the pull request when you send it.
Speaking of the nios2 part: I'm the author of some parts of this code,
but now also reviewed this series (or rather, intend to do so for the
few remaining patches I didn't review yet).
What is the appropriate way to account for this? Is Reviewed-By still OK? I
guess I won't be considered an entirely neutral reviewer ;)
Cheers
Tobias
next prev parent reply other threads:[~2014-09-26 13:17 UTC|newest]
Thread overview: 97+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-08 9:22 [PATCH v3 00/29] nios2 Linux kernel port Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 01/29] asm-generic: add generic futex for !CONFIG_SMP Ley Foon Tan
2014-09-23 10:20 ` LF.Tan
2014-09-23 10:20 ` LF.Tan
2014-09-23 10:40 ` Arnd Bergmann
2014-09-23 10:40 ` Arnd Bergmann
2014-09-24 10:18 ` Ley Foon Tan
2014-09-24 10:18 ` Ley Foon Tan
2014-09-24 10:40 ` Arnd Bergmann
2014-09-24 10:40 ` Arnd Bergmann
2014-09-24 10:57 ` Ley Foon Tan
2014-09-24 11:10 ` Arnd Bergmann
2014-09-24 11:10 ` Arnd Bergmann
2014-09-25 8:33 ` Ley Foon Tan
2014-09-25 8:33 ` Ley Foon Tan
2014-09-25 10:54 ` Arnd Bergmann
2014-09-25 10:54 ` Arnd Bergmann
2014-09-26 2:22 ` Ley Foon Tan
2014-09-26 2:22 ` Ley Foon Tan
2014-09-26 6:41 ` Geert Uytterhoeven
2014-09-26 6:41 ` Geert Uytterhoeven
2014-09-26 8:30 ` Ley Foon Tan
2014-09-26 13:17 ` Tobias Klauser [this message]
2014-09-26 13:17 ` Tobias Klauser
2014-09-26 13:33 ` Arnd Bergmann
2014-09-26 13:33 ` Arnd Bergmann
2014-09-23 21:47 ` Thomas Gleixner
2014-09-23 21:47 ` Thomas Gleixner
2014-09-24 10:29 ` Ley Foon Tan
2014-09-24 10:29 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 02/29] nios2: Assembly macros and definitions Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 03/29] nios2: Kernel booting and initialization Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 04/29] nios2: Exception handling Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 05/29] nios2: Traps exception handling Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 06/29] nios2: Memory management Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 07/29] nios2: I/O Mapping Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 08/29] nios2: MMU Fault handling Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 09/29] nios2: Page table management Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 10/29] nios2: Process management Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 11/29] nios2: Cache handling Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 12/29] nios2: TLB handling Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 13/29] nios2: Interrupt handling Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 14/29] nios2: DMA mapping API Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 15/29] Add ELF machine define for Nios2 Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 16/29] nios2: ELF definitions Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 17/29] nios2: System calls handling Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 18/29] nios2: Signal handling support Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 19/29] nios2: Library functions Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 20/29] nios2: Device tree support Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 21/29] nios2: Time keeping Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 22/29] nios2: Cpuinfo handling Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 23/29] nios2: Miscellaneous header files Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 24/29] nios2: Nios2 registers Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 25/29] nios2: Module support Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-09 21:25 ` Valdis.Kletnieks
2014-09-10 7:29 ` Chung-Lin Tang
2014-09-10 7:29 ` Chung-Lin Tang
2014-09-10 7:49 ` Tobias Klauser
2014-09-08 9:22 ` [PATCH v3 26/29] nios2: ptrace support Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-16 9:43 ` Tobias Klauser
2014-09-16 9:43 ` Tobias Klauser
2014-09-18 5:10 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 27/29] MAINTAINERS: Add nios2 maintainer Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 28/29] Documentation: Add documentation for Nios2 architecture Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-26 2:27 ` Ley Foon Tan
2014-09-26 2:27 ` Ley Foon Tan
2014-09-26 8:31 ` Ley Foon Tan
2014-09-08 9:22 ` [PATCH v3 29/29] nios2: Build infrastructure Ley Foon Tan
2014-09-08 9:22 ` Ley Foon Tan
2014-09-08 12:13 ` [PATCH v3 00/29] nios2 Linux kernel port David Howells
2014-09-09 2:02 ` Al Viro
2014-09-09 2:02 ` Al Viro
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=20140926131746.GA3279@distanz.ch \
--to=tklauser@distanz.ch \
--cc=arnd@arndb.de \
--cc=cltang@codesourcery.com \
--cc=lftan@altera.com \
--cc=linux-arch@vger.kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=tglx@linutronix.de \
/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).