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 4DECA45F15; Sun, 22 Dec 2024 17:43:24 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 76AAA40615; Sun, 22 Dec 2024 17:43:23 +0100 (CET) Received: from mail-pl1-f170.google.com (mail-pl1-f170.google.com [209.85.214.170]) by mails.dpdk.org (Postfix) with ESMTP id 84F0140263 for ; Sun, 22 Dec 2024 17:43:22 +0100 (CET) Received: by mail-pl1-f170.google.com with SMTP id d9443c01a7336-21636268e43so39326655ad.2 for ; Sun, 22 Dec 2024 08:43:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1734885801; x=1735490601; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=uuIqC76H4W3svXbYjWGWwV1ZUr+ykI/nbFM5nc95Edk=; b=ZcK1zXt151GUgTpNZUytr/4QPC+D/afDuzhNmIBAEGDHrMMlSUMoBzAHkZEEEYoCSc 06p0dERDFV7Vr1DE8Uk/DWRw6OHS24DaenWwfVpsVtye4qUFA97WqyisHThF9fzQXh5R hoLSqiKrf0CXAJBbtrwLVJrto5OvSj66v9tUhLMRZML4l0MSFSRdPvN3jf2MVgfoLDCN RyPJAjGTVe/FBPDZx/Feq26KOARWO7+GDg5ddn21f4MWqbbidXHTJLJbTgi7pkbA+Qi2 dECbhr0Pc1RqgZjJv1VYMcTtijHjgVrqjd5SuSfPGAjFMOpnp3YV3VHXyXdMXdVPJk5B JHUg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1734885801; x=1735490601; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=uuIqC76H4W3svXbYjWGWwV1ZUr+ykI/nbFM5nc95Edk=; b=cMfjqOdnGfivIar6PivJUcmmnGuBj1uHui8WGesYDz6whxgyhjCsRiYl2M0TB5ZhNU aoJemk+K6s8tKFvcDSjv+Pyj/ra65twcOrjaNOOnQ+FLpj0h7O5j3sjv/HmAg+BufEnu bKw4zbBmRnr7wBNuWfTs8yqLsyVKAmQfsR55Y8cRco8x3nr+3+L+uDpClUgn/h00ON/9 ZnuPdOXnHpNTAo1cb/ECnMq7rcptYVSG7O71uiZL+nzB+gAPLR74ROAOJeZRo7/SYzsy Uun8jS9mEq7PYd1Sv16piWv/gfMW4hZ1ZHldO5VIVOpr2Bmcog3Hvohk1Wh7qGLcOAn2 +wQg== X-Gm-Message-State: AOJu0YwEAEg5b90gfuojv0aGMu5RPtgTMwQYmKwalFd5CLnuVyUDCzF0 bhw9+DtdqECmSkXHI/BA4eQnr8itj5aUSrnl9seL454tA9WdMybJNuPqBCDu8So= X-Gm-Gg: ASbGnctTvEBMypOEY+/SRuKqSx+ZncOTV6cPtQmRfMuaa52oCsaDDQF3shZIIJCB8+F qaDPgnkQVwOSzWoQyci+oWiy+hw4CtzJL8nyuX/UcXyNGePB+WQx7Nt3NBsy9wDROy3ufvliqIg RO9s/lZi1NWkJvoChC0wuANtyb67tzRh8YvS0P47hoSGeVWbdbJyTatLbDmZbQ6J5/xoSwPXE6g bDWjRrhePRV1Vks/2eLZPmA7krkssxr2BHXRyPH7LIerJxZEhhhLAQwyaasUx20xz/zeFfcA7i9 3S7hwkAoG15G5dPbbY5R7Fhs38bspZ0oLA== X-Google-Smtp-Source: AGHT+IGNjTSEC5LYyRB8zv8adj+4vQzVIaEfNdRMCWs3Tfs8ifxKX3hyfCldO54Uc6zofLURJKcWng== X-Received: by 2002:a17:903:2291:b0:216:4a8a:2665 with SMTP id d9443c01a7336-219e6f267femr114009185ad.50.1734885801530; Sun, 22 Dec 2024 08:43:21 -0800 (PST) Received: from hermes.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-219dc964e92sm58749415ad.18.2024.12.22.08.43.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 22 Dec 2024 08:43:21 -0800 (PST) Date: Sun, 22 Dec 2024 08:43:16 -0800 From: Stephen Hemminger To: Ariel Otilibili Cc: dev@dpdk.org, stable@dpdk.org, Andrew Boyer Subject: Re: [PATCH v3 01/10] devtools/cocci, lib/eal: remove unused rte_bitmap_free() Message-ID: <20241222084316.1b2b7eb9@hermes.local> In-Reply-To: <20241222013328.1362225-2-otilibil@eurecom.fr> References: <20241222013328.1362225-1-otilibil@eurecom.fr> <20241222013328.1362225-2-otilibil@eurecom.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit 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 On Sun, 22 Dec 2024 02:19:53 +0100 Ariel Otilibili wrote: > Coverity reports return values of calls to rte_bitmap_free() are not used. Some existing user code may actually be using this so this would be an API change. Therefore this patch can not be accepted for current releases. It is worth addressing this in future but: - need to go through deprecation process - function to be void - it should be possible to implement a real free function just was never done.