From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.groups.io with SMTP id smtpd.web10.9607.1621773074880664293 for ; Sun, 23 May 2021 05:31:15 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=YfRNMmPr; spf=pass (domain: redhat.com, ip: 170.10.133.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1621773074; 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=lr/xtVifTU0MSXSvxpPW9sHlXr/vjI1QGC5BGj/aOOw=; b=YfRNMmPriyOFEisbnSciRliagAS6CUqI1aVuyW1e3IIso4bje5k7nY7NRB8R/97Fuf7gad 5TcTLFB2ZJ0MMYE7t+kIybPeOv4RKbmZRJtDQRSj9OJCAsIIWzgFlYtFKX0lL2kSe0BnSN hQn6L2lBvkIQYQU3CTZ7q41EmsxV068= 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-564-iLPk2yyGMgGJgneBsBG1xg-1; Sun, 23 May 2021 08:31:10 -0400 X-MC-Unique: iLPk2yyGMgGJgneBsBG1xg-1 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 1DE18801817; Sun, 23 May 2021 12:31:09 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-8.ams2.redhat.com [10.36.112.8]) by smtp.corp.redhat.com (Postfix) with ESMTPS id CE7B85D705; Sun, 23 May 2021 12:31:05 +0000 (UTC) Subject: Re: [edk2-devel] [edk2-stable202105 PATCH] MdeModulePkg/VariableLock: downgrade compatibility warnings to DEBUG_WARN From: "Laszlo Ersek" To: edk2-devel-groups-io Cc: Bret Barkelew , Hao A Wu , Jian J Wang , Liming Gao , Michael D Kinney , =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= Reply-To: devel@edk2.groups.io, lersek@redhat.com References: <20210521204037.11980-1-lersek@redhat.com> Message-ID: <703829d4-2a2b-271e-9ac0-333560ffae75@redhat.com> Date: Sun, 23 May 2021 14:31:04 +0200 MIME-Version: 1.0 In-Reply-To: <20210521204037.11980-1-lersek@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit On 05/21/21 22:40, Laszlo Ersek wrote: > Commit a18a9bde36d2 ("MdeModulePkg/Variable/RuntimeDxe: Restore Variable > Lock Protocol behavior", 2020-12-15), for bug 3111, added two such sets of > debug messages that: > > (a) are relevant for developers, > > (b) yet should not necessarily poke end-users, because no functionality > suffers in practice. > > Both message sets are in function VariableLockRequestToLock(): the first > is a generic interface deprecation warning; the second is the > double-locking situation, which we permit for compatibility (return status > EFI_SUCCESS). > > Both message sets should be emitted with the DEBUG_WARN mask, not the most > serious DEBUG_ERROR mask. On some platforms, the serial console carries > both terminal traffic, and grave (DEBUG_ERROR-only) log messages. On such > platforms, both message sets may be perceived as a nuisance by end-users, > as there is nothing they can do, and there's nothing they *should* do -- > in practice, nothing malfunctions. > > (Such a platform is ArmVirtQemu, built with "-D > DEBUG_PRINT_ERROR_LEVEL=0x80000000".) > > Cc: Bret Barkelew > Cc: Hao A Wu > Cc: Jian J Wang > Cc: Liming Gao > Cc: Michael D Kinney > Cc: Philippe Mathieu-Daudé > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3410 > Fixes: a18a9bde36d2ffc12df29cdced1efa1f8f9f2021 > Signed-off-by: Laszlo Ersek > --- > MdeModulePkg/Universal/Variable/RuntimeDxe/VariableLockRequestToLock.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/MdeModulePkg/Universal/Variable/RuntimeDxe/VariableLockRequestToLock.c b/MdeModulePkg/Universal/Variable/RuntimeDxe/VariableLockRequestToLock.c > index 7d87e50efdcd..4e1efef9a7e4 100644 > --- a/MdeModulePkg/Universal/Variable/RuntimeDxe/VariableLockRequestToLock.c > +++ b/MdeModulePkg/Universal/Variable/RuntimeDxe/VariableLockRequestToLock.c > @@ -48,9 +48,9 @@ VariableLockRequestToLock ( > EFI_STATUS Status; > VARIABLE_POLICY_ENTRY *NewPolicy; > > - DEBUG ((DEBUG_ERROR, "!!! DEPRECATED INTERFACE !!! %a() will go away soon!\n", __FUNCTION__)); > - DEBUG ((DEBUG_ERROR, "!!! DEPRECATED INTERFACE !!! Please move to use Variable Policy!\n")); > - DEBUG ((DEBUG_ERROR, "!!! DEPRECATED INTERFACE !!! Variable: %g %s\n", VendorGuid, VariableName)); > + DEBUG ((DEBUG_WARN, "!!! DEPRECATED INTERFACE !!! %a() will go away soon!\n", __FUNCTION__)); > + DEBUG ((DEBUG_WARN, "!!! DEPRECATED INTERFACE !!! Please move to use Variable Policy!\n")); > + DEBUG ((DEBUG_WARN, "!!! DEPRECATED INTERFACE !!! Variable: %g %s\n", VendorGuid, VariableName)); > > NewPolicy = NULL; > Status = CreateBasicVariablePolicy( > @@ -69,13 +69,13 @@ VariableLockRequestToLock ( > // > // If the error returned is EFI_ALREADY_STARTED, we need to check the > // current database for the variable and see whether it's locked. If it's > - // locked, we're still fine, but also generate a DEBUG_ERROR message so the > + // locked, we're still fine, but also generate a DEBUG_WARN message so the > // duplicate lock can be removed. > // > if (Status == EFI_ALREADY_STARTED) { > Status = ValidateSetVariable (VariableName, VendorGuid, 0, 0, NULL); > if (Status == EFI_WRITE_PROTECTED) { > - DEBUG ((DEBUG_ERROR, " Variable: %g %s is already locked!\n", VendorGuid, VariableName)); > + DEBUG ((DEBUG_WARN, " Variable: %g %s is already locked!\n", VendorGuid, VariableName)); > Status = EFI_SUCCESS; > } else { > DEBUG ((DEBUG_ERROR, " Variable: %g %s can not be locked!\n", VendorGuid, VariableName)); > Merged as commit cfa6ffb113f2, via . Thanks! Laszlo