All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabio Fantoni <fabio.fantoni@m2r.biz>
To: Jan Beulich <JBeulich@suse.com>
Cc: anthony.perard@citrix.com,
	xen-devel <xen-devel@lists.xenproject.org>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: gross qemu behavior
Date: Mon, 05 May 2014 13:10:33 +0200	[thread overview]
Message-ID: <536771A9.3080002@m2r.biz> (raw)
In-Reply-To: <5367857A020000780000EF1A@mail.emea.novell.com>

Il 05/05/2014 12:35, Jan Beulich ha scritto:
>>>> On 05.05.14 at 12:04, <fabio.fantoni@m2r.biz> wrote:
>> Il 04/04/2014 18:54, Stefano Stabellini ha scritto:
>>> On Fri, 4 Apr 2014, Jan Beulich wrote:
>>>>>>> On 04.04.14 at 17:32, <stefano.stabellini@eu.citrix.com> wrote:
>>>>> On Fri, 4 Apr 2014, Jan Beulich wrote:
>>>>>>>>> On 04.04.14 at 15:53, <stefano.stabellini@eu.citrix.com> wrote:
>>>>>>> On Fri, 4 Apr 2014, Jan Beulich wrote:
>>>>>>>> No, it should just never appear at the wrong address. As I said above,
>>>>>>>> I'd consider it halfway acceptable if it remained mapped despite an
>>>>>>>> unmap, but I do think that a proper solution (properly unmapping
>>>>>>>> without de-allocating) can and should be found.
>>>>>>> There is no way to do it today AFAICT.
>>>>>>> Would you care of proposing an hypercall that would support this
>>>>>>> scenario?
>>>>>> Hypercall? Everything you need is there afaict.
>>>>> Maybe I am missing something.
>>>>>
>>>>> Moving the PCI ROM to the right place in the guest physmap is easy.
>>>>> However how do you think we could unmap the memory (remove it from the
>>>>> guest physmap) without deallocating it?
>>>>> The only hypercall we have is xc_domain_add_to_physmap at the moment.
>>>> XENMEM_remove_from_physmap should be quite suitable here, but
>>>> that is not your problem. The problem is that XENMEM_add_to_physmap
>>>> requires a GFN to be passed in, i.e. assumes that a page to be mapped
>>>> is already mapped somewhere in the guest. (The term "add" in this
>>>> context is rather confusing, as in the GMFN map space case the page
>>>> isn't being added, but moved.) So indeed there is functionality missing
>>>> in the hypervisor.
>>> Right.
>>>
>>> After we call XENMEM_remove_from_physmap, there is no way of adding back
>>> the pages to the physmap without knowing the corresponding mfns.  Even
>>> if we knew the mfns of the original allocation, relying on them is
>>> probably not a good idea because the pages could theoretically be
>>> offlined or shared.
>>>
>>> This is the reason why I was asking for the hypercall you had in mind to
>>> solve this problem.
>> Any news about this discussion?
> Someone would need to write both hypervisor and qemu side patches
> - are you volunteering?
>
> Jan
>
Thanks for your reply.
Unfortunately I don't have enough knowledge to do such patches.
I just wanted to know if there was any news.

      reply	other threads:[~2014-05-05 11:10 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-28  7:48 gross qemu behavior Jan Beulich
2014-03-28  9:21 ` Jan Beulich
2014-03-28  9:30 ` Fabio Fantoni
2014-03-28 10:37   ` Jan Beulich
2014-03-28 17:46 ` Stefano Stabellini
2014-03-28 17:52   ` Stefano Stabellini
2014-03-28 18:01     ` Paolo Bonzini
2014-03-28 18:30       ` Stefano Stabellini
2014-03-29  7:31         ` Paolo Bonzini
2014-03-30  7:57           ` Fabio Fantoni
2014-03-31  9:07   ` Jan Beulich
2014-04-03 16:12     ` Stefano Stabellini
2014-04-04  6:45       ` Jan Beulich
2014-04-04  9:34         ` Paolo Bonzini
2014-04-04  9:45           ` Jan Beulich
2014-04-04 13:53         ` Stefano Stabellini
2014-04-04 14:58           ` Jan Beulich
2014-04-04 15:32             ` Stefano Stabellini
2014-04-04 16:00               ` Jan Beulich
2014-04-04 16:54                 ` Stefano Stabellini
2014-05-05 10:04                   ` Fabio Fantoni
2014-05-05 10:35                     ` Jan Beulich
2014-05-05 11:10                       ` Fabio Fantoni [this message]

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=536771A9.3080002@m2r.biz \
    --to=fabio.fantoni@m2r.biz \
    --cc=JBeulich@suse.com \
    --cc=anthony.perard@citrix.com \
    --cc=pbonzini@redhat.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xenproject.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.