From: Max Krasnyanskiy <maxk@qualcomm.com>
To: Jens Axboe <jens.axboe@oracle.com>
Cc: linux-kernel@vger.kernel.org, npiggin@suse.de, dgc@sgi.com
Subject: Re: [PATCH 0/7] IO CPU affinity testing series
Date: Wed, 12 Mar 2008 13:37:09 -0700 [thread overview]
Message-ID: <47D83EF5.6070302@qualcomm.com> (raw)
In-Reply-To: <1205322940-20127-1-git-send-email-jens.axboe@oracle.com>
Jens Axboe wrote:
> Hi,
>
> Here's a new round of patches to play with io cpu affinity. It can,
> as always, also be found in the block git repo. The branch name is
> 'io-cpu-affinity'.
>
> The major change since last post is the abandonment of the kthread
> approach. It was definitely slower then may 'add IPI to signal remote
> block softirq' hack. So I decided to base this on the scalable
> smp_call_function_single() that Nick posted. I tweaked it a bit to
> make it more suitable for my use and also faster.
>
> As for functionality, the only change is that I added a bio hint
> that the submitter can use to ask for completion on the same CPU
> that submitted the IO. Pass in BIO_CPU_AFFINE for that to occur.
>
> Otherwise the modes are the same as last time:
>
> - You can set a specific cpumask for queuing IO, and the block layer
> will move submitters to one of those CPUs.
> - You can set a specific cpumask for completion of IO, in which case
> the block layer will move the completion to one of those CPUs.
> - You can set rq_affinity mode, in which case IOs will always be
> completed on the CPU that submitted them.
>
> Look in /sys/block/<dev>/queue/ for the three sysfs variables that
> modify this behaviour.
>
> I'd be interested in getting some testing done on this, to see if
> it really helps the larger end of the scale. Dave, I know you
> have a lot of experience in this area and would appreciate your
> input and/or testing. I'm not sure if any of the above modes will
> allow you to do what you need for eg XFS - if you want all meta data
> IO completed on one (or a set of) CPU(s), then I can add a mode
> that will allow you to play with that. Or if something else, give me
> some input and we can take it from there!
Very cool stuff. I think I can use it for cpu isolation purposes.
ie Isolating a cpu from the io activity.
You may have noticed that I started a bunch of discussion on CPU isolation.
One thing that came out of that is the suggestion to use cpusets for managing
this affinity masks. We're still discussing the details, the general idea is
to provide extra flags in the cpusets that enable/disable various activities
on the cpus that belong to the set.
For example in this particular case we'd have something like "cpusets.io" flag
that would indicate whether cpus in the set are allowed to to the IO or not.
In other words:
/dev/cpuset/io (cpus=0,1,2; io=1)
/dev/cpuset/no-io (cpus=3,4,5; io=0)
I'm not sure whether this makes sense or not. One advantage is that it's more
dynamic and more flexible. If for example you add cpu to the io cpuset it will
automatically start handling io requests.
btw What did you mean by "to see if it really helps the larger end of the
scale", what problem were you guys trying to solve ? I'm guessing cpu
isolation would probably be an unexpected user of io cpu affinity :).
Max
next prev parent reply other threads:[~2008-03-12 20:37 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-12 11:55 [PATCH 0/7] IO CPU affinity testing series Jens Axboe
2008-03-12 11:55 ` [PATCH 1/7] x86-64: introduce fast variant of smp_call_function_single() Jens Axboe
2008-03-14 18:21 ` Jeremy Fitzhardinge
2008-03-16 18:45 ` Jens Axboe
2008-03-16 22:58 ` Jeremy Fitzhardinge
2008-03-17 2:24 ` Nick Piggin
2008-03-17 7:25 ` Jens Axboe
2008-03-12 11:55 ` [PATCH 2/7] x86-64: speedup and tweak smp_call_function_single() Jens Axboe
2008-03-12 11:55 ` [PATCH 3/7] x86: add fast smp_call_function_single() Jens Axboe
2008-03-12 11:55 ` [PATCH 4/7] block: split softirq handling into blk-softirq.c Jens Axboe
2008-03-12 11:55 ` [PATCH 5/7] Add interface for queuing work on a specific CPU Jens Axboe
2008-03-12 11:55 ` [PATCH 6/7] block: make kblockd_schedule_work() take the queue as parameter Jens Axboe
2008-03-12 11:55 ` [PATCH 7/7] block: add test code for testing CPU affinity Jens Axboe
2008-03-12 16:41 ` [PATCH 0/7] IO CPU affinity testing series Alan D. Brunelle
2008-03-12 17:54 ` Jens Axboe
2008-03-12 20:37 ` Max Krasnyanskiy [this message]
2008-03-13 12:13 ` Jens Axboe
2008-03-13 14:54 ` Alan D. Brunelle
2008-03-13 15:00 ` Jens Axboe
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=47D83EF5.6070302@qualcomm.com \
--to=maxk@qualcomm.com \
--cc=dgc@sgi.com \
--cc=jens.axboe@oracle.com \
--cc=linux-kernel@vger.kernel.org \
--cc=npiggin@suse.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