From: Robin Holt <holt@sgi.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: Robin Holt <holt@sgi.com>,
Andrew Morton <akpm@linux-foundation.org>,
"H. Peter Anvin" <hpa@zytor.com>,
tglx@linutronix.de, Jack Steiner <steiner@sgi.com>,
linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [patch 0/8] x86: UV - XPC fixes with related support functionality V2.
Date: Mon, 23 Nov 2009 17:57:51 -0600 [thread overview]
Message-ID: <20091123235751.GX6671@sgi.com> (raw)
In-Reply-To: <20091123181049.GA5220@elte.hu>
> Regarding the patches - i still very much dislike how the code
> interfaces to the 'BIOS'. The drivers/misc/sgi-xp/ code looks rather
> messy.
>
> Why isnt a pure hardware interface exposed, as both hpa and me suggested
> in the past?
The information is stored on hardware (UV-Hub and RAM) outside this instance of hardware.
The only safe way to access that hardware is via the GRU.
No GRU resources are available to the BIOS.
Ergo, this interface method with multiple passes is needed.
If there is a better way, I am open to suggestions.
> Anyway ... these bits got merged via -mm in the past so i guess it could
> be done there in the future too - the series does not look horrible -
> but the whole concept ought to be done in a cleaner way.
I will ask Andrew to take them.
Thanks,
Robin
next prev parent reply other threads:[~2009-11-23 23:57 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-19 17:31 [patch 0/8] x86: UV - XPC fixes with related support functionality V2 Robin Holt
2009-11-19 17:31 ` [patch 1/8] x86: UV - Introduce a means to translate from gpa -> socket_paddr Robin Holt
2009-11-19 17:31 ` [patch 2/8] x86: UV - XPC needs to provide an abstraction for uv_gpa Robin Holt
2009-11-19 17:32 ` [patch 3/8] x86: UV - Introduce uv_gpa_is_mmr Robin Holt
2009-11-19 17:32 ` [patch 4/8] X86: UV - Implement a gru_read_gpa kernel function Robin Holt
2009-11-19 17:32 ` [patch 5/8] x86: UV - Update XPC to handle updated BIOS interface Robin Holt
2009-11-19 17:32 ` [patch 6/8] x86: UV - XPC NULL deref when mesq becomes empty Robin Holt
2009-11-19 17:32 ` [patch 7/8] X86: UV - xpc_make_first_contact hang due to not accepting ACTIVE state Robin Holt
2009-11-19 17:32 ` [patch 8/8] X86: UV - XPC receive message reuse triggers invalid BUG_ON() Robin Holt
2009-11-23 18:03 ` [patch 0/8] x86: UV - XPC fixes with related support functionality V2 Ingo Molnar
2009-11-23 18:10 ` Ingo Molnar
2009-11-23 23:57 ` Robin Holt [this message]
2009-11-24 7:19 ` Ingo Molnar
-- strict thread matches above, loose matches on Subject: below --
2009-11-24 1:39 Robin Holt
2009-12-15 17:04 ` Robin Holt
2009-12-15 17:06 ` Robin Holt
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=20091123235751.GX6671@sgi.com \
--to=holt@sgi.com \
--cc=akpm@linux-foundation.org \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=steiner@sgi.com \
--cc=tglx@linutronix.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