public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@qumranet.com>
To: Glauber Costa <gcosta@redhat.com>
Cc: kvm-devel@lists.sourceforge.net, marcelo@kvack.org, chrisw@sous-sol.org
Subject: Re: [PATCH 4/15] mark processors as presents
Date: Mon, 25 Feb 2008 16:49:53 +0200	[thread overview]
Message-ID: <47C2D591.8030104@qumranet.com> (raw)
In-Reply-To: <47C2D2CB.6000803@redhat.com>

Glauber Costa wrote:
>>
>> There is now code in rombios32.c to do this.  It needs to be 
>> removed.  See acpi_build_processor_ssdt().
>
> Building the table by hand is trivial in the case where the processors 
> are just _listed_, and can be easily justified. This first patch just 
> add the _STA method, but other follows, which turns the processor 
> block into a quite complicated thing. Not to mention the operational 
> region,
> the notifications that have to refer to the processor objects, etc.
>
> So I can't see any valid justification for using the code in 
> rombios32.c, instead of a high level language such as the one provided 
> by acpi.

I meant, remove the code in rombios32.c.  Sorry about the confusion.

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


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

  reply	other threads:[~2008-02-25 14:49 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-22 21:34 [PATCH 0/15] Review: acpi processor hotplug Glauber Costa
2008-02-22 21:34 ` [PATCH 1/15] Make a GPE register block be acessible Glauber Costa
2008-02-22 21:34   ` [PATCH 2/15] mark extra cpus as present Glauber Costa
2008-02-22 21:34     ` [PATCH 3/15] introduce cpu_set to qemu monitor Glauber Costa
2008-02-22 21:34       ` [PATCH 4/15] mark processors as presents Glauber Costa
2008-02-22 21:34         ` [PATCH 5/15] provide gpe _L0x methods Glauber Costa
2008-02-22 21:34           ` [PATCH 6/15] provide operation region for pio to the gpes Glauber Costa
2008-02-22 21:34             ` [PATCH 7/15] implement method _L00 for GPE0 Glauber Costa
2008-02-22 21:34               ` [PATCH 8/15] isolate cpu initialization function in hw/pc.c Glauber Costa
2008-02-22 21:34                 ` [PATCH 9/15] initialize hot add system Glauber Costa
2008-02-22 21:34                   ` [PATCH 10/15] handle gpe data for pio Glauber Costa
2008-02-22 21:34                     ` [PATCH 11/15] manipulate the gpe bits and send sci up the os Glauber Costa
2008-02-22 21:34                       ` [PATCH 12/15] isolate cpu thread creation in qemu-kvm.c Glauber Costa
2008-02-22 21:34                         ` [PATCH 13/15] provide _MAT to acpi processor Glauber Costa
2008-02-22 21:34                           ` [PATCH 14/15] start a new cpu thread Glauber Costa
2008-02-22 21:34                             ` [PATCH 15/15] provide _EJ0 method for processor removal Glauber Costa
2008-02-24  9:33         ` [PATCH 4/15] mark processors as presents Avi Kivity
2008-02-25 14:38           ` Glauber Costa
2008-02-25 14:49             ` Avi Kivity [this message]
2008-02-25 14:43               ` Glauber Costa
2008-02-24  9:34       ` [PATCH 3/15] introduce cpu_set to qemu monitor Avi Kivity
2008-02-24  9:44 ` [PATCH 0/15] Review: acpi processor hotplug Avi Kivity
  -- strict thread matches above, loose matches on Subject: below --
2008-02-26 19:56 [PATCH 0/15] " Glauber Costa
2008-02-26 19:56 ` [PATCH 1/15] Make a GPE register block be acessible Glauber Costa
2008-02-26 19:56   ` [PATCH 2/15] mark extra cpus as present Glauber Costa
2008-02-26 19:56     ` [PATCH 3/15] introduce cpu_set to qemu monitor Glauber Costa
2008-02-26 19:56       ` [PATCH 4/15] mark processors as presents Glauber Costa
2008-02-26 23:05         ` Alexander Graf
2008-02-26 23:07           ` Alexander Graf

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=47C2D591.8030104@qumranet.com \
    --to=avi@qumranet.com \
    --cc=chrisw@sous-sol.org \
    --cc=gcosta@redhat.com \
    --cc=kvm-devel@lists.sourceforge.net \
    --cc=marcelo@kvack.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