From: Nathan Whitehorn <nwhitehorn@freebsd.org>
To: Paul Mackerras <paulus@samba.org>, Alexander Graf <agraf@suse.de>
Cc: "kvm-ppc@vger.kernel.org" <kvm-ppc@vger.kernel.org>,
KVM devel mailing list <kvm@vger.kernel.org>
Subject: Re: H_CLEAR_REF and H_CLEAR_MOD
Date: Wed, 18 Feb 2015 15:33:33 -0800 [thread overview]
Message-ID: <54E5214D.40201@freebsd.org> (raw)
In-Reply-To: <20150218220001.GA24858@iris.ozlabs.ibm.com>
On 02/18/15 14:00, Paul Mackerras wrote:
> On Wed, Feb 18, 2015 at 09:34:54AM +0100, Alexander Graf wrote:
>>> Am 18.02.2015 um 07:12 schrieb Nathan Whitehorn <nwhitehorn@freebsd.org>:
>>>
>>> It seems like KVM doesn't implement the H_CLEAR_REF and H_CLEAR_MOD hypervisor calls, which are absolutely critical for memory management in the FreeBSD kernel (and are marked "mandatory" in the PAPR manual). It seems some patches have been contributed already in https://lists.ozlabs.org/pipermail/linuxppc-dev/2011-December/095013.html, so it would be fantastic if these could end up upstream.
>> Paul, I guess we never included this because there was no user. If FreeBSD does use it though, I think it makes a lot of sense to resend it for inclusion.
> I agree. I just need to check the locking and synchronization around
> the reference and change bit recording, then I'll resend it.
Thanks much! Please let me know if I can help at all with this.
>>> I'm going to try to get some kind of workaround in the meantime so we can at least run on existing kernels.
>> Please don't add hacks in FreeBSD only because kvm is missing a feature. Let's just get this done properly :).
> Right.
Sounds good -- the possible workarounds are difficult anyway. I have
added a warning at boot that checks if these return H_FUNCTION and urges
people to upgrade their kernels if so.
-Nathan
> Paul.
>
next prev parent reply other threads:[~2015-02-18 23:33 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <54E42D5F.9080303@freebsd.org>
2015-02-18 8:34 ` H_CLEAR_REF and H_CLEAR_MOD Alexander Graf
2015-02-18 22:00 ` Paul Mackerras
2015-02-18 23:33 ` Nathan Whitehorn [this message]
2015-04-11 19:57 ` Nathan Whitehorn
2015-04-14 11:50 ` Paul Mackerras
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=54E5214D.40201@freebsd.org \
--to=nwhitehorn@freebsd.org \
--cc=agraf@suse.de \
--cc=kvm-ppc@vger.kernel.org \
--cc=kvm@vger.kernel.org \
--cc=paulus@samba.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