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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 37318FD376A for ; Wed, 25 Feb 2026 16:38:57 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1vvHuN-0002Ff-V0; Wed, 25 Feb 2026 11:38:43 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1vvHuL-0002F1-Ve for qemu-rust@nongnu.org; Wed, 25 Feb 2026 11:38:41 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1vvHuI-0000GN-Lz for qemu-rust@nongnu.org; Wed, 25 Feb 2026 11:38:41 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1772037515; h=from:from:reply-to: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=VdZHvlU8W6vWLV70h737ZMrXZc3Bsg0j2kz5VPlt6Ho=; b=Q+dmuVfDAeo3IR226Jwx3sNslhpFL37sVxj13R9KT4txHEqGBE49QjsMyIA/Eroun4Ef+V FNXr4ZWI1NEWaI2aZciJCQNvq3O9+GXv8o4RrL2XfFJtOHmu3cyrvuasFm6bBgD0/rFxxc xFbcKI4nJF98pTKbwVVjAe9VNA3oar8= Received: from mx-prod-mc-08.mail-002.prod.us-west-2.aws.redhat.com (ec2-35-165-154-97.us-west-2.compute.amazonaws.com [35.165.154.97]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-45-gnOxJW8yPIaM4znBGcZaHQ-1; Wed, 25 Feb 2026 11:38:32 -0500 X-MC-Unique: gnOxJW8yPIaM4znBGcZaHQ-1 X-Mimecast-MFC-AGG-ID: gnOxJW8yPIaM4znBGcZaHQ_1772037510 Received: from mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.17]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-08.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id AADC41800341; Wed, 25 Feb 2026 16:38:30 +0000 (UTC) Received: from redhat.com (unknown [10.45.225.165]) by mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 5A7071955F43; Wed, 25 Feb 2026 16:38:25 +0000 (UTC) Date: Wed, 25 Feb 2026 16:38:21 +0000 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= To: Peter Maydell Cc: qemu-devel@nongnu.org, Manos Pitsidianakis , Stefan Weil , "Dr. David Alan Gilbert" , Pierrick Bouvier , devel@lists.libvirt.org, Philippe =?utf-8?Q?Mathieu-Daud=C3=A9?= , =?utf-8?Q?Marc-Andr=C3=A9?= Lureau , Hanna Reitz , Kevin Wolf , qemu-block@nongnu.org, qemu-rust@nongnu.org, Paolo Bonzini , Markus Armbruster , Gerd Hoffmann , Christian Schoenebeck , Richard Henderson Subject: Re: [PATCH v6 25/27] util: add support for formatting a program name in messages Message-ID: References: <20260211152508.732487-1-berrange@redhat.com> <20260211152508.732487-26-berrange@redhat.com> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/2.2.14 (2025-02-20) X-Scanned-By: MIMEDefang 3.0 on 10.30.177.17 X-Mimecast-MFC-PROC-ID: RyghYceY_KbQNKPX0gJh-iGL1xbTTSZfFwKuy_z-_ns_1772037510 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=170.10.129.124; envelope-from=berrange@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -5 X-Spam_score: -0.6 X-Spam_bar: / X-Spam_report: (-0.6 / 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, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.734, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.78, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-rust@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: QEMU Rust-related patches and discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Errors-To: qemu-rust-bounces+qemu-rust=archiver.kernel.org@nongnu.org Sender: qemu-rust-bounces+qemu-rust=archiver.kernel.org@nongnu.org On Thu, Feb 19, 2026 at 10:23:36AM +0000, Peter Maydell wrote: > On Wed, 11 Feb 2026 at 15:29, Daniel P. Berrangé wrote: > > > > The error_report function can include the program name in any > > messages it prints. The qemu_log function has no equivalent > > behaviour. > > > > This introduces support for a "program name" in the new > > messages API, which will be included by default for all > > binaries. > > > > This change tweaks the output of the error_report function, > > adding a space between the program name and the location > > info. The qemu_log function will gain the program name. This > > can be easily seen with the 'log' trace backend, and how it > > is now more closely matching error_report output. > > > > Before: > > > > # qemu-system-x86_64 -object tls-creds-x509,id=t0,dir=fish -d 'trace:qcrypto*' > > qcrypto_tls_creds_x509_load TLS creds x509 load creds=0x5584e13937f0 dir=fish > > qcrypto_tls_creds_get_path TLS creds path creds=0x5584e13937f0 filename=ca-cert.pem path= > > qemu-system-x86_64: Unable to access credentials fish/ca-cert.pem: No such file or directory > > > > After: > > > > # qemu-system-x86_64 -object tls-creds-x509,id=t0,dir=fish -d 'trace:qcrypto*' > > qemu-system-x86_64: qcrypto_tls_creds_x509_load TLS creds x509 load creds=0x5584e13937f0 dir=fish > > qemu-system-x86_64: qcrypto_tls_creds_get_path TLS creds path creds=0x5584e13937f0 filename=ca-cert.pem path= > > qemu-system-x86_64: Unable to access credentials fish/ca-cert.pem: No such file or directory > > On the other hand if you're using the logs for debugging > then you now have an extra big pointless prefix on them that > you have to turn off. Especially if you're logging to a file > that's a lot of extra garbage in the output. I'm not really > looking forward to now having to give QEMU an extra long > and unwieldy command line argument every time I want to > do debug or tracepoint logging. > > Why do we care about the qemu_log output matching the > error-report output? The logs are expected to be > quite frequent, to only be there if you've explicitly > turned them on, and to be usually directed to a log file. Personally I never use the log-to-file facility. Instead typical use case is relying on the trace points 'log' facility as illustrated in the commit message example to dump to stderr for simple debugging, and having consistent formatting between the logs & errors is useful there. In effect this patch series is implicitly viewing the error report output as a being a variant of log output with a level of "ERROR", despite us having completely separate logic for logs vs errors internally. In this particular patch, I didn't especially have a use case for the program name in the log output, it instead of fell out of the general goal of using the same formatting logic for both subsystems. I'll think about how to alter this. > The error reporting is more likely to be infrequent, to > be on stdout, and to be important/fatal things. > > -- PMM > With regards, Daniel -- |: https://berrange.com ~~ https://hachyderm.io/@berrange :| |: https://libvirt.org ~~ https://entangle-photo.org :| |: https://pixelfed.art/berrange ~~ https://fstop138.berrange.com :|