public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
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.
>

  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