All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Chinner <dgc@sgi.com>
To: Justin Piszcz <jpiszcz@lucidpixels.com>
Cc: xaiki@sgi.com, xfs@oss.sgi.com
Subject: Re: [[PATCH, RESEND]] less AGs for single disks configs.
Date: Tue, 13 Nov 2007 07:31:47 +1100	[thread overview]
Message-ID: <20071112203147.GA995458@sgi.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0711120956340.2701@p34.internal.lan>

On Mon, Nov 12, 2007 at 09:57:21AM -0500, Justin Piszcz wrote:
> On Mon, 12 Nov 2007, David Chinner wrote:
> >A single spindle, regardless of it's size, will have similar
> >seek characteristics so scaling the number of AGs with size
> >is the wrong thing to do - you don't get better parallelism
> >out of a single spindle, just more seeks and lower performance.
> >hence keeping the number of AGs fixed up to the point where
> >the AG size tops out (i.e. 4TB) seems like a better scaling
> >factor to me. i.e. something like:
> >
> >
> >	if (!multidisk) {
> >		if (dblocks >= TERABYTES(4, blocklog)) {
> >			blocks = XFS_AG_MAX_BLOCKS(blocklog);
> >			goto done;
> >		}
> >		agcount = 4;
> >		/* work out ag size here */
> >		goto done;
> >	}
> >
> >I'd also like to see some test results showing the mkfs output
> >for the different configurations to confirm it works correctly
> >(i.e. that the corner cases work correctly).
> 
> Dave, when this is put into place do you recommend people re-format their 
> XFS partitions for those with a 750GiB drive -or- with a < 2TB RAID5 
> array,

No. If you are having performance problems, then changing the way the
filesystem is laid out *may* improve performance but if everything
is working fine then don't change it.

> would one see any increase in speed?

On a single disk, yes. On RAID5 - who knows. There are so many other
variables to raid5 performance (esp software raid) that such single disk
optimisations could degrade performance. On other RAID hardware, it
might improve - it really depends on the RAID implementation....

Cheers,

Dave.
-- 
Dave Chinner
Principal Engineer
SGI Australian Software Group

  reply	other threads:[~2007-11-12 20:32 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-29  7:56 Default mount options (that suck less) Niv Sardi
2007-10-29  8:55 ` David Chinner
2007-10-29 10:44   ` nscott
2007-10-29 14:01   ` Eric Sandeen
2007-10-29 21:26     ` David Chinner
2007-10-29 14:03 ` Eric Sandeen
2007-10-29 15:05   ` Hannes Dorbath
2007-10-29 15:07     ` Eric Sandeen
2007-10-30  8:40       ` Stewart Smith
2007-10-31  4:11         ` Nathan Scott
2007-10-31  4:13           ` Eric Sandeen
2007-10-29 21:05   ` David Chinner
2007-10-29 15:26 ` Eric Sandeen
2007-10-29 15:44   ` Chris Wedgwood
2007-10-29 15:51     ` Eric Sandeen
2007-10-30  0:45   ` Timothy Shimmin
2007-10-31 11:05     ` James Braid
2007-10-31 11:27       ` Justin Piszcz
2007-11-01  0:47         ` James Braid
2007-10-31 15:21       ` Eric Sandeen
2007-10-31 15:41       ` Chris Wedgwood
2007-11-01  0:32         ` James Braid
2007-10-29 23:48 ` David Chinner
2007-10-31 23:35 ` Niv Sardi
2007-10-31 23:40   ` Niv Sardi
2007-11-01  1:17     ` Niv Sardi
2007-11-01  2:27   ` Eric Sandeen
2007-11-12  2:28   ` Niv Sardi
2007-11-12  3:10     ` David Chinner
2007-11-12  3:48   ` [[PATCH, RESEND]] Default to log version 2 xaiki
2007-11-12  3:48     ` [[PATCH, RESEND]] Default to version 2 attributes xaiki
2007-11-12  3:48       ` [[PATCH, RESEND]] Drop the ability to turn unwritten extents off completly xaiki
2007-11-12  3:48         ` [[PATCH, RESEND]] V2 inodes per default, and move DFL bits to XFS_DFL_SB_VERSION_BITS, xaiki
2007-11-12  3:48           ` [[PATCH, RESEND]] reduce imaxpct for big filesystems, xaiki
2007-11-12  3:48             ` [[PATCH, RESEND]] less AGs for single disks configs xaiki
2007-11-12  9:01               ` David Chinner
2007-11-12 14:57                 ` Justin Piszcz
2007-11-12 20:31                   ` David Chinner [this message]
2007-11-12  6:33             ` [[PATCH, RESEND]] reduce imaxpct for big filesystems, David Chinner
2007-11-12  6:31           ` [[PATCH, RESEND]] V2 inodes per default, and move DFL bits to XFS_DFL_SB_VERSION_BITS, David Chinner
2007-11-13  0:51             ` Niv Sardi
2007-11-12  6:27         ` [[PATCH, RESEND]] Drop the ability to turn unwritten extents off completly David Chinner
2007-11-12  6:24       ` [[PATCH, RESEND]] Default to version 2 attributes David Chinner
2007-11-12  6:23     ` [[PATCH, RESEND]] Default to log version 2 David Chinner
2007-11-13  4:10 ` RESEND(2) xaiki
2007-11-13  4:10   ` [PATCH TAKE 2 1/6] Default to log version 2 xaiki
2007-11-13  4:10     ` [PATCH TAKE 2 2/6] Default to version 2 attributes xaiki
2007-11-13  4:10       ` [PATCH TAKE 2 3/6] Drop the ability to turn unwritten extents off completly xaiki
2007-11-13  4:10         ` [PATCH TAKE 2 4/6] V2 inodes per default, and move DFL bits to XFS_DFL_SB_VERSION_BITS, xaiki
2007-11-13  4:10           ` [PATCH TAKE 2 5/6] reduce imaxpct for big filesystems, xaiki
2007-11-13  4:10             ` [PATCH TAKE 2 6/6] less AGs for single disks configs xaiki
2007-11-13  4:47   ` RESEND(2) David Chinner

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=20071112203147.GA995458@sgi.com \
    --to=dgc@sgi.com \
    --cc=jpiszcz@lucidpixels.com \
    --cc=xaiki@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.