From: Paul Jackson <pj@sgi.com>
To: 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 01:57:11 -0500 [thread overview]
Message-ID: <20080311015711.9c4615a6.pj@sgi.com> (raw)
In-Reply-To: <1205215046-27085-2-git-send-email-maxk@qualcomm.com>
Max K wrote:
> this could also provide the desired semantics.
Could you spell out what you mean by "the desired semantics" ?
I don't see any Documentation or much comments, which would
help understand this. It helps to describe both what has
changed, and, from the top, the why, what and how of what
you're doing, in part as Documentation or code comments,
for the benefit of future readers.
Did you see my discussion of this with Peter on March 6 and 7
in the lkml "[RFC/PATCH] cpuset: cpuset irq affinities" thread?
This latest patch of yours seems, offhand, to predate that discussion.
I don't see any explanation of what locking is needed when.
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 6:57 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 [this message]
2008-03-11 17:25 ` Max Krasnyansky
2008-03-11 19:08 ` Paul Jackson
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=20080311015711.9c4615a6.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®