From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54121) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aLcJv-0006qc-0c for qemu-devel@nongnu.org; Tue, 19 Jan 2016 14:52:03 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aLcJt-0002iT-OG for qemu-devel@nongnu.org; Tue, 19 Jan 2016 14:52:02 -0500 Received: from mail-ig0-x241.google.com ([2607:f8b0:4001:c05::241]:35971) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aLcJt-0002i9-Hl for qemu-devel@nongnu.org; Tue, 19 Jan 2016 14:52:01 -0500 Received: by mail-ig0-x241.google.com with SMTP id o2so10269042iga.3 for ; Tue, 19 Jan 2016 11:52:01 -0800 (PST) MIME-Version: 1.0 Sender: alistair23@gmail.com In-Reply-To: <20160108110500.GB29396@toto> References: <20160108110500.GB29396@toto> From: Alistair Francis Date: Tue, 19 Jan 2016 11:51:31 -0800 Message-ID: Content-Type: text/plain; charset=UTF-8 Subject: Re: [Qemu-devel] [PATCH v1 00/15] data-driven device registers List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Edgar E. Iglesias" Cc: Edgar Iglesias , Peter Maydell , "qemu-devel@nongnu.org Developers" , Alistair Francis , Peter Crosthwaite , =?UTF-8?Q?Andreas_F=C3=A4rber?= On Fri, Jan 8, 2016 at 3:05 AM, Edgar E. Iglesias wrote: > On Fri, Jan 08, 2016 at 10:40:28AM +0000, Peter Maydell wrote: >> On 8 January 2016 at 00:39, Alistair Francis >> wrote: >> > On Wed, Dec 16, 2015 at 8:33 AM, Alistair Francis >> > wrote: >> >> On Tue, Dec 15, 2015 at 1:56 PM, Peter Maydell wrote: >> >>> On 15 December 2015 at 20:52, Peter Crosthwaite >> >>> wrote: >> >>>> It needs to exist before it can be used so there is a bit of a chicken >> >>>> and egg problem there. >> > >> > No one seems to be jumping at reviewing this. Can we just send a pull request? >> >> I don't necessarily require review [*]. I would like *somebody* other >> than you Xilinx folk to say "yes, I think I would use this for >> modelling devices". Otherwise all we have is "weird thing used >> only in two or three Xilinx devices and nowhere else", which I'm >> a bit reluctant to let into the tree. We already have a pretty >> wide divergence in how devices look just based on the various >> transitions from older to newer qdev/QOM/etc that are not complete. >> >> [*] by which I mean, I will review this series if you can find >> somebody else who's going to say they'd use it. >> > > Hi, > > I have two general comments to the series. > > 1. I think we need to do something to allow mem-attributes to be passed to the reg-access callbacks and possibly also to add a way to filter on attrs in the data structure. I would prefer to add this in the future. We are already having enough trouble getting it accepted now and this will add complexity. In saying that, it shouldn't be too hard to add in the future. The basic infrastructure is all there. > > 2. We had trouble in the Xilinx tree with the number of memory regions created when using the style were each reg becomes an MR. I think that style should either be disallowed or we need to fix the low limit (IIRC it was at around 1K MRs/Regs). This I can help with. I am sending a V2 which doesn't print the memory regions with infro mtree. When you start getting hundreds of registers this becomes really painful. I can't see a nice way to avoid actually adding the memory subregions though. It is just a linked list, what limit is there for memory subregions? Thanks, Alistair > > I'm part of the Xilinx team so I this outside of Peters review request but anyawy.... > > Cheers, > Edgar >