qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: John Williams <john.williams@petalogix.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: Grant Likely <grant.likely@secretlab.ca>,
	stefanha@linux.vnet.ibm.com, qemu-devel@nongnu.org,
	Peter Crosthwaite <peter.crosthwaite@petalogix.com>,
	michal.simek@petalogix.com,
	"Edgar E. Iglesias" <edgar.iglesias@gmail.com>
Subject: Re: [Qemu-devel] [RFC PATCH V1 00/14] Dynamic machine model creation from device trees
Date: Thu, 8 Sep 2011 10:29:01 +1000	[thread overview]
Message-ID: <CACPZLaVJ4WkGbXKg3uq4n4Kzy1Wi6Ok6jR9ZMC9ceQ5O3dj2Ew@mail.gmail.com> (raw)
In-Reply-To: <CACPZLaWp=evaua8yA2MhzQ=FfN9GheuPaUUcSyd0=MwGazgGeg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2330 bytes --]

On Fri, Sep 2, 2011 at 12:45 PM, John Williams
<john.williams@petalogix.com>wrote:

> On Fri, Aug 26, 2011 at 7:17 AM, Anthony Liguori <anthony@codemonkey.ws>wrote:
>
>> On 08/25/2011 03:20 PM, Edgar E. Iglesias wrote:
>>
>>> On Thu, Aug 25, 2011 at 02:54:13PM -0500, Anthony Liguori wrote:
>>>
>>>> On 08/25/2011 02:10 PM, Edgar E. Iglesias wrote:
>>>>
>>>>> This is the goal of QOM except it does this by fixing the problems
>>>>>> in qdev instead of adding another layer on top of things.
>>>>>>
>>>>>
>>>>>
>>>>> Then maybe the FDT machinery could be respinned to work on top of your
>>>>> QOM
>>>>> objects?
>>>>>
>>>>> Or are FDT's a complete no go? So external conversion is the only
>>>>> option?
>>>>>
>>>>
>>>> No, DTS is fine but not as proposed.  You shouldn't mix the logic of
>>>> creating the nodes in the tree with the format of how you're
>>>> describing what nodes to be there.
>>>>
>>>
>>> Thanks. Would you mind spending a few lines on how far you've gotten with
>>> QOM and if there is, where to find more info about it (sorry, I havent
>>> been
>>> following it at all).
>>>
>>
>> Stay tuned.  I'm going to spend the day tomorrow getting the next series
>> ready and writing some stuff on the wiki.
>>
>
> It seems that the main issue raised is not about the quality of the work
> being contributed, nor the overall intent, but rather an expression that it
> might be better done building on a layer which itself is still a work in
> progress, I'd like to propose a compromise.
>
> This is a long term investment for us, we have two working architectures,
> numerous users, and immediate plans to add support for a 3rd architecture.
>  We are committed to doing it the right way, and letting people other than
> our immediate customers benefit from our work. However, there is also a
> significant cost to us maintaining this out-of-tree - something breaks every
> time we pull from mainline.
>
> So, may I ask that the code be reviewed on its technical merit as it stands
> today, and if that is suitable then it be merged, with a commitment from us
> that when there is suitable plumbing in place our work will be refactored
> around that?  Indeed, the process of us doing so will probably provide
> useful feedback in the design of these underlying layers.
>

Any thoughts on our proposal?

Thanks,

John

[-- Attachment #2: Type: text/html, Size: 3491 bytes --]

      reply	other threads:[~2011-09-08  0:29 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-25  6:41 [Qemu-devel] [RFC PATCH V1 00/14] Dynamic machine model creation from device trees Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 01/14] qemu-coroutine: Add simple work queue support Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 02/14] device_tree: Extended interface for fdt_generic Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 03/14] fdt_generic: First revision Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 04/14] xilinx_uartlite: Added fdt gen. platform support Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 05/14] pflash_cfi01: Added fdt generic " Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 06/14] qdev: Added fn for querying device property types Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 07/14] fdt_generic_qdev: first revision Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 08/14] xilinx_timer: Added fdt_generic platform support Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 09/14] xilinx_intc: Added fdt generic " Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 10/14] xilinx_ethlite: " Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 11/14] vl.c: Added hw_dtb/kern_dtb command line opts Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 12/14] microblaze: Make the MSR PVR bit non writable Peter A. G. Crosthwaite
2011-08-25  9:34   ` Peter Maydell
2011-08-25 10:17     ` Peter Crosthwaite
2011-08-25 18:28       ` Edgar E. Iglesias
2011-08-25 21:04       ` Edgar E. Iglesias
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 13/14] microblaze: Add an MSR_PVR constant and use it Peter A. G. Crosthwaite
2011-08-25  6:41 ` [Qemu-devel] [RFC PATCH V1 14/14] microblaze_generic_fdt: first revision Peter A. G. Crosthwaite
2011-08-25 13:27 ` [Qemu-devel] [RFC PATCH V1 00/14] Dynamic machine model creation from device trees Anthony Liguori
2011-08-25 15:43   ` Peter Crosthwaite
2011-08-25 15:59     ` Paolo Bonzini
2011-08-25 16:04     ` Anthony Liguori
2011-08-25 19:10       ` Edgar E. Iglesias
2011-08-25 19:54         ` Anthony Liguori
2011-08-25 20:20           ` Edgar E. Iglesias
2011-08-25 21:17             ` Anthony Liguori
2011-09-02  2:45               ` John Williams
2011-09-08  0:29                 ` John Williams [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=CACPZLaVJ4WkGbXKg3uq4n4Kzy1Wi6Ok6jR9ZMC9ceQ5O3dj2Ew@mail.gmail.com \
    --to=john.williams@petalogix.com \
    --cc=anthony@codemonkey.ws \
    --cc=edgar.iglesias@gmail.com \
    --cc=grant.likely@secretlab.ca \
    --cc=michal.simek@petalogix.com \
    --cc=peter.crosthwaite@petalogix.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@linux.vnet.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).