From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] lib/librte_ether: change socket_id passed to rte_memzone_reserve Date: Tue, 27 Jan 2015 10:29:21 +0100 Message-ID: <8331960.qMyqHR1SLh@xps13> References: <1421939108-28666-1-git-send-email-cian.ferriter@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Cian Ferriter Return-path: In-Reply-To: <1421939108-28666-1-git-send-email-cian.ferriter-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" Hi, 2015-01-22 15:05, Cian Ferriter: > Removes the dependency that this memzone reserve has on the > socket currently running on. Following the socket of the master > core will yield more predictable results when calling this > function after initialisation. You don't describe what is the problem. In another mail, you say "The original suggestion also fixes the crash that I was seeing because of memory being reserved from a numa node with no "--socket-mem" allocated." Please describe it clearly in the commit log. You should also explain what this rte_memzone_reserve() is for, and what are the incidences of your changes. Thanks -- Thomas > @@ -184,7 +184,7 @@ rte_eth_dev_data_alloc(void) > if (rte_eal_process_type() == RTE_PROC_PRIMARY){ > mz = rte_memzone_reserve(MZ_RTE_ETH_DEV_DATA, > RTE_MAX_ETHPORTS * sizeof(*rte_eth_dev_data), > - rte_socket_id(), flags); > + rte_lcore_to_socket_id(rte_get_master_lcore()), flags);