From: "Andres Lagar-Cavilla" <andres@lagarcavilla.org>
To: Olaf Hering <olaf@aepfle.de>
Cc: keir@xen.org, andres@gridcentric.ca, Tim Deegan <tim@xen.org>,
xen-devel@lists.xen.org, wei.wang2@amd.com, jbeulich@suse.com,
adin@gridcentric.ca
Subject: Re: [PATCH 0 of 2] Support for Paging/Sharing on AMD
Date: Thu, 29 Mar 2012 09:05:48 -0700 [thread overview]
Message-ID: <394032cd12b140589a01a427ffd76876.squirrel@webmail.lagarcavilla.org> (raw)
In-Reply-To: <20120329160133.GA2616@aepfle.de>
> On Thu, Mar 29, Tim Deegan wrote:
>
>> That is, can we just drop this whole test?
>
> I'm ok with removing the existing boot_cpu_data.x86_vendor test,
> check only hap_enabled() and return -ENODEV otherwise.
>
> A recent change added need_iommu() check, this should be moved up into
> the -ENODEV case becasue -EXDEV is for PoD.
That was me. Either it has to be lumped with ENODEV or EXDEV. I considered
that EXDEV is the right answer. Both PoD and iommu are Xor cases with
another DEV. As opposed to no support at all (ENODEV)
Andres
>
> Olaf
>
next prev parent reply other threads:[~2012-03-29 16:05 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-27 15:43 [PATCH 0 of 2] Support for Paging/Sharing on AMD Andres Lagar-Cavilla
2012-03-27 15:43 ` [PATCH 1 of 2] x86/mm: Teach paging to page table-based p2m Andres Lagar-Cavilla
2012-03-27 15:43 ` [PATCH 2 of 2] x86/mm: Make iommu passthrough and mem paging/sharing mutually exclusive Andres Lagar-Cavilla
2012-03-29 11:03 ` [PATCH 0 of 2] Support for Paging/Sharing on AMD Tim Deegan
2012-03-29 14:46 ` Andres Lagar-Cavilla
2012-03-29 15:01 ` Tim Deegan
2012-03-29 15:26 ` Andres Lagar-Cavilla
2012-03-29 15:42 ` Tim Deegan
2012-03-29 15:48 ` Andres Lagar-Cavilla
2012-03-29 16:00 ` Tim Deegan
2012-03-29 16:01 ` Olaf Hering
2012-03-29 16:05 ` Andres Lagar-Cavilla [this message]
2012-03-29 16:08 ` Tim Deegan
2012-03-29 16:41 ` Olaf Hering
2012-03-29 16:58 ` Tim Deegan
2012-04-03 14:42 ` Olaf Hering
2012-04-03 15:10 ` Andres Lagar-Cavilla
2012-04-03 15:16 ` Olaf Hering
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=394032cd12b140589a01a427ffd76876.squirrel@webmail.lagarcavilla.org \
--to=andres@lagarcavilla.org \
--cc=adin@gridcentric.ca \
--cc=andres@gridcentric.ca \
--cc=jbeulich@suse.com \
--cc=keir@xen.org \
--cc=olaf@aepfle.de \
--cc=tim@xen.org \
--cc=wei.wang2@amd.com \
--cc=xen-devel@lists.xen.org \
/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;
as well as URLs for NNTP newsgroup(s).