From: Vitaly Kuznetsov <vkuznets@redhat.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: xen-devel@lists.xenproject.org, Andrew Jones <drjones@redhat.com>,
David Vrabel <david.vrabel@citrix.com>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: [PATCH RFC 0/1] Introduce VCPUOP_reset_vcpu_info
Date: Thu, 07 Aug 2014 13:54:56 +0200 [thread overview]
Message-ID: <87mwbg4ir3.fsf@vitty.brq.redhat.com> (raw)
In-Reply-To: <53E383B2020000780002A1E5@mail.emea.novell.com> (Jan Beulich's message of "Thu, 07 Aug 2014 12:48:34 +0100")
"Jan Beulich" <JBeulich@suse.com> writes:
>>>> On 07.08.14 at 13:41, <vkuznets@redhat.com> wrote:
>> "Jan Beulich" <JBeulich@suse.com> writes:
>>>>>> On 06.08.14 at 17:03, <vkuznets@redhat.com> wrote:
>>>> In xen their vcpu_info will point to dummy_vcpu_info (that's what we
>>>> have there before VCPUOP_register_vcpu_info I guess).
>>>
>>> Right, but such a vCPU can't be brought up. Did you verify that
>>> this holds with your additions?
>>
>> That's fine as new kernel will call VCPUOP_register_vcpu_info on
>> bootup.
>
> But my question was targeting the hypervisor side, i.e. I was
> asking that you get away from your "works fine in my limited use
> case" way of thinking, and rather check that you don't subtly
> break anything else.
Thanks, that's fine, that's what the RFC is about. I'll definitely try
looking deeper and I'll be grateful for some examples of what can
actually go wrong.
--
Vitaly
prev parent reply other threads:[~2014-08-07 11:55 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-06 13:08 [PATCH RFC 0/1] Introduce VCPUOP_reset_vcpu_info Vitaly Kuznetsov
2014-08-06 13:08 ` [PATCH RFC 1/1] " Vitaly Kuznetsov
2014-08-06 14:36 ` Jan Beulich
2014-08-06 14:09 ` [PATCH RFC 0/1] " Jan Beulich
2014-08-06 14:49 ` Konrad Rzeszutek Wilk
2014-08-06 14:54 ` Jan Beulich
2014-08-06 15:03 ` Konrad Rzeszutek Wilk
2014-08-06 14:51 ` Konrad Rzeszutek Wilk
2014-08-06 15:00 ` Jan Beulich
2014-08-06 15:02 ` Konrad Rzeszutek Wilk
2014-08-06 15:38 ` Jan Beulich
2014-08-06 15:03 ` Vitaly Kuznetsov
2014-08-06 15:43 ` Jan Beulich
2014-08-07 11:41 ` Vitaly Kuznetsov
2014-08-07 11:48 ` Jan Beulich
2014-08-07 11:54 ` Vitaly Kuznetsov [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=87mwbg4ir3.fsf@vitty.brq.redhat.com \
--to=vkuznets@redhat.com \
--cc=JBeulich@suse.com \
--cc=david.vrabel@citrix.com \
--cc=drjones@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.