All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
To: "Dong, Eddie" <eddie.dong-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: kvm-devel <kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>
Subject: Re: kernel device reset support
Date: Tue, 09 Oct 2007 12:55:55 +0200	[thread overview]
Message-ID: <470B5E3B.4060006@qumranet.com> (raw)
In-Reply-To: <10EA09EFD8728347A513008B6B0DA77A0231C1B2-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>

Dong, Eddie wrote:
> Avi Kivity wrote:
>   
>>> If BSP mp_state becomes VCPU_MP_STATE_UNINITIALIZED, current code
>>> can't wakeup it. We need additional code that I am not aware of now.
>>>
>>> Current VCPU must be BSP, otherwise the code executing in Qemu will
>>> have problem too. 
>>>
>>>       
>> But, for an ungraceful reset, nothing prevents an AP from
>> issuing a reset?
>>     
>
> Mmm, Yes, but I think current architecture can't handle this.
> The thread where AP issues "RESET" will continue run, which
> means it becomes BSP now and wake up other APs later on.
> Or We can block that AP first and then inform BSP to do 
> RESET job. Here we need to block the AP in kernel 
> so that we can wake up.
>   

It should call vcpu_halt() immediately after reset.

> It can be a future task which is not that high priority IMO. 
> I will focus on SMP boot 1st. Your opnion?
>   

Agree.  But let's make it close to the complete solution.

>>>
>>>       
>> What I mean is to use mp_state within the vcpu code (while holding the
>> vcpu mutex) and to use vcpu->requests as a means to tell the vcpu it
>> needs to change state. 
>>
>>     
>
> Then we need to add code to enter waitqueque here. I think force_to_quit
> is much simple and efficient since we don't need to test (atomic test) 
> at each VM Exit even light weight VM Exit.
>   

The test for vcpu->requests already exists (and is needed for tlb 
flushes) so there is no additional performance hit.

> But certainly it can. If you want to save the per VCPU force_to_quit, we
> can share it with vcpu_request, but test in external IRQ is better IMO
> since we take a kick now.
>   

The vcpu may have exited due to some other reason, and the interrupt 
taken in host context?


-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/

  parent reply	other threads:[~2007-10-09 10:55 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-08 10:17 kernel device reset support Dong, Eddie
     [not found] ` <10EA09EFD8728347A513008B6B0DA77A0231BB36-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-08 10:24   ` Avi Kivity
     [not found]     ` <470A0556.80903-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-09  1:58       ` Dong, Eddie
     [not found]         ` <10EA09EFD8728347A513008B6B0DA77A0231BD83-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-09  9:34           ` Avi Kivity
     [not found]             ` <470B4B2E.1000500-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-09  9:53               ` Avi Kivity
2007-10-09 10:11               ` Dong, Eddie
     [not found]                 ` <10EA09EFD8728347A513008B6B0DA77A0231C1AA-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-09 10:17                   ` Avi Kivity
     [not found]                     ` <470B5528.2010605-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-09 10:36                       ` Dong, Eddie
     [not found]                         ` <10EA09EFD8728347A513008B6B0DA77A0231C1B2-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-09 10:55                           ` Avi Kivity [this message]
     [not found]                             ` <470B5E3B.4060006-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-10  6:17                               ` Dong, Eddie
     [not found]                                 ` <10EA09EFD8728347A513008B6B0DA77A02364242-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-10 10:23                                   ` Avi Kivity
     [not found]                                     ` <470CA814.9050907-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-11  1:32                                       ` Dong, Eddie
     [not found]                                         ` <10EA09EFD8728347A513008B6B0DA77A02364638-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-11  7:24                                           ` Dong, Eddie
2007-10-11 12:11                                           ` Avi Kivity
     [not found]                                             ` <470E130D.6080808-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-12  1:07                                               ` Dong, Eddie
     [not found]                                                 ` <10EA09EFD8728347A513008B6B0DA77A02364C43-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-12  6:18                                                   ` Avi Kivity
     [not found]                                                     ` <470F11B9.4050501-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-12  7:17                                                       ` Dong, Eddie
     [not found]                                                         ` <10EA09EFD8728347A513008B6B0DA77A02364F85-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-13  7:16                                                           ` Avi Kivity
     [not found]                                                             ` <471070D8.7030402-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-15  4:40                                                               ` Dong, Eddie
     [not found]                                                                 ` <10EA09EFD8728347A513008B6B0DA77A023A6DFE-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-15  9:10                                                                   ` Avi Kivity
     [not found]                                                                     ` <47132E9D.7030500-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-16  8:44                                                                       ` Dong, Eddie
     [not found]                                                                         ` <10EA09EFD8728347A513008B6B0DA77A023A763C-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-10-16 10:25                                                                           ` Avi Kivity
     [not found]                                                                         ` <471491A9. 8040207@qumranet.com>
     [not found]                                                                           ` <471491A9.8040207-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-10-17  1:40                                                                             ` Dong, Eddie
2007-10-08 10:27   ` Avi Kivity

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=470B5E3B.4060006@qumranet.com \
    --to=avi-atkuwr5tajbwk0htik3j/w@public.gmane.org \
    --cc=eddie.dong-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.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.