All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: ronnie sahlberg <ronniesahlberg@gmail.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [Patch 0/14] builtin iscsi support
Date: Fri, 03 Dec 2010 15:22:02 -0600	[thread overview]
Message-ID: <4CF95F7A.6090104@codemonkey.ws> (raw)
In-Reply-To: <AANLkTinn2K2mAC9BKhD-zwU1mCMRspgiAPo14-gSRCEf@mail.gmail.com>

On 12/03/2010 01:57 PM, ronnie sahlberg wrote:
> On Sat, Dec 4, 2010 at 2:05 AM, Anthony Liguori<anthony@codemonkey.ws>  wrote:
>    
>> On 12/03/2010 05:09 AM, ronniesahlberg@gmail.com wrote:
>>      
>    
>>> Note that ./block/iscsi/* is aimed at being re-used outisde of qemu/kvm
>>> in other applications why qemu/kvm specific calkls are not used there.
>>>
>>>        
>> So should the library be packaged outside of QEMU and then we'll just link
>> against it?
>>      
> Eventually I guess. But that would take a long time before it is ready
> for standalone
> distrution.
>    

I don't think stand alone distribution is a big burden.

> I see this as similar to the situation with LIBTALLOC and LIBTDB that
> originated from samba.
> For many years these libraries were kept as local copies inside the
> source trees of many projects,
> including samba3, samba4, ctdb, openchange, etc where the tdb/talloc maintainers
> merged patches and fixes across the different comsumer projects manually.
>    

Yeah, and I think this creates a distro nightmare :-)

Figuring out which project has which version with which modifications 
and whether a security patch is applicable is very painful.

 From a strictly QEMU perspective, I'm not very keen on having code in 
the repository that doesn't use common infrastructure and doesn't follow 
our coding style.

Regards,

Anthony Liguori

> While these libraries are now at a stage where they are mature enough
> to stand on their own,
> they now start to appear as separate standalone packages for distros.
> So in time, once all distros ship them as standalone
> the local copies held in some of the projects may be removed and
> replaced with linking with the
> standalone library instead.
>
>
> I see that as one possible path how a library is started to be used,
> how it evolves and once proven and mature enough
> it becomes a standalone package.
>
>
> Do you see a problem with that path and/or would you want that path
> not to be used in qemu/kvm ?
> While the library is complete enough for the use and the features that
> qemu/kvm needs, I think there are a lot
> of work in other areas that kvm/qemu does not need/use before it can
> become standalone.
>
>
> regards
> ronnie sahlberg
>    

      reply	other threads:[~2010-12-03 21:22 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-03 11:09 [Qemu-devel] [Patch 0/14] builtin iscsi support ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 01/14] ./block/iscsi/init.c ronniesahlberg
2010-12-03 20:32   ` Stefan Hajnoczi
2010-12-03 21:23     ` ronnie sahlberg
2010-12-06 16:09       ` Kevin Wolf
2010-12-03 11:09 ` [Qemu-devel] [PATCH 02/14] ./block/iscsi/socket.c ronniesahlberg
2010-12-04 13:06   ` Stefan Hajnoczi
2010-12-03 11:09 ` [Qemu-devel] [PATCH 03/14] ./block/iscsi/login.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 04/14] ./block/iscsi/discovery.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 05/14] ./block/iscsi/crc32c.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 06/14] ./block/iscsi/nop.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 07/14] ./block/iscsi/pdu.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 08/14] ./block/iscsi/sscsi-command.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 09/14] ./block/iscsi/scsi-lowlevel.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 10/14] ./block/iscsi/sync.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 11/14] ./block/iscsi/connect.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 12/14] ./block/iscsi/*.h ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 13/14] ./block/iscsi.c ronniesahlberg
2010-12-03 11:09 ` [Qemu-devel] [PATCH 14/14] iscsi support ronniesahlberg
2010-12-03 11:42 ` [Qemu-devel] [Patch 0/14] builtin " Stefan Hajnoczi
2010-12-03 11:50   ` ronnie sahlberg
2010-12-03 15:05 ` Anthony Liguori
2010-12-03 19:57   ` ronnie sahlberg
2010-12-03 21:22     ` Anthony Liguori [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=4CF95F7A.6090104@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=qemu-devel@nongnu.org \
    --cc=ronniesahlberg@gmail.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.