From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aaron Conole Subject: Re: [PATCH] eal: deprecate rte_cpu_check_supported Date: Tue, 04 Apr 2017 09:39:53 -0400 Message-ID: References: <20170331183122.22961-1-aconole@redhat.com> <2833575.ok4SoAkeVM@xps13> Mime-Version: 1.0 Content-Type: text/plain Cc: dev@dpdk.org To: Thomas Monjalon Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id A2D79293B for ; Tue, 4 Apr 2017 15:39:55 +0200 (CEST) In-Reply-To: <2833575.ok4SoAkeVM@xps13> (Thomas Monjalon's message of "Tue, 04 Apr 2017 15:26:47 +0200") List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Thomas, Thomas Monjalon writes: > 2017-03-31 14:31, Aaron Conole: >> It's likely that this function isn't used anywhere, but since it was part of >> the public API, mark the function for deprecation for at least one release. >> >> Signed-off-by: Aaron Conole Thanks so much for the review! I'll make sure to fix these with v2. > [...] >> +DPDK_17.05 { >> + global; >> + >> + rte_cpu_is_supported; >> + >> +} DPDK_17.02; > > Should not it be a separate patch? I thought it made sense to keep it rolled in with this patch, but if you prefer it separate, I will do that. Expect it as 1/2 in v2, unless you say otherwise. -Aaron