From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS3215 2.6.0.0/16 X-Spam-Status: No, score=-4.1 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from sourceware.org (server2.sourceware.org [IPv6:2620:52:3:1:0:246e:9693:128c]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by dcvr.yhbt.net (Postfix) with ESMTPS id E16291F51E for ; Wed, 28 Sep 2022 19:24:42 +0000 (UTC) Authentication-Results: dcvr.yhbt.net; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.b="WKHV0rBV"; dkim-atps=neutral Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id D9EAA3857BA6 for ; Wed, 28 Sep 2022 19:24:41 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org D9EAA3857BA6 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1664393081; bh=hADIooByKUmfOcv2autM7imI6XCWkdARvCtvt6hX1Y8=; h=References:In-Reply-To:Date:Subject:To:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=WKHV0rBVzbPd8yNl1+L3pvbFVbnq6JwZnOV1LFMkwka7BNzZFl5cQ4iuTcVao8ZhU 0eV7GDcACxpgb31UkQel290moo8gl4vWRYrklMbfibosV/yb4RHovPQfJxKo/CNtQV X3E7sgy3msljdVd8kdFO+7tXOL0p4oHo7ljAUGuE= Received: from mail-qv1-xf2d.google.com (mail-qv1-xf2d.google.com [IPv6:2607:f8b0:4864:20::f2d]) by sourceware.org (Postfix) with ESMTPS id 537523858016; Wed, 28 Sep 2022 19:24:23 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 537523858016 Received: by mail-qv1-xf2d.google.com with SMTP id s13so8737418qvq.10; Wed, 28 Sep 2022 12:24:23 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date; bh=hADIooByKUmfOcv2autM7imI6XCWkdARvCtvt6hX1Y8=; b=O0CSt/UgFSd2NifyCyaVUVyh5CUhWz3z50hjxyKbpcEin6IDB1qpiLD66yHvlYU5/O iXTeKusLL0G1VRcfPcTEMEDoXnZaixjGO5LKXQBPlmTVb/3WkN7iCc5ij5Ie/UCziw8h 3yliA3eHhjIIAcec3ibc//fIB+uBwZXo4hnWXwuR4FfAiJhdAAMlPkWdLevUx3IEI8cU bUUiy7foZ3DjC1B/tpDfAvp0NbvO5KzGclZHPej2jZbgxe1eSbdLyvpFJ56eoR9vf9kq RY++/fJ+ucZdj02mqC8bJJWVDavpZq9G7IGr+xDZiWVJlEMk6e5v73cFNIFwthhUQ9GT P3KQ== X-Gm-Message-State: ACrzQf3A7yADHBJG145t16yT7DYoFh0gqfd0xn+Ysup9v12C6lf4aoX9 Lnahj4s+B4h7UR29s5JUixjCzNskkyMJpBW9qKD8wr3Tl44= X-Google-Smtp-Source: AMsMyM7VjROJxajailq4qeW0KudHDX+PPPcmN1h8LmdYbcRrg0sEt+gYos+/wWa4OYuUhOZebMQehyJrTXmR8u6iNU4= X-Received: by 2002:ad4:5944:0:b0:4ad:7802:c35a with SMTP id eo4-20020ad45944000000b004ad7802c35amr27030683qvb.84.1664393062621; Wed, 28 Sep 2022 12:24:22 -0700 (PDT) MIME-Version: 1.0 References: <20210419233607.916848-1-goldstein.w.n@gmail.com> <20210419233607.916848-2-goldstein.w.n@gmail.com> In-Reply-To: Date: Wed, 28 Sep 2022 12:23:46 -0700 Message-ID: Subject: Re: [PATCH v5 2/2] x86: Optimize strlen-avx2.S To: Sunil Pandey Content-Type: text/plain; charset="UTF-8" X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: "H.J. Lu via Libc-alpha" Reply-To: "H.J. Lu" Cc: GNU C Library , Libc-stable Mailing List Errors-To: libc-alpha-bounces+e=80x24.org@sourceware.org Sender: "Libc-alpha" On Wed, Sep 28, 2022 at 12:09 PM Sunil Pandey wrote: > > On Wed, Sep 28, 2022 at 11:24 AM H.J. Lu wrote: > > > > On Wed, Sep 28, 2022 at 6:55 AM Sunil Pandey wrote: > > > > > > Attached patch fixes BZ# 29611. > > > > > > I would like to backport it to 2.32,2.31,2.30,2.29 and 2.29. Let me know > > > if there is any objection. > > > > It doesn't fully fix BZ #29611. Like Noah mentioned, we need to add > > BMI2 check to ifunc-impl-list.c for all functions which use "ifunc-avx2.h". > > > > > > H.J. > > Pulling up corresponding patches are extremely difficult as they are not > modular. I can modify existing patches (as posted on bug report) to > incorporate ifunc-impl-list.c functionality. If it is OK? Please mention BZ #29611 in the commit log of the backport and submit a separate patch to fully fix BZ #29611. We should use a patch set for BZ #29611. > For backporting small incremental changes are preferred. Single monolithic > patch makes backporting extremely difficult, if not impossible. -- H.J.