From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 5D970A034F for ; Fri, 8 Oct 2021 13:09:41 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 3610B410E1; Fri, 8 Oct 2021 13:09:41 +0200 (CEST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mails.dpdk.org (Postfix) with ESMTP id 953FA410E1 for ; Fri, 8 Oct 2021 13:09:39 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1633691379; 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=uVqEQtaruN3cNTnKyb45Hd+lHOQMdWbpY6qNc0VTXEw=; b=CHiQjr/cPKPyLbT6Cw/W8THfmW0eFwJq87APl5OZXJa/eeGAETEnpX+ECifeewKjFxyChG xqnx+Ma8eSxbUF8rBHbFJuLkgPq7WR/+lNFS8yzbHipteUTD/1Coqi16LiigTIb6XMLAGY ekzFwtrYsxoHw2TUqzm+oVsss0/dEHE= Received: from mail-lf1-f69.google.com (mail-lf1-f69.google.com [209.85.167.69]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-41-kHS65WK3PRutlSBRlGo4sw-1; Fri, 08 Oct 2021 07:09:07 -0400 X-MC-Unique: kHS65WK3PRutlSBRlGo4sw-1 Received: by mail-lf1-f69.google.com with SMTP id c42-20020a05651223aa00b003fd328cfeccso6850996lfv.4 for ; Fri, 08 Oct 2021 04:09:06 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=uVqEQtaruN3cNTnKyb45Hd+lHOQMdWbpY6qNc0VTXEw=; b=gGRrR7XIsd0s38QmwrId/ec4SGHgimYEfso0x8zLIvabv4Y54cAL3HO6Yq6I53KGlH K99me5ENCD4cZU3tUn2WYlqMRc8hAxdGNKagEZ+ExIEMt33IQU2MMiVliVbydTfosF90 3WdbrvGessA8ao+V5RAef3E7gDgO7Ap2bIMJn7tDPxWgjiGzyi1ZVcCibO8h4trqPpH4 4upu+LqxuS6Tj+gIPLoT/omEJND2QCeB5Vu4tgNPHUzaalrP23n7MrqrJmsZno3Htv6m jJR6MQOM/tDhLNVKjnSW5EjGub85/bXMZ47HzSolC1j7qcFzhq3Nud0FDg/6NQjr3fQb eSrg== X-Gm-Message-State: AOAM531WXmc88EypWJrnLyIejtzgacRHvO18/EJKVsNreTPpu76YqlMy vTpQUCEV+b5gvL4PHDFE4rAC+a+a+aEA/h8bXFrA6peWEutqckVks2a5uOF7202HHz4WbSsKhpT A8L5PMV60XPXK5pJvqRo4cBA= X-Received: by 2002:a2e:8584:: with SMTP id b4mr2961676lji.477.1633691345633; Fri, 08 Oct 2021 04:09:05 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxCzwWXfmun9HqUVo7210vNpOo1yqi6nk+9rPDt9NjerKzjFpimpUKEJYXd8zDJw+c75Mq/W/PPQd5cR6RrS4E= X-Received: by 2002:a2e:8584:: with SMTP id b4mr2961657lji.477.1633691345417; Fri, 08 Oct 2021 04:09:05 -0700 (PDT) MIME-Version: 1.0 References: <20211006192008.23369-1-david.marchand@redhat.com> In-Reply-To: From: David Marchand Date: Fri, 8 Oct 2021 13:08:54 +0200 Message-ID: To: Bruce Richardson Cc: dev , dpdk stable , Konstantin Ananyev Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=dmarchan@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-stable] [PATCH] eal/x86: fix some CPU extended features X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On Fri, Oct 8, 2021 at 12:15 PM Bruce Richardson wrote: > > On Wed, Oct 06, 2021 at 09:20:08PM +0200, David Marchand wrote: > > Caught while checking CPUID related stuff in OVS. > > > > According to [1], for Structured Extended Feature Flags Enumeration Leaf > > (EAX = 0x07H, ECX = 0): > > > > - BMI1 is associated to EBX, bit 3 (was incorrectly 2), > > - SMEP is associated to EBX, bit 7 (was incorrectly 6), > > - BMI2 is associated to EBX, bit 8 (was incorrectly 7), > > - ERMS is associated to EBX, bit 9 (was incorrectly 8), > > > > This patch then sorts the rest of the extended features (leaf 0) for > > readability. > > > > 1: https://software.intel.com/sites/default/files/managed/c5/15/architecture-instruction-set-extensions-programming-reference.pdf > > > > Fixes: af75078fece3 ("first public release") > > Wow, I'm surprised at it being incorrect this long, but checking the > original commit of code, it does indeed seem true that this is the correct > fixes line! > > > Cc: stable@dpdk.org > > > > Signed-off-by: David Marchand > > Acked-by: Bruce Richardson > > Ideally, I think the fixes should go in a separate patch from the > reordering, but I don't feel strongly about it so this is ok as either one > or two patches IMHO. I was reading the datasheet and comparing the defines one by one, so easier with sorted defines. Now that I identified the wrong entries, I am fine with splitting since it eases backport on 19.11. I'll send v2 with your ack on both. -- David Marchand