From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: Rusty away 18th September -- 11th October Date: Sun, 14 Sep 2014 09:21:17 +0300 Message-ID: <20140914062116.GA23868@redhat.com> References: <87oaunht4b.fsf@rustcorp.com.au> <20140911114351.GA25736@redhat.com> <87tx4dhbks.fsf@rustcorp.com.au> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <87tx4dhbks.fsf@rustcorp.com.au> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Rusty Russell Cc: netdev , Linus Torvalds , LKML , virtualization@lists.linux-foundation.org List-Id: virtualization@lists.linuxfoundation.org On Fri, Sep 12, 2014 at 10:58:03AM +0930, Rusty Russell wrote: > "Michael S. Tsirkin" writes: > > On Thu, Sep 11, 2014 at 10:26:52AM +0930, Rusty Russell wrote: > >> Hi all, > >> > >> Probably won't read mail. Linus, I'll have pull requests early > >> next week; if there's anything needed I'm sure Michael Tsirkin can > >> handle it. > > > > Sure. > > Rusty, there's a small chance virtio 1.0 bits will be ready in time. > > I started working on them based on your virtio-pci-new-layout branch. > > > > If ready before the merge window, are you ok with me merging this > > support (without you having the opportunity to review first)? > > Sorry, absolutely not. I *really* want to review this in depth; if we > make a mistake, it's going to hurt us significantly. Right but it's only a merge window - we can fix bugs and even disable the new driver afterwards, can't we? > And until we have the qemu bits ready, it's really hard to tell if we've > got this right. Sure, I meant if qemu bits are ready too, and if things work together. It wouldn't do to merge a driver that isn't ready. > So I'd rather delay and make sure we're solid. > > Thanks, > Rusty. Something else we can do, is refactoring that splits driver out to common and legacy bits, and changes APIs. No? -- MSt