LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Sam Ravnborg <sam@ravnborg.org>
To: ananth@in.ibm.com
Cc: Christoph Hellwig <hch@infradead.org>,
	linux-kernel@vger.kernel.org, mingo@elte.hu, ak@suse.de,
	arjan@linux.intel.com, Andrew Morton <akpm@linux-foundation.org>,
	Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [GIT PULL?] Create and populate toplevel tests/ for kernel tests
Date: Tue, 19 Feb 2008 20:21:53 +0100	[thread overview]
Message-ID: <20080219192153.GA11948@uranus.ravnborg.org> (raw)
In-Reply-To: <20080212223918.GA3927@uranus.ravnborg.org>

Hi Anath.

Linus did not pull this in the -rc1 to -rc2 timeframe
so please resubmit the patch serie one week into the
next merge window (when most of the trees has hit linus' tree
and Andrew has made his first merge).

IF you need an extra eye balling then you can submit
a few weeks before the merge window opens.
Thats typical after an -rc with only a few patches.

Thanks,
	Sam

On Tue, Feb 12, 2008 at 11:39:18PM +0100, Sam Ravnborg wrote:
> Hi Linus.
> 
> Will you consider such a primary code-movement for -rc1
> or shall we wait until next merge window?
> 
> Had we hit -rc2 I would not have sent this pull req and
> feel free to flame me anyway.
> 
> The rationale to get it merged is obviously to avoid
> merge conflicts and the only reason I ask is that I consider
> it a low risk patch.
> 
> I have not included 8/8 since it was questioned and it
> will wait until next merge window. But the first 7 was
> straightforward.
> 
> You can pull from:
> ssh://master.kernel.org/pub/scm/linux/kernel/git/sam/tests.git
> 
> diffstat and shortlog below.
> I also included mail last with a few of the merge related comments.
> 
> 	Sam
> 
>  Makefile                                        |    1 +
>  drivers/misc/Makefile                           |    1 -
>  kernel/Makefile                                 |    4 -
>  lib/Kconfig.debug                               |   71 +--------------------
>  lib/Makefile                                    |    1 -
>  tests/Kconfig                                   |   79 +++++++++++++++++++++++
>  tests/Makefile                                  |   10 +++
>  {kernel => tests}/backtracetest.c               |    0 
>  {drivers/misc => tests}/lkdtm.c                 |   12 ++--
>  {lib => tests}/locking-selftest-hardirq.h       |    0 
>  {lib => tests}/locking-selftest-mutex.h         |    0 
>  {lib => tests}/locking-selftest-rlock-hardirq.h |    0 
>  {lib => tests}/locking-selftest-rlock-softirq.h |    0 
>  {lib => tests}/locking-selftest-rlock.h         |    0 
>  {lib => tests}/locking-selftest-rsem.h          |    0 
>  {lib => tests}/locking-selftest-softirq.h       |    0 
>  {lib => tests}/locking-selftest-spin-hardirq.h  |    0 
>  {lib => tests}/locking-selftest-spin-softirq.h  |    0 
>  {lib => tests}/locking-selftest-spin.h          |    0 
>  {lib => tests}/locking-selftest-wlock-hardirq.h |    0 
>  {lib => tests}/locking-selftest-wlock-softirq.h |    0 
>  {lib => tests}/locking-selftest-wlock.h         |    0 
>  {lib => tests}/locking-selftest-wsem.h          |    0 
>  {lib => tests}/locking-selftest.c               |    0 
>  {kernel => tests}/rcutorture.c                  |    0 
>  {kernel => tests}/rtmutex-tester.c              |    2 +-
>  {kernel => tests}/test_kprobes.c                |    0 
>  27 files changed, 99 insertions(+), 82 deletions(-)
> 
> Ananth N Mavinakayanahalli (7):
>       Create tests/ directory
>       Move locking selftests to tests/
>       Move rcutorture to tests/
>       Move rtmutex-tests to tests/
>       Move lkdtm to tests/
>       Move kprobes smoke tests to tests/
>       Move backtrace tests to tests/
> 
> 
> 
> On Tue, Feb 12, 2008 at 01:22:46PM -0800, Andrew Morton wrote:
> > On Tue, 12 Feb 2008 11:44:52 -0500
> > Christoph Hellwig <hch@infradead.org> wrote:
> > 
> > > On Mon, Feb 11, 2008 at 04:14:52PM +0530, Ananth N Mavinakayanahalli wrote:
> > > > The following series of patches create and populate the toplevel tests/
> > > > directory. This will henceforth be the place where all in-kernel tests
> > > > live.
> > > > 
> > > > All patches against 2.6.25-rc1 and are just code movement without any
> > > > change in functionality.
> > > 
> > > ACK to patches 1-7, and I agree with Ingo that the x86-specific test
> > > should stay under arch/x86.
> > 
> > OK.  But now is basically the worst time for me (or anyone else) to merge
> > large code-motion changes like this, because they need to be carried for
> > two months or more.
> > 
> > And even though git can track renames, putting them into a git tree (say,
> > git-kbuild) won't help, because if some other git tree tries to modify a
> > file in its original place, I get to fix up the fallout.
> > 
> > Which I _could_ do, and would do if the patches were particularly risky or
> > added/changed functionality or whatever.  But they don't do that, and there
> > is little advantage in maintaining them for the >2 months.
> > 
> > So.  Please redo and resend the patches when we hit 2.6.25-rc6 or so?
> > 
> > Thanks.
> > 
> > (linux-next will largely fix all this: git will take care of the renames
> > and I'll just base the -mm queue on the consolidated linux-next.  But we
> > aren't there yet).
> > 
> --
> 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/

  parent reply	other threads:[~2008-02-19 19:22 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-11 10:44 [PATCH 0/8] " Ananth N Mavinakayanahalli
2008-02-11 10:45 ` [PATCH 1/8] Create tests/ directory Ananth N Mavinakayanahalli
2008-02-11 10:46 ` [PATCH 2/8] Move locking selftests to tests/ Ananth N Mavinakayanahalli
2008-02-11 10:48 ` [PATCH 3/8] Move rcutorture " Ananth N Mavinakayanahalli
2008-02-11 10:49 ` [PATCH 4/8] Move rtmutex-tests " Ananth N Mavinakayanahalli
2008-02-11 10:50 ` [PATCH 5/8] Move lkdtm " Ananth N Mavinakayanahalli
2008-02-11 10:51 ` [PATCH 6/8] Move kprobes smoke tests " Ananth N Mavinakayanahalli
2008-02-11 10:51 ` [PATCH 7/8] Move backtrace " Ananth N Mavinakayanahalli
2008-02-11 15:00   ` Arjan van de Ven
2008-02-11 10:52 ` [PATCH 8/8] Move RODATA and NX " Ananth N Mavinakayanahalli
2008-02-11 14:42   ` Ingo Molnar
2008-02-11 15:49     ` Ananth N Mavinakayanahalli
2008-02-11 15:01   ` Arjan van de Ven
2008-02-11 18:26 ` [PATCH 0/8] Create and populate toplevel tests/ for kernel tests Randy Dunlap
2008-02-11 18:30   ` Randy Dunlap
2008-02-12 16:44 ` Christoph Hellwig
2008-02-12 21:22   ` Andrew Morton
2008-02-12 21:34     ` Arjan van de Ven
2008-02-12 22:39     ` [GIT PULL?] " Sam Ravnborg
2008-02-12 22:47       ` Arjan van de Ven
2008-02-14  7:47       ` Ingo Molnar
2008-02-19 19:21       ` Sam Ravnborg [this message]
2008-02-20 15:56         ` Ananth N Mavinakayanahalli
2008-02-22  3:42         ` Ananth N Mavinakayanahalli
2008-02-22  4:57           ` Stephen Rothwell
2008-02-22  5:03             ` Ananth N Mavinakayanahalli

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=20080219192153.GA11948@uranus.ravnborg.org \
    --to=sam@ravnborg.org \
    --cc=ak@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=ananth@in.ibm.com \
    --cc=arjan@linux.intel.com \
    --cc=hch@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=torvalds@linux-foundation.org \
    --subject='Re: [GIT PULL?] Create and populate toplevel tests/ for kernel tests' \
    /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

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).