From mboxrd@z Thu Jan 1 00:00:00 1970 From: Markus Armbruster Subject: Re: [Qemu-devel] [PATCH V6 3/4] qmp, nmi: convert do_inject_nmi() to QObject Date: Fri, 25 Feb 2011 10:54:07 +0100 Message-ID: References: <4D412AD2.1090509@cn.fujitsu.com> <4D655F32.30307@codemonkey.ws> <4D666DE5.9090800@linux.vnet.ibm.com> <4D66917D.2030206@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Lai Jiangshan , Lai Jiangshan , kvm@vger.kernel.org, qemu-devel@nongnu.org, Luiz Capitulino , Avi Kivity To: Anthony Liguori Return-path: Received: from mx1.redhat.com ([209.132.183.28]:62429 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932375Ab1BYJyY (ORCPT ); Fri, 25 Feb 2011 04:54:24 -0500 In-Reply-To: <4D66917D.2030206@linux.vnet.ibm.com> (Anthony Liguori's message of "Thu, 24 Feb 2011 11:12:29 -0600") Sender: kvm-owner@vger.kernel.org List-ID: Anthony Liguori writes: > On 02/24/2011 10:20 AM, Markus Armbruster wrote: >> Anthony Liguori writes: >> >> >>> On 02/24/2011 02:33 AM, Markus Armbruster wrote: >>> >>>> Anthony Liguori writes: [...] >>>>> Please describe all expected errors. >>>>> >>>>> >>>> Quoting qmp-commands.hx: >>>> >>>> 3. Errors, in special, are not documented. Applications should NOT check >>>> for specific errors classes or data (it's strongly recommended to only >>>> check for the "error" key) >>>> >>>> Indeed, not a single error is documented there. This is intentional. >>>> >>>> >>> Yeah, but we're not 0.14 anymore and for 0.15, we need to document >>> errors. If you are suggesting I send a patch to remove that section, >>> I'm more than happy to. >>> >> Two separate issues here: 1. Are we ready to commit to the current >> design of errors, and 2. Is it fair to reject Lai's patch now because he >> doesn't document his errors. >> >> I'm not commenting on 1. here. >> >> Regarding 2.: rejecting a patch because it doesn't document an aspect >> that current master intentionally leaves undocumented is not how you >> treat contributors. At least not if you want any other than certified >> masochists who enjoy pain, and professionals who get adequately >> compensated for it. >> >> Lead by example, not by fiat. >> > > http://repo.or.cz/w/qemu/aliguori.git/blob/refs/heads/glib:/qmp-schema.json > > I am in the process of documenting the errors of every command. It's > a royal pain but I'm going to document everything we have right now. > It's actually the last bit of work I need to finish before sending > QAPI out. > > So for new commands being added, it would be hugely helpful for the > authors to document the errors such that I don't have to reverse > engineer all of the possible error conditions. The moment this lands in master, you can begin to demand error descriptions from contributors. Until then, I'll NAK error descriptions in qmp-commands.txt. We left them undocumented there for good reasons: >>>> Once we have an error design in place that has a reasonable hope to >>>> stand the test of time, and have errors documented for at least some of >>>> the commands here, we can start to require proper error documentation >>>> for new commands. But not now. I won't NAK non-normative error descriptions, say in commit messages, or in comments. And I won't object to you asking for them. But I feel you really shouldn't make it a condition for committing patches. Especially not for simple patches that have been on list for months.