All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@infradead.org>
To: Ben Myers <bpm@sgi.com>
Cc: Mark Tinguely <tinguely@sgi.com>, xfs@oss.sgi.com
Subject: Re: [RFC] [PATCH 0/18] xfstests: move tests out of top level
Date: Tue, 28 Aug 2012 14:02:07 -0400	[thread overview]
Message-ID: <20120828180207.GA18356@infradead.org> (raw)
In-Reply-To: <20120828174304.GB3274@sgi.com>

On Tue, Aug 28, 2012 at 12:43:04PM -0500, Ben Myers wrote:
> I got the impression that Christoph is doing a review of this patch set as
> well.  If you wait for Christoph's review before moving bench it will give Phil
> a little time to come up with a plan for discussion.

I'll try to get to it quickly.  I shared Dave's opinion on the current
bench code, but I also agree that if SGI has it in use right now we
can't remove it.  I'm still not quite clear if you guys use it right now
or just maybe plan to use it real soon.  If it's the latter I'd be in
favour of removing it, if not we must keep it.

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

  reply	other threads:[~2012-08-28 18:01 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-26  9:27 [RFC] [PATCH 0/18] xfstests: move tests out of top level Dave Chinner
2012-07-26  9:27 ` [PATCH 01/18] xfstests: remove remake script Dave Chinner
2012-08-28 19:50   ` Christoph Hellwig
2012-07-26  9:27 ` [PATCH 02/18] xfstests: remove bench infrastructure Dave Chinner
2012-07-26  9:27 ` [PATCH 03/18] xfstests: kill useless test owner fields Dave Chinner
2012-08-28 19:51   ` Christoph Hellwig
2012-07-26  9:27 ` [PATCH 04/18] xfstests: remove stale machine configs Dave Chinner
2012-08-28 19:51   ` Christoph Hellwig
2012-07-26  9:27 ` [PATCH 05/18] xfstests: fold common into check Dave Chinner
2012-08-28 19:52   ` Christoph Hellwig
2012-07-26  9:28 ` [PATCH 06/18] xfstests: clean up and simply check CLI option parsing Dave Chinner
2012-08-28 19:52   ` Christoph Hellwig
2012-07-26  9:28 ` [PATCH 07/18] xfstests: kill hangcheck stuff from check Dave Chinner
2012-08-28 19:53   ` Christoph Hellwig
2012-07-26  9:28 ` [PATCH 08/18] xfstests: remove test expunge file support Dave Chinner
2012-08-28 19:54   ` Christoph Hellwig
2012-07-26  9:28 ` [PATCH 09/18] xfstests: remove undocumented TESTS_REMAINING_LOG Dave Chinner
2012-08-28 19:54   ` Christoph Hellwig
2012-07-26  9:28 ` [PATCH 10/18] xfstests: include test subdirectory support Dave Chinner
2012-07-26  9:28 ` [PATCH 11/18] xfstests: move generic tests out of top level dir Dave Chinner
2012-07-26  9:28 ` [PATCH 12/18] xfstests: move xfs specific tests out of top directory Dave Chinner
2012-07-26  9:28 ` [PATCH 13/18] xfstests: move remaining tests out of top level directory Dave Chinner
2012-07-26  9:28 ` [PATCH 14/18] xfstests: rework CLI individual test specification Dave Chinner
2012-07-26  9:28 ` [PATCH 15/18] xfstests: make exclude groups aware of multiple subdirectories Dave Chinner
2012-07-26  9:28 ` [PATCH 16/18] xfstests: Introduce a results directory Dave Chinner
2012-07-26  9:28 ` [PATCH 17/18] xfstests: convert tests to use new " Dave Chinner
2012-09-05 12:00   ` Boris Ranto
2012-09-05 23:04     ` Dave Chinner
2012-09-06 12:34       ` Boris Ranto
2012-09-06 23:14         ` Dave Chinner
2012-09-07 12:47           ` Boris Ranto
2012-07-26  9:28 ` [PATCH 18/18] xfstests: fix _link_out_file callers Dave Chinner
2012-08-14 21:39 ` [RFC] [PATCH 0/18] xfstests: move tests out of top level Dave Chinner
2012-08-15 17:23   ` Mark Tinguely
2012-08-20 21:27 ` Mark Tinguely
2012-08-20 22:43   ` Dave Chinner
2012-08-21 16:33     ` Ben Myers
2012-08-21 22:09       ` Dave Chinner
2012-08-22 19:16         ` Ben Myers
2012-08-22 23:42           ` Dave Chinner
2012-08-23 17:00             ` Ben Myers
2012-08-24  4:07               ` Dave Chinner
2012-08-28 17:43                 ` Ben Myers
2012-08-28 18:02                   ` Christoph Hellwig [this message]
2013-02-25 15:50               ` Eric Sandeen
2013-02-25 21:52                 ` Dave Chinner
2013-02-26  0:27                   ` Theodore Ts'o
2013-02-26  3:18                     ` Dave Chinner
2013-02-26  3:22                     ` Ben Myers

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=20120828180207.GA18356@infradead.org \
    --to=hch@infradead.org \
    --cc=bpm@sgi.com \
    --cc=tinguely@sgi.com \
    --cc=xfs@oss.sgi.com \
    /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 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.