From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.3 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A4718C4363C for ; Fri, 2 Oct 2020 17:33:25 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 2BB0320795 for ; Fri, 2 Oct 2020 17:33:25 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="NeK6t4d3" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 2BB0320795 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([::1]:55390 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kOOvs-0007Rn-11 for qemu-devel@archiver.kernel.org; Fri, 02 Oct 2020 13:33:24 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:42990) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kOOos-0001WA-80 for qemu-devel@nongnu.org; Fri, 02 Oct 2020 13:26:10 -0400 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:28196) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1kOOoq-0002Gc-AC for qemu-devel@nongnu.org; Fri, 02 Oct 2020 13:26:09 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1601659566; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=F1WP+68csPIP6ZYkXawaQJF2HFQzp02bl9fuYPA2cSs=; b=NeK6t4d3oJeblrignkFhM4rjBC2K5VYT/E5G514ILwlwa/al/RUeqUGTHjoZbtsLBds0O6 H3JxF6WTiHk/9okCW/PXJJ/jWoLCOw3PI603ypkspLKt+Tcx7ZoDEXPUrURwTaIMC+y5tp AdJRzC2mkGOSIf+A25kBq8q3zZX5qlQ= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-439-c6LGK15iNDazI2lvlGqT3g-1; Fri, 02 Oct 2020 13:26:04 -0400 X-MC-Unique: c6LGK15iNDazI2lvlGqT3g-1 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 0E69364097 for ; Fri, 2 Oct 2020 17:26:03 +0000 (UTC) Received: from [10.40.192.152] (unknown [10.40.192.152]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 105EC5C1DC; Fri, 2 Oct 2020 17:26:01 +0000 (UTC) Subject: Re: [PULL v8 00/86] Misc QEMU patches for 2020-09-24 To: Eduardo Habkost References: <20200930195850.278104-1-pbonzini@redhat.com> <20201002162208.GB7303@habkost.net> From: =?UTF-8?B?TWljaGFsIFByw612b3puw61r?= Message-ID: Date: Fri, 2 Oct 2020 19:26:00 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 MIME-Version: 1.0 In-Reply-To: <20201002162208.GB7303@habkost.net> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=mprivozn@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=216.205.24.124; envelope-from=mprivozn@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/10/02 01:13:31 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -23 X-Spam_score: -2.4 X-Spam_bar: -- X-Spam_report: (-2.4 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, NICE_REPLY_A=-0.256, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H5=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Paolo Bonzini , John Snow , qemu-devel@nongnu.org Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On 10/2/20 6:22 PM, Eduardo Habkost wrote: > On Fri, Oct 02, 2020 at 05:58:55PM +0200, Michal Prívozník wrote: >> On 9/30/20 9:58 PM, Paolo Bonzini wrote: >>> >>> Eduardo Habkost (10): >> >>> docs: Create docs/devel/qom.rst >> >> cd442a45db60a1a72fcf980c24bd1227f13f8a87 is the first bad commit >> >> Sorry for noticing this earlier, but is this known? The build starts failing >> for me after this commit: >> >> /usr/bin/sphinx-build -Dversion=5.1.50 -Drelease= -W -Ddepfile=docs/devel.d >> -Ddepfile_stamp=docs/devel.stamp -b html -d >> /home/zippy/work/qemu/qemu.git/build/docs/devel.p >> /home/zippy/work/qemu/qemu.git/docs/devel >> /home/zippy/work/qemu/qemu.git/build/docs/devel >> Running Sphinx v3.2.1 >> building [mo]: targets for 0 po files that are out of date >> building [html]: targets for 20 source files that are out of date >> updating environment: [new config] 20 added, 0 changed, 0 removed >> reading sources... [100%] testing >> >> >> >> >> Warning, treated as error: >> /home/zippy/work/qemu/qemu.git/docs/../include/qom/object.h:747:Error in >> declarator >> If declarator-id with parameters (e.g., 'void f(int arg)'): >> Invalid C declaration: Expected identifier in nested name. [error at 24] >> object_initialize_child ( parent, propname, child, type) >> ------------------------^ >> If parenthesis in noptr-declarator (e.g., 'void (*f(int arg))(double)'): >> Error in declarator or parameters >> Invalid C declaration: Expecting "(" in parameters. [error at 32] >> object_initialize_child ( parent, propname, child, type) >> --------------------------------^ >> >> make[1]: *** [Makefile.ninja:9898: docs/devel.stamp] Error 2 >> make[1]: *** Deleting file 'docs/devel.stamp' >> make[1]: Leaving directory '/home/zippy/work/qemu/qemu.git/build' >> make: *** [GNUmakefile:11: all] Error 2 > > I can't reproduce it using Sphinx v2.2.2. I'm still trying to > understand what exactly the error means. > Same here. > I really wish we used virtualenv + requirements.txt to require a > specific version of Sphinx instead of wasting time dealing a wide > range of Sphinx versions. > I already have a patch that I keep locally to build with v3: diff --git a/docs/qemu-option-trace.rst.inc b/docs/qemu-option-trace.rst.inc index 7e09773a9c..ae83f6a1a8 100644 --- a/docs/qemu-option-trace.rst.inc +++ b/docs/qemu-option-trace.rst.inc @@ -1,7 +1,7 @@ Specify tracing options. -.. option:: [enable=]PATTERN +.. option:: enable=PATTERN Immediately enable events matching *PATTERN* (either event name or a globbing pattern). This option is only That said, I'm not objecting to requiring v2 for now and switching to v3 later. But interestingly, through trial and error I've came across this hack, which allows me to build again. I have no idea why it works: diff --git i/include/qom/object.h w/include/qom/object.h index 27aaa67e63..59c729ebb7 100644 --- i/include/qom/object.h +++ w/include/qom/object.h @@ -762,13 +762,14 @@ bool object_initialize_child_with_propsv(Object *parentobj, * child, sizeof(*child), type, * &error_abort, NULL) */ -#define object_initialize_child(parent, propname, child, type) \ - object_initialize_child_internal((parent), (propname), \ - (child), sizeof(*(child)), (type)) void object_initialize_child_internal(Object *parent, const char *propname, void *child, size_t size, const char *type); +#define object_initialize_child(parent, propname, child, type) \ + object_initialize_child_internal((parent), (propname), \ + (child), sizeof(*(child)), (type)) + /** * object_dynamic_cast: * @obj: The object to cast.