From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:47305) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJmJU-0005Ax-2Z for qemu-devel@nongnu.org; Thu, 04 Oct 2012 10:23:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TJmIy-0002nO-V0 for qemu-devel@nongnu.org; Thu, 04 Oct 2012 10:22:08 -0400 Received: from e23smtp04.au.ibm.com ([202.81.31.146]:56955) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJmIy-0002mX-DN for qemu-devel@nongnu.org; Thu, 04 Oct 2012 10:21:36 -0400 Received: from /spool/local by e23smtp04.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 5 Oct 2012 00:18:13 +1000 Received: from d23av03.au.ibm.com (d23av03.au.ibm.com [9.190.234.97]) by d23relay05.au.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id q94EBgvH62587102 for ; Fri, 5 Oct 2012 00:11:44 +1000 Received: from d23av03.au.ibm.com (loopback [127.0.0.1]) by d23av03.au.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id q94ELOTI018649 for ; Fri, 5 Oct 2012 00:21:24 +1000 From: Anthony Liguori In-Reply-To: <506D56B1.8090804@web.de> References: <20121003105255.972669952@amt.cnet> <20121003105509.391284251@amt.cnet> <87fw5vhb5a.fsf@codemonkey.ws> <20121003150341.GA15164@amt.cnet> <506C5DED.7000705@web.de> <878vbn4gs9.fsf@codemonkey.ws> <506C74E0.8080409@web.de> <20121003182652.GA32381@amt.cnet> <506D56B1.8090804@web.de> Date: Thu, 04 Oct 2012 09:21:16 -0500 Message-ID: <87626qiahv.fsf@codemonkey.ws> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Subject: Re: [Qemu-devel] [patch 2/6] Use machine options to emulate -no-kvm-irqchip List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Kiszka , Marcelo Tosatti Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org, Gerd Hoffmann Jan Kiszka writes: > On 2012-10-03 20:26, Marcelo Tosatti wrote: >> On Wed, Oct 03, 2012 at 07:24:48PM +0200, Jan Kiszka wrote: >>> On 2012-10-03 19:16, Anthony Liguori wrote: >>>> Jan Kiszka writes: >>>> >>>>> On 2012-10-03 17:03, Marcelo Tosatti wrote: >>>>>> On Wed, Oct 03, 2012 at 09:40:17AM -0500, Anthony Liguori wrote: >>>>>>> Marcelo Tosatti writes: >>>>>>> >>>>>>>> Commit 3ad763fcba5bd0ec5a79d4a9b6baeef119dd4a3d from qemu-kvm.git. >>>>>>>> >>>>>>>> From: Jan Kiszka >>>>>>>> >>>>>>>> Upstream is moving towards this mechanism, so start using it in qemu-kvm >>>>>>>> already to configure the specific defaults: kvm enabled on, just like >>>>>>>> in-kernel irqchips. >>>>>>>> >>>>>>>> Signed-off-by: Marcelo Tosatti >>>>>>> >>>>>>> >>>>>>> Reviewed-by: Anthony Liguori >>>>>>> >>>>>>> Although it's a little odd to have From: Jan without a SoB... >>>>>> >>>>>> Agree, Jan can you ACK? >>>>> >>>>> I wasn't able to join the call yesterday: Is there a removal schedule >>>>> associated with those switches? Also, why pushing things upstream, even >>>>> when only for one release, that have been loudly deprecated for a while >>>>> in qemu-kvm? Some switches are lacking deprecated warnings on the >>>>> console, and -no-kvm is missing completely. I tend to focus on patch 1 & >>>>> 5, dropping the rest - based on relevance for production use. >>>> >>>> The distros need to keep these flags to do the switch. >>> >>> Why? Should be documented in commit log. >>> >>>> I see no point >>>> in deprecating them since they're trivially easy to maintain. >>> >>> Given the level of cr** we already have in the command line, they are >>> kind of noise, yes. But even then, these patches are not consistent as >>> pointed out above. >>> >>> Also, they should not be documented to avoid being spread. That's what >>> we did with other deprecated switches in QEMU. >>> >>> Jan >> >> Jan, >> >> You're comments to the patch are: >> >> - No documentation. > > See e.g. how -M is handled in qemu-options.hx. > >> - Expiration date. > > Anthony said "forever", but I think we should remove all those that > issue deprecation warnings after 1-2 years. > >> - Changelog explaining what?? (didnt get that). Perhaps better changelog >> in general? > > I'm still failing to understand who could depend on -no-kvm-irqchip or > -no-kvm-pit. And I don't understand why -no-kvm was not included. Soe > the reasons for include -X should be provided. Also check your patch > subjects again, at least one was wrong. -no-kvm should be included too. I just ran across a user that was injecting '-no-kvm-irqchip' in their libvirt XML via a custom attribute. It turned out it was to work around broken MSI support in their funky guest they were running. It was the wrong solution to the problem but they were doing it regardless. The point is, there are users in the wild using these options. There's no reason to remove them if they are trivial to maintain (and they are in their current form). Regards, Anthony Liguori > > Jan