Received: by 10.192.165.148 with SMTP id m20csp2313765imm; Sun, 6 May 2018 11:46:49 -0700 (PDT) X-Google-Smtp-Source: AB8JxZoTB0KBeDQtV27yC/pSIJzv2JK9gtcyksvdDb52RkDhdLJZSBgU9ce/IJAmT2+cVvopLvUG X-Received: by 2002:a17:902:680c:: with SMTP id h12-v6mr36038867plk.113.1525632409891; Sun, 06 May 2018 11:46:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1525632409; cv=none; d=google.com; s=arc-20160816; b=fr+DUNNUzwrJGHKCzcndbI/CPrq/29VHCpp82QUrNpMI0GiPz1D7N/BdHfnoCWxxO/ /iSigOkLHbbR5Lpbml0sppGsTgjLHsAujFgJwLQkDQlsUs0skBjatQJRa65t1Q64XtmH 6uT6sTLQh8orIGNW37/WudeEekQ6PhyeruWjIc0Jb0pI72nbfFZ+LM5oT6bYXLVHgXqz OY0BM+goMW51sdGbanU+0sX+nbp3E4dgAZHN2GRQDPxpv0s2O7bKZzgcAWv273fdMm/Y GFkkU1geY0wGdu7sjHU1q7IR37kLwgKPH1LCcx01foRIcGAbph79PFvRWa+ZUE8FgHwz RpuQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :arc-authentication-results; bh=TmKdTwdPE8FraUrrUSpvK7p0n0j75/1Tb93vNA8uxpo=; b=wC7d9MxCllQ08cmk1AhRYZ9Z/L6lK1T9yjNEYyeGyZiZx6WlQemYUart7k3Mq9EH3p Ysy7ddj8XBTbDC7wB1sU0a/7lBj5Sn7Nu8yGl84K7E+M2BGH+oQE1U7pGUHTERP+DUmd JoOsAcyeqw7wV/gw+m0Z4DyL3RtrUHU1qoeqrNFoPxJ5CKOcXnGD1/bBrdof6vLwZ2Yb vtd4Ji5vCmloTicROW3IK+1823V5QkryUbXCKpdAiVfBfNPjwikhQvhiwx6XWnK6FunX hBZpVtox0mMMSl8pSkVllolpbrReFIyxqCChcQf8E1VXFOWOOwwilRJle/5V2avfLaDd JfhA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c2-v6si20487621plb.77.2018.05.06.11.46.22; Sun, 06 May 2018 11:46:49 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751811AbeEFSqM (ORCPT + 99 others); Sun, 6 May 2018 14:46:12 -0400 Received: from mslow2.mail.gandi.net ([217.70.178.242]:45108 "EHLO slow1-d.mail.gandi.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751771AbeEFSqK (ORCPT ); Sun, 6 May 2018 14:46:10 -0400 Received: from relay10.mail.gandi.net (unknown [217.70.178.230]) by slow1-d.mail.gandi.net (Postfix) with ESMTP id 0C0B13A0B0C; Sun, 6 May 2018 20:24:34 +0200 (CEST) Received: from localhost (unknown [212.92.108.144]) (Authenticated sender: amit@amitshah.net) by relay10.mail.gandi.net (Postfix) with ESMTPSA id 112C5240005; Sun, 6 May 2018 20:24:30 +0200 (CEST) Date: Sun, 6 May 2018 20:24:30 +0200 From: Amit Shah To: "Michael S. Tsirkin" Cc: linux-kernel@vger.kernel.org, Arnd Bergmann , Greg Kroah-Hartman , virtualization@lists.linux-foundation.org, stable@vger.kernel.org, Tiwei Bie , Jason Wang Subject: Re: [PATCH 0/6] virtio-console: spec compliance fixes Message-ID: <20180506182430.GB19628@grmbl.mre> References: <1524248223-393618-1-git-send-email-mst@redhat.com> <20180424214104-mutt-send-email-mst@kernel.org> <20180503034529.GA16676@grmbl.mre> <20180503222702-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180503222702-mutt-send-email-mst@kernel.org> X-Spam-Level: Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On (Thu) 03 May 2018 [22:28:32], Michael S. Tsirkin wrote: > On Thu, May 03, 2018 at 05:45:29AM +0200, Amit Shah wrote: > > (apologies if you received a dup) > > > > On (Tue) 24 Apr 2018 [21:41:29], Michael S. Tsirkin wrote: > > > On Fri, Apr 20, 2018 at 09:17:59PM +0300, Michael S. Tsirkin wrote: > > > > Turns out virtio console tries to take a buffer out of an active vq. > > > > Works by sheer luck, and is explicitly forbidden by spec. And while > > > > going over it I saw that error handling is also broken - > > > > failure is easy to trigger if I force allocations to fail. > > > > > > > > Lightly tested. > > > > > > Amit - any feedback before I push these patches? > > > > The changes look good spec-wise. > > > > There are a bunch of tests in avocado-vt that test virtio-console > > functionality. Can you give those a try before pushing? > > > > Amit > > I pushed before I did that test, will try to find the time later. BTW > do you still want to be on the maintainers list? Yes, I want to be on the maintainers list; but won't mind co-maintainers. The recent changes seem to be spec-related, and I'd expect you to have a good handle on that anyway. Amit -- http://amitshah.net/