public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Hollis Blanchard <hollisb@us.ibm.com>
To: Alexander Graf <agraf@suse.de>
Cc: kvm-devel <kvm-devel@lists.sourceforge.net>,
	kvm-ppc-devel <kvm-ppc-devel@lists.sourceforge.net>
Subject: Re: [kvm-ppc-devel] Top level kvm-userspace directory	getting	crowded ...	need new dir	for qemu dependencies
Date: Wed, 27 Feb 2008 16:19:15 -0600	[thread overview]
Message-ID: <1204150755.2532.110.camel@basalt> (raw)
In-Reply-To: <4CDE892F-8AE7-4846-84A0-C37B9FE328C1@suse.de>

On Wed, 2008-02-27 at 22:20 +0100, Alexander Graf wrote:
> On Feb 27, 2008, at 9:22 PM, Hollis Blanchard wrote:
> >
> > So again, we the potential users are qemu and dtc.
> 
> Just while reading this I thought "Hey cool, dtc is packaged in most  
> distributions anyway. So why not modify dtc to provide the library, so  
> we have a common code base and make it a build dependency?"

That's a strange assertion, considering that Debian (and thus Ubuntu)
doesn't have it.

> > There is no need to equate "copy" with "fork". We will not be  
> > modifying this code, so there is no fork.
> 
> Cool! No need to provide a copy of it then, as we can use the  
> 'upstream' one.

I'm aware that we *could* use an upstream version of libfdt, if
everybody packaged and distributed it. However, they don't, I'm not
going to create and maintain those packages, and apparently you're not
volunteering either. So what "upsteam" could we use if we wanted to?

> >> This is a question of taste though and I don't want to have this
> >> ending as a flame war. So please just ask the other users if they  
> >> like
> >> the idea. As I lack real knowledge of device trees and PPC specifics,
> >> I wouldn't make a good moderator.
> >
> > The one piece of feedback I've gotten is (verbatim): "Unless they  
> > have a
> > really good reason why, I think it's pointless."
> >
> > I agree, this is a ridiculous thing to be arguing over, and I expected
> > to spend my day actually being productive. Maybe the problem here is
> > really the abbreviation "lib" in the name. How about I just call it
> > "fdt"?
> 
> I'm sorry. In the end it's more or less your decision anyway.

Is it? If so, I think I've made my decision clear...

> If you  
> plan to make frequent changes to the code (aka fork), include it in  
> kvm. If you are only planning on using code already available without  
> changes (aka copy), please change dtc to make the functionality that  
> exists available to kvm (e.g. a dot a file).
> 
> This mostly seems to be Avi's opinion as well as far as I understood it.

Have you actually looked at the code in question, or just saw that it
has "lib" in the name?

It's 1600 lines of C. In contrast, zlib, which is used in a large number
of projects, and despite that is often statically linked, is 8500.

-- 
Hollis Blanchard
IBM Linux Technology Center


-------------------------------------------------------------------------
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-27 22:19 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-25  6:50 Top level kvm-userspace directory getting crowded ... need new dir for qemu dependencies Jerone Young
2008-02-25  9:00 ` Avi Kivity
2008-02-26 17:24   ` Jerone Young
2008-02-27 10:59     ` Avi Kivity
2008-02-27 16:29     ` Hollis Blanchard
2008-02-27 16:34       ` Avi Kivity
2008-02-27 16:48         ` Alexander Graf
2008-02-27 16:59           ` Avi Kivity
2008-02-27 17:07             ` Alexander Graf
2008-02-27 18:56           ` Hollis Blanchard
2008-02-27 19:18             ` Alexander Graf
2008-02-27 20:22               ` [kvm-ppc-devel] " Hollis Blanchard
2008-02-27 21:20                 ` Alexander Graf
2008-02-27 22:19                   ` Hollis Blanchard [this message]
2008-02-27 22:32                     ` Alexander Graf
2008-03-02 18:38                     ` Luca Barbato
2008-02-27 19:25             ` Avi Kivity
2008-02-27 19:57               ` [kvm-ppc-devel] " Hollis Blanchard
2008-02-28  8:16                 ` Avi Kivity
2008-02-28 20:28                   ` Jerone Young
2008-03-02 16:41                     ` 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=1204150755.2532.110.camel@basalt \
    --to=hollisb@us.ibm.com \
    --cc=agraf@suse.de \
    --cc=kvm-devel@lists.sourceforge.net \
    --cc=kvm-ppc-devel@lists.sourceforge.net \
    /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