From: Paul Jackson <pj@sgi.com>
To: Max Krasnyansky <maxk@qualcomm.com>
Cc: mingo@elte.hu, a.p.zijlstra@chello.nl,
linux-kernel@vger.kernel.org, menage@google.com
Subject: Re: [PATCH 1/2] cpuset: cpuset irq affinities
Date: Tue, 11 Mar 2008 14:08:58 -0500 [thread overview]
Message-ID: <20080311140858.d0ecd47f.pj@sgi.com> (raw)
In-Reply-To: <47D6C08E.9040701@qualcomm.com>
Max wrote:
> Please take a look at
> [PATCH 2/2] cpusets: Improved irq affinity handling
> I'm treating irqs just like tasks (at least I think I'm :).
Well, I see the one comment in your Patch 2/2 noting you're unsure
of the locking in one place.
I don't see any further comments on or additional code involving
locking.
I don't see where you respond to my discussion with Peter of March
6 and 7, where I expressed some doubts about Peters patch (which you
built on in your patch 1/2 in this series).
I see only a little bit of additional comments in your patch 2/2
regarding handling of moving irqs to higher non-empty cpusets if a
cpuset is emptied of its CPUs.
I don't see any explanation of what you mean by "desired semantics."
I don't see any response to the alternatives to Peter's earlier patch
(your Patch 1/2 here) that Peter and I discussed in that discussion of
March 6 and 7.
And, in particular, could you respond to the question in my last
message:
> What semantics to you impose on irqs in overlapping cpusets,
> which would seem to lead to conflicting directives as to
> whether one set or another of irqs was to be applied to the
> CPUs in the overlap?
--
I won't rest till it's the best ...
Programmer, Linux Scalability
Paul Jackson <pj@sgi.com> 1.940.382.4214
next prev parent reply other threads:[~2008-03-11 19:09 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-11 5:57 [PATCH 0/2] cpusets: support for irqs maxk
2008-03-11 5:57 ` [PATCH 1/2] cpuset: cpuset irq affinities maxk
2008-03-11 5:57 ` [PATCH 2/2] cpusets: Improved irq affinity handling maxk
2008-03-11 6:35 ` [PATCH 1/2] cpuset: cpuset irq affinities Christoph Hellwig
2008-03-11 17:05 ` Max Krasnyansky
2008-03-11 18:58 ` Paul Jackson
2008-03-11 19:50 ` Max Krasnyansky
2008-03-11 6:57 ` Paul Jackson
2008-03-11 17:25 ` Max Krasnyansky
2008-03-11 19:08 ` Paul Jackson [this message]
2008-03-11 21:31 ` Max Krasnyansky
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=20080311140858.d0ecd47f.pj@sgi.com \
--to=pj@sgi.com \
--cc=a.p.zijlstra@chello.nl \
--cc=linux-kernel@vger.kernel.org \
--cc=maxk@qualcomm.com \
--cc=menage@google.com \
--cc=mingo@elte.hu \
/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.