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 E90ECA0A03 for ; Mon, 18 Jan 2021 09:18:23 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C2006140D13; Mon, 18 Jan 2021 09:18:23 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mails.dpdk.org (Postfix) with ESMTP id D7EF1140D12 for ; Mon, 18 Jan 2021 09:18:22 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1610957902; 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=1amxqV2icCT88gsyBYEQlL/in9LJ/rL4LvZfDYJgcPg=; b=YPbVvL3LHnfjjdFzEoiN8wFbnwx600BLmccmISnT7iJwps5ijE5Q9TqZGoHTBlY9HPhsdw nop8QDyY+UWuDYoLm/wmy4lyxllSESIGFL8/Yb69/llhh/PtKGNNngWIfwboljbu/bf/ux bONlJPfLsfGH7uybgThewC7HsMjgEfk= Received: from mail-vk1-f200.google.com (mail-vk1-f200.google.com [209.85.221.200]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-593-jZAxRUrqOLiBrzVH_3xpzA-1; Mon, 18 Jan 2021 03:18:19 -0500 X-MC-Unique: jZAxRUrqOLiBrzVH_3xpzA-1 Received: by mail-vk1-f200.google.com with SMTP id 84so9061436vkx.1 for ; Mon, 18 Jan 2021 00:18:19 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=1amxqV2icCT88gsyBYEQlL/in9LJ/rL4LvZfDYJgcPg=; b=fHlUR5PmegjnlsfgOyREj4T/RH7jXFHdEsy370Q8lJgG2w5Jj/qhrhoJIfus/87F5U 6R1UnzsCbgkFQOTIkm77yNun1exLsEi6MTtst5haju4TB8eXw0Gdg81wYlBll6ck2s0I w3ohfvvQ1LbJ6Q+8Hq6pF2EVcVIC32yYLRyvQUIuNJKU7G5qLpUvFweL74czlrDj/jIN K4V7M6lgB4DF+YCw95A7AEIWLu0Bn+YwVx/4nCg5z6tlPDz8aAAB9+NqCdsZB7mwcMiQ yuauunGzsH0YNE8VEHUVJmHiMuiHP5SFs86p6CjtQ1pxkP3MTmwGQiNJcn+3/gxcw4nr OpNQ== X-Gm-Message-State: AOAM5321QZWg7nSnBc+2vYR0ElwPYPtk2XTLSOJVVzm8T2juBlT5Aloe EQyfLK6vQSn31VKckZz+/tPyeQcIuuIhIJKglRihDvJ7aCnEYEN9PpG2Qa/JRCjOSFUQ6F7biEu O2FXgXB2p9f/+qpwoFrWLUVo= X-Received: by 2002:a1f:9e83:: with SMTP id h125mr17777084vke.18.1610957898800; Mon, 18 Jan 2021 00:18:18 -0800 (PST) X-Google-Smtp-Source: ABdhPJwW6nHzLwuNm7mcHSRlUpByMBV6Qoj2JuRRO8t75TdmwP0j3SHjxn1v3wC6O1hkAVE+QGKSTjCP5+QGfn8DATI= X-Received: by 2002:a1f:9e83:: with SMTP id h125mr17777079vke.18.1610957898602; Mon, 18 Jan 2021 00:18:18 -0800 (PST) MIME-Version: 1.0 References: <20210115134021.7391-1-david.marchand@redhat.com> <20210115134021.7391-2-david.marchand@redhat.com> In-Reply-To: From: David Marchand Date: Mon, 18 Jan 2021 09:18:07 +0100 Message-ID: To: Ruifeng Wang Cc: "dev@dpdk.org" , "ferruh.yigit@intel.com" , "stable@dpdk.org" , Ziyang Xuan , Xiaoyun Wang , Guoyang Zhou , Ciara Power , "thomas@monjalon.net" , nd 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 1/3] net/hinic: restore vectorised code 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 Sat, Jan 16, 2021 at 3:39 PM Ruifeng Wang wrote: > > Subject: [PATCH 1/3] net/hinic: restore vectorised code > > > > Following make support removal, the vectorised code is not built anymore, > > fix the build flag check. > > > > Fixes: 3cc6ecfdfe85 ("build: remove makefiles") > > Cc: stable@dpdk.org > > > > Signed-off-by: David Marchand > > --- > > drivers/net/hinic/hinic_pmd_rx.c | 6 +++--- > > drivers/net/hinic/hinic_pmd_tx.c | 10 +++++----- > > 2 files changed, 8 insertions(+), 8 deletions(-) > > > > diff --git a/drivers/net/hinic/hinic_pmd_rx.c > > b/drivers/net/hinic/hinic_pmd_rx.c > > index a49769a863..842399cc4c 100644 > > --- a/drivers/net/hinic/hinic_pmd_rx.c > > +++ b/drivers/net/hinic/hinic_pmd_rx.c > > @@ -4,7 +4,7 @@ > > > > #include > > #include > > -#ifdef __ARM64_NEON__ > > +#ifdef RTE_ARCH_ARM64 > > We can test '__ARM_NEON' which will be defined by compilers. > https://developer.arm.com/documentation/ihi0053/latest/ On the principle, I agree, but this was not what was tested before. The activation was only gated by checking for the arm architecture. See: https://git.dpdk.org/dpdk/tree/drivers/net/hinic/Makefile?id=05b6eee7bdabf7c17ed69c44515e0cd7d6e1da23#n14 So I restored the test as it was before make removal. What you propose can be done in a followup from the hinic developers, but the minimal fix is still this current patch. -- David Marchand