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 22D99A0A02; Fri, 26 Mar 2021 11:50:34 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id F0AF4140D4F; Fri, 26 Mar 2021 11:50:33 +0100 (CET) Received: from mail-io1-f53.google.com (mail-io1-f53.google.com [209.85.166.53]) by mails.dpdk.org (Postfix) with ESMTP id BA31B40685 for ; Fri, 26 Mar 2021 11:50:32 +0100 (CET) Received: by mail-io1-f53.google.com with SMTP id z136so4895484iof.10 for ; Fri, 26 Mar 2021 03:50:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=OLKMPOmyODaJol+ESJ+ZOGv4DFHoODNHDvUSpR05Qmg=; b=O3hhQ3U1LE71G805U8WkEy1Ydc8UThj21t/CVkAqbMvq5B3xvQ4uDHGavBQ6yKweKx nYrr4piWtK1pfSc8V30w+SrLIMzfsvLMvOq5dLUN3u87OTYnFdcMmp69/Zof8LI8DUuW 4uWHEgBp1tmiRHL3ZKvxOSSQeVqkeBAJ5omdkTuZkIefQpvSvsxi0SCvkAUZJqPeo5Fx 3E7Yzo1lFg7twwpjQK9GpQBNSbexInX6UntniCcv+VCCG40LWmmLrgn7zQxwHeYHLqXj Uw8Nw5/Fl/MjGUzdz9I/Lu7t4EhVOovOeCrl9cA2533I/JvHvQlWdKxqz+O0LoVYm9Vd Gk9g== 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=OLKMPOmyODaJol+ESJ+ZOGv4DFHoODNHDvUSpR05Qmg=; b=n+yoFsE6tn0J9lCu4oHySE3r7aBQ5MDXWagOmknRwzwsvqsHUVYVVBCrmZmbuWpgqc X9FJGNE/Yd9aBA7is11x8Naa/v2lqz2fkFdwVlgK4CYHkawX9bzTl19pLBleXQvznBKC TyP4QVxLKDq/XwbPIzX17JRHqJsrR65+zfei5zBfRqGl/OjaQsCBgkJliu7eRE8bUEcE OAvAJhN3yFGebs+fhXqyi2jcNxQsoV4sL46VQSLEO1ar28B63TcXeapB7qpxhAqTmZrT VjtUFQ66d2aeFuKSfhFk79w0YRFzjgf547DO+t8rcEfFO0cUe9JgReL92i2pqtKxbYu9 Zj0Q== X-Gm-Message-State: AOAM531IBybjN+OsOGvsx7Cpa9ICbLrCOFUqv6fTwK+IyWL6drsoYN/9 gi7Cw/DpRAgJIEo/5hPanjpJ+zg+wHzpXCDEotE= X-Google-Smtp-Source: ABdhPJz2DZZV7xlXhUTcobNh1uLuzJlolfkaMEO9hHDyFh7pBAyAiYp5xWj/ZtTrKQ+PXjSXs3KwLX/fH7lkHQMz/cI= X-Received: by 2002:a6b:b7cd:: with SMTP id h196mr9438463iof.59.1616755831991; Fri, 26 Mar 2021 03:50:31 -0700 (PDT) MIME-Version: 1.0 References: <20210225170238.15581-1-pbhagavatula@marvell.com> <11975004.s7UMkAiPBC@thomas> <5cb9bec3-6d4e-7867-e8d7-9c3d55ec78ef@ashroe.eu> <5442273.ffSK4IfYTI@thomas> <0564d636-3088-9b3e-1226-2767996d7f2a@ashroe.eu> <0b0986ba-a3e3-916d-25fc-cb37730e8fdf@ashroe.eu> In-Reply-To: <0b0986ba-a3e3-916d-25fc-cb37730e8fdf@ashroe.eu> From: Jerin Jacob Date: Fri, 26 Mar 2021 16:20:16 +0530 Message-ID: To: "Kinsella, Ray" , "Richardson, Bruce" , Neil Horman Cc: Thomas Monjalon , Pavan Nikhilesh , Jerin Jacob , dpdk-dev , Harman Kalra , Nithin Dabilpuram , David Marchand Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH 1/7] common/octeontx: enable build only on 64bit Linux X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thu, Mar 25, 2021 at 6:32 PM Kinsella, Ray wrote: > > > > On 25/03/2021 12:58, Jerin Jacob wrote: > > On Thu, Mar 25, 2021 at 6:17 PM Kinsella, Ray wrote: > >> > >> > >> > >> On 25/03/2021 12:46, Jerin Jacob wrote: > >>> On Thu, Mar 25, 2021 at 4:33 PM Thomas Monjalon wrote: > >>>> > >>>> 25/03/2021 11:58, Kinsella, Ray: > >>>>> On 25/03/2021 10:46, Thomas Monjalon wrote: > >>>>>> 25/03/2021 11:42, Thomas Monjalon: > >>>>>>> 24/03/2021 11:55, Jerin Jacob: > >>>>>>>> On Thu, Feb 25, 2021 at 10:33 PM wrote: > >>>>>>>>> > >>>>>>>>> From: Pavan Nikhilesh > >>>>>>>>> > >>>>>>>>> Due to Linux kernel dependency, only enable build for 64bit Linux. > >>>>>>>>> > >>>>>>>>> Signed-off-by: Pavan Nikhilesh > >>>>>>>> > >>>>>>>> Series Acked-by: Jerin Jacob > >>>>>>> > >>>>>>> I've reorganized the commits per family of drivers, > >>>>>>> so it makes more sense than grouping per driver class > >>>>>>> with "common/octeontx" for title for all: > >>>>>>> > >>>>>>> net/thunderx: enable build only on 64-bit Linux > >>>>>>> common/octeontx: enable build only on 64-bit Linux > >>>>>>> common/octeontx2: enable build only on 64-bit Linux > >>>>>>> > >>>>>>> and applied. > >>>>>> > >>>>>> Actually not applied yet. > >>>>>> I'm not sure what to do for the ABI check which is broken > >>>>>> because some drivers are not compiled anymore in 32-bit build. > >>>>>> I've workarounded locally by removing the dump files in the reference build. > >>>>>> Should we add an exception in libabigail.abignore? > >>>>>> > >>>>> In the past we said that depreciating HW support would be considered to be same as an ABI Breakage. > >>>>> > >>>>> From the policy ... > >>>>> "Updates to the minimum hardware requirements, which drop support for hardware which was previously supported, should be treated as an ABI change." > >>>> > >>>> So the patches should wait 21.11. > >>>> Everybody agree? > >>> > >>> Looks good to me to postpone. > >>> > >>> @Ray Kinsella @Thomas Monjalon @McDaniel, Timothy @David Marchand @Neil Horman > >>> > >>> Currently, I merged DLB v1 driver removal patch to next-eventdev. Is > >>> this ABI breakge[1]? > >>> > >>> http://patches.dpdk.org/project/dpdk/patch/20210316210812.15614-1-timothy.mcdaniel@intel.com/ > >>> > >>> [1] > >>> From the policy ... > >>> "Updates to the minimum hardware requirements, which drop support for > >>> hardware which was previously supported, should be treated as an ABI > >>> change." > >> > >> +1 > > > > Is +1 for not to remove the dlb driver or remove it? > > > > You'll note the original reply pointing at the ABI Policy was from myself. > So I would be in favor of retention until 21.11. > > Thats said... > > We should think about making the rules less strict for 32bit in future, from 21.11 onwards perhaps. > How many OS Vendors are shipping 32bit OSs these days for example? Agree on 32bit. The DLB driver removal patch from @McDaniel, Timothy http://patches.dpdk.org/project/dpdk/patch/20210316210812.15614-1-timothy.mcdaniel@intel.com/ is deleting the complete driver. So it does not look like it is updating the minimum HW requirements as mentioned in the policy. So IMO, it is OK to accept his patch for 21.05(ie. remove the driver). Let me know if there is any objection on this?