All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Christian Borntraeger <borntraeger@de.ibm.com>
Cc: Carsten Otte <cotte@de.ibm.com>, Avi Kivity <avi@qumranet.com>,
	kvm <kvm@vger.kernel.org>, Olaf Schnapper <os@de.ibm.com>,
	Hollis Blanchard <hollisb@us.ibm.com>
Subject: Re: [PATCH] libkvm-s390
Date: Mon, 14 Jul 2008 13:00:52 -0500	[thread overview]
Message-ID: <487B9454.8030102@codemonkey.ws> (raw)
In-Reply-To: <200807141900.10467.borntraeger@de.ibm.com>

Christian Borntraeger wrote:
> Am Montag, 14. Juli 2008 schrieb Anthony Liguori:
>   
>> Carsten Otte wrote:
>>     
>>> This patch makes libkvm build and work on s390. It should be a noop for
>>> all other architectures.
>>>   
>>>       
>> I'm thinking of spending some time eliminate libkvm altogether in the 
>> next month or so.  Noone seems to be using it for alternative userspaces 
>> and the code that interacts with it in QEMU badly needs refactoring.
>>
>> Dropping the abstraction will probably result in a big drop in lines of 
>> code and would probably make it more clear anyway.
>>     
>
> Does that mean, that I can stop finishing Carstens patch?
>   

Depends.  I'd say there's not a lot of utility making kuli use libkvm in 
it's current form.  If you're trying to get QEMU working, then getting 
libkvm working is probably a good start.

It's still not clear whether libkvm is a useful abstraction.  It may be 
that we want to refactor libkvm to be more useful so it's not 100% clear 
atm that it's going away.

Regards,

Anthony Liguori

> Christian
>   


  reply	other threads:[~2008-07-14 18:01 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-11 17:29 [PATCH] libkvm-s390 Carsten Otte
2008-07-13  8:29 ` Avi Kivity
2008-07-14 11:33   ` Christian Borntraeger
2008-07-14 11:44     ` Avi Kivity
2008-07-14 12:25       ` Christian Borntraeger
2008-07-14 15:34 ` Anthony Liguori
2008-07-14 17:00   ` Christian Borntraeger
2008-07-14 18:00     ` Anthony Liguori [this message]
2008-07-16 15:28 ` [PATCH v2/RFC] libkvm-s390 Christian Borntraeger
2008-07-17 11:08   ` Christian Ehrhardt
2008-07-17 15:28     ` [PATCH v3] libkvm-s390 Christian Borntraeger
2008-07-19  7:56       ` 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=487B9454.8030102@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=avi@qumranet.com \
    --cc=borntraeger@de.ibm.com \
    --cc=cotte@de.ibm.com \
    --cc=hollisb@us.ibm.com \
    --cc=kvm@vger.kernel.org \
    --cc=os@de.ibm.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.