[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1355220811.2400.5.camel@dabdike.int.hansenpartnership.com>
Date: Tue, 11 Dec 2012 10:13:31 +0000
From: James Bottomley <James.Bottomley@...senPartnership.com>
To: Jeff Moyer <jmoyer@...hat.com>
Cc: linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org,
Bart Van Assche <bvanassche@....org>
Subject: Re: [patch,v3,repost 00/10] make I/O path allocations more
numa-friendly
On Mon, 2012-12-10 at 12:59 -0500, Jeff Moyer wrote:
> Jeff Moyer <jmoyer@...hat.com> writes:
>
> > Hi,
> >
> > This patch set makes memory allocations for data structures used in
> > the I/O path more numa friendly by allocating them from the same numa
> > node as the storage device. I've only converted a handful of drivers
> > at this point. My testing is limited by the hardware I have on hand.
> > Using these patches, I was able to max out the bandwidth of the storage
> > controller when issuing I/O from any node on my 4 node system. Without
> > the patch, I/O from nodes remote to the storage device would suffer a
> > penalty ranging from 6-12%. Given my relatively low-end setup[1], I
> > wouldn't be surprised if others could show a more significant performance
> > advantage.
> >
> > This is a repost of the last posting. The only changes are additional
> > reviewed-by/acked-by tags. I think this version is ready for inclusion.
> > James, would you mind taking a look?
>
> James? Do you have any objections to including this for 3.8?
Probably for 3.9 since the 3.8 merge window is upon us.
Do we actually have any performance numbers from the big system people?
That's really a must for this type of work.
It's missing acks from the affected drivers; that's not a show stopper
but it would be better to have them.
James
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists