All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sheng Yang <sheng@linux.intel.com>
To: Cristi Magherusan <Cristi.Magherusan@net.utcluj.ro>
Cc: Jordan Justen <jljusten@gmail.com>,
	kvm@vger.kernel.org, Jordan Justen <jordan.l.justen@intel.com>,
	Marcelo Tosatti <mtosatti@redhat.com>
Subject: Re: [PATCH] Update qemu-kvm to allow a larger BIOS image.
Date: Thu, 23 Jul 2009 09:42:01 +0800	[thread overview]
Message-ID: <200907230942.03064.sheng@linux.intel.com> (raw)
In-Reply-To: <1248289473.12578.5.camel@hyperion>

On Thursday 23 July 2009 03:04:33 Cristi Magherusan wrote:
> On Thu, 2009-07-16 at 10:57 -0700, Jordan Justen wrote:
> > On Wed, Jul 15, 2009 at 10:37 PM, Sheng Yang<sheng@linux.intel.com> wrote:
> > > Make sense to me. So what's mattered here is not bios, but qemu-kvm and
> > > kvm code. The user can replace bios binary by UEFI binary easily, but
> > > not with kvm related part.
> > >
> > > I realized you still need separate the qemu-kvm patch into two: one for
> > > bios and another for qemu.
> >
> > Okay, I will split this change apart.
> >
> > > And I just hope this modification won't break some old
> > > OS(any OS got assumption about bios size? I think Tiano team should
> > > have idea on it)...
> >
> > I don't think the OS should care about the bios size, but it is
> > important to update the INT15-E820 memory ranges to help
> > make sure the OS knows which regions are in use.
> >
> > Even without the E820 change, I think it would be unlikely
> > for an OS to try utilize these memory regions, but it is
> > definitely better to properly describe it.
>
> Hello,
>
> I don't know if I get this right, but is it enough to apply this patch
> to KVM-userspace, or do I also need to have a patched kernel in order to
> get large images (my image is 4MB) to work in KVM?
>
You need a patched KVM kernel modules for this. And you can use Jordan's patch 
currently(http://article.gmane.org/gmane.comp.emulators.kvm.devel/37344). The 
first one for BIOS(apply to qemu-kvm tree), second of qemu-kvm, and the third 
for KVM modules.

-- 
regards
Yang, Sheng

      reply	other threads:[~2009-07-23  1:42 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-16  0:12 [PATCH] Update KVM kernel module to allow a larger BIOS image Jordan Justen
2009-07-16  0:12 ` [PATCH] Update qemu-kvm " Jordan Justen
2009-07-16  1:34   ` Sheng Yang
2009-07-16  2:58     ` Jordan Justen
2009-07-16  3:08       ` Sheng Yang
2009-07-16  4:57         ` Jordan Justen
2009-07-16  5:37           ` Sheng Yang
2009-07-16 17:57             ` Jordan Justen
2009-07-22 19:04               ` Cristi Magherusan
2009-07-23  1:42                 ` Sheng Yang [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=200907230942.03064.sheng@linux.intel.com \
    --to=sheng@linux.intel.com \
    --cc=Cristi.Magherusan@net.utcluj.ro \
    --cc=jljusten@gmail.com \
    --cc=jordan.l.justen@intel.com \
    --cc=kvm@vger.kernel.org \
    --cc=mtosatti@redhat.com \
    /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.