From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 0E25DA05D3 for ; Thu, 23 May 2019 16:30:00 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id AD06A1B951; Thu, 23 May 2019 16:29:59 +0200 (CEST) Received: from mail-ua1-f67.google.com (mail-ua1-f67.google.com [209.85.222.67]) by dpdk.org (Postfix) with ESMTP id B54E01B947 for ; Thu, 23 May 2019 16:29:56 +0200 (CEST) Received: by mail-ua1-f67.google.com with SMTP id 49so2259838uas.0 for ; Thu, 23 May 2019 07:29:56 -0700 (PDT) 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=+8eJuf8FA7sQtSo2zFBkQcVg2Z55t/yEgAgDEaSMV7Q=; b=AXhXwgZTMeN9bJntKSVQ36gs9sfVShWCwHCjc/GFqdwt7RrLJBDMn2xCeo6lep2uBa FTdPvLlWTNdEFMdIWTWTCjcQIq3XYpmJvF+ngiEyqVj8OpmhbzPvLwAGHSL9blaXKx2S kjX/SlRwDNshB5z74xAznX5k7I35s4c+prm1yuvZfli3xk1MKRrmyKEZPXouVMgr5fis 01UVWL/yvDDXZWRDLSN7Ob20KwF5t7hYUfTX6x7WS9wZCtganQGe806EL4cAfUXYOCoU lCzzP3tNtPZ+NLOMnVah9ghL5EteAO4YxepOffBs8kJKolxLbC9jbUuaha715X8IVi3o qzpQ== X-Gm-Message-State: APjAAAUY5Wg7WIPZAXoCKiNO8c6kkGD/qZFyCJoiMnbiCmCh+25Ehq8U NHY85qTkZn1O4pu7r3XuH+eJpkE+c7Cz0K4yoyollA== X-Google-Smtp-Source: APXvYqzvottPS04+a+rtrhGASV+V1ueMb6+RyHOX5xROEGCpCaDEcQQhkbFvaLEQ2ejbPNJQvuD1yAiHvAluI1oILh8= X-Received: by 2002:ab0:5930:: with SMTP id n45mr35000896uad.87.1558621796024; Thu, 23 May 2019 07:29:56 -0700 (PDT) MIME-Version: 1.0 References: <20190523095231.1091-1-nicolas.dichtel@6wind.com> In-Reply-To: <20190523095231.1091-1-nicolas.dichtel@6wind.com> From: David Marchand Date: Thu, 23 May 2019 16:29:44 +0200 Message-ID: To: Nicolas Dichtel Cc: dev , Anatoly Burakov , dpdk stable , Olivier Matz , Didier Pallard Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-stable] [PATCH v2] mem: ease init in a docker container X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 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 Thu, May 23, 2019 at 11:53 AM Nicolas Dichtel wrote: > move_pages() is only used to get the numa node id, but this function > is not allowed by default in docker (it needs CAP_SYS_NICE and an update of > the seccomp profile). > get_mempolicy() also requires CAP_SYS_NICE but doesn't need any change in > the default seccomp profile. > > Note that the returned value of move_pages() was not checked, thus some > errors could be hidden (if the requested id was 0). > > Cc: stable@dpdk.org > Fixes: 582bed1e1d1d ("mem: support mapping hugepages at runtime") > Signed-off-by: Nicolas Dichtel > Reviewed-by: Olivier Matz > Reviewed-by: Didier Pallard > Acked-by: Anatoly Burakov > --- > > v2: add fixes tag + cc stable > fix title prefix > > lib/librte_eal/linux/eal/eal_memalloc.c | 10 +++++++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/lib/librte_eal/linux/eal/eal_memalloc.c > b/lib/librte_eal/linux/eal/eal_memalloc.c > index 1e9ebb86dd1b..438faa0ab168 100644 > --- a/lib/librte_eal/linux/eal/eal_memalloc.c > +++ b/lib/librte_eal/linux/eal/eal_memalloc.c > @@ -600,9 +600,13 @@ alloc_seg(struct rte_memseg *ms, void *addr, int > socket_id, > } > > #ifdef RTE_EAL_NUMA_AWARE_HUGEPAGES > - move_pages(getpid(), 1, &addr, NULL, &cur_socket_id, 0); > - > - if (cur_socket_id != socket_id) { > + ret = get_mempolicy(&cur_socket_id, NULL, 0, addr, > + MPOL_F_NODE | MPOL_F_ADDR); > + if (ret < 0) { > + RTE_LOG(DEBUG, EAL, "%s(): get_mempolicy: %s\n", > + __func__, strerror(errno)); > + goto mapped; > + } else if (cur_socket_id != socket_id) { > RTE_LOG(DEBUG, EAL, > "%s(): allocation happened on wrong socket > (wanted %d, got %d)\n", > __func__, socket_id, cur_socket_id); > -- > 2.21.0 > > Reviewed-by: David Marchand -- David Marchand