From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id 9833F3858D28 for ; Mon, 8 May 2023 17:06:33 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 9833F3858D28 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=redhat.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1683565593; 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: in-reply-to:in-reply-to:references:references; bh=nm3mCdJlsggnc0YjalZugRRDKhh8mCjV4QNtcQeDbJ8=; b=etbMKUtjqii69niivqOdl4F0R18r8YeR4LG46gd17NKM78UUkw+Wj8WOvwU97JwzGZJygV +EMrWYk9HDiuSieSMKeqImu+lqVnC2rzmTWaIT8P7CkKixVijoNtWf0L5mFupkYJLreJc7 HoPwf/aQx2fBb48EeZmlJHVTQfKqSMc= Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-342-9_o-xL07NaeaFedZvOXxvQ-1; Mon, 08 May 2023 13:06:32 -0400 X-MC-Unique: 9_o-xL07NaeaFedZvOXxvQ-1 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.rdu2.redhat.com [10.11.54.6]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id CB40C3822DE8; Mon, 8 May 2023 17:06:31 +0000 (UTC) Received: from oldenburg.str.redhat.com (unknown [10.2.16.35]) by smtp.corp.redhat.com (Postfix) with ESMTPS id C268F2166B40; Mon, 8 May 2023 17:06:30 +0000 (UTC) From: Florian Weimer To: "Karumanchi, Sajan" Cc: "libc-alpha@sourceware.org" , "carlos@redhat.com" , "Mallappa, Premachandra" Subject: Re: [PATCH v1] x86: Cache computation for AMD architecture. References: <87edrubqvs.fsf@oldenburg.str.redhat.com> Date: Mon, 08 May 2023 19:06:29 +0200 In-Reply-To: (Sajan Karumanchi's message of "Wed, 18 Jan 2023 07:39:42 +0000") Message-ID: <87a5yepxbe.fsf@oldenburg.str.redhat.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.6 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain X-Spam-Status: No, score=-4.9 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: I'm sorry to report this new cache size computation approach is not compatible with older AMD CPUs. Some existing hypervisors set these CPUID values to zero while still identifying these CPUs as AMD. I filed a new bug to track this regression: New AMD cache size computation logic does not work for some CPUs, hypervisors It looks like we need to keep the old way of doing things. Would you be able to work on a new patch? I'm also going to file a separate bug, suggesting that we deselect string functions that require cache size information if this information is not available (mostly due to hypervisor masking). But that's a different issue. Thanks, Florian