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 --]
prev parent 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).