All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Roger Pau Monné" <roger.pau@citrix.com>
To: Wei Liu <wei.liu2@citrix.com>, Igor Kozhukhov <ikozhukhov@gmail.com>
Cc: xen-devel@lists.xen.org
Subject: Re: sources of freebsd with xen-4.2 kernel updates
Date: Mon, 30 Dec 2013 13:53:01 +0100	[thread overview]
Message-ID: <52C16CAD.4040009@citrix.com> (raw)
In-Reply-To: <20131230124221.GA14746@zion.uk.xensource.com>

On 30/12/13 13:42, Wei Liu wrote:
> On Mon, Dec 30, 2013 at 04:18:28PM +0400, Igor Kozhukhov wrote:
>>
>> On Dec 30, 2013, at 3:25 PM, Roger Pau Monné wrote:
>>
>>> On 30/12/13 11:57, Igor Kozhukhov wrote:
>>>> Hi Roger,
>>>>
>>>> thanks for the link.
>>>> I have the same/(similar the same) implementation on illumos(OpenSolaris).
>>>>
>>>> do you have plans for grant table v1 & v2 implementation on FreeBSD ?
>>>
>>> Grant table v1 is the current implementation, and I don't have plans
>>> currently to add support for v2 (cannot speak of others though).
>>>
>>> Linux has v2 implemented, but AFAIK there's no user of the new features
>>> introduced by v2. Right now with v2 you just end up with less grants,
>>> because v2 grants metadata use twice as much memory as v1 grants, and
>>> the number of grant frames is limited by the hypervisor.
>>>
>> Thanks for your info.
>>
>> Could you please let me know - can we use v1 for PV and HVM guests ?
>> I looked v2 implementation on Linux and found that v1 is using for HVM, and v2 for PV - but maybe i'm wrong.
>>
> 
> You can use V1 for all guests for sure.
> 
> My understanding is that the code in Linux's grant-table.c requests V2
> form the hypervisor, but it will fall back to using V1 if V2 is not
> avaiable.

Linux explicitly requests v1 when running as an HVM guest (I have no
idea why), for the rest it tries v2 first and falls back to v1 if
there's no v2 support:

http://lxr.free-electrons.com/source/drivers/xen/grant-table.c#L1116

As Wei points out, you can safely use v1 for all kind of guests, at
least on x86.

Roger.

  reply	other threads:[~2013-12-30 12:53 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-29 17:54 sources of freebsd with xen-4.2 kernel updates Igor Kozhukhov
2013-12-30  8:10 ` Roger Pau Monné
2013-12-30 10:57   ` Igor Kozhukhov
2013-12-30 11:25     ` Roger Pau Monné
2013-12-30 12:18       ` Igor Kozhukhov
2013-12-30 12:42         ` Wei Liu
2013-12-30 12:53           ` Roger Pau Monné [this message]
2013-12-30 14:52             ` Konrad Rzeszutek Wilk
2013-12-30 12:59       ` Igor Kozhukhov
2013-12-30 14:53         ` Konrad Rzeszutek Wilk

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=52C16CAD.4040009@citrix.com \
    --to=roger.pau@citrix.com \
    --cc=ikozhukhov@gmail.com \
    --cc=wei.liu2@citrix.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 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.