From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-vk0-f44.google.com (mail-vk0-f44.google.com [209.85.213.44]) by dpdk.org (Postfix) with ESMTP id EDA6311D9 for ; Thu, 3 Mar 2016 04:39:41 +0100 (CET) Received: by mail-vk0-f44.google.com with SMTP id c3so9231439vkb.3 for ; Wed, 02 Mar 2016 19:39:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc; bh=7VWHf3AZJPUp5itcGitEBUemPfDQSCBvk6OBJ61COBY=; b=GFG36ctboOC9SO2BiRin+6G4Cv6nBREofXN+Oe3bB64qOBD2fLi2ELlCEdp1ltCKdg /kdcTxPzrflR1YP6FJhyoW/fYDfLvdHSLTpy445KCxj1EAt6nNJleYkhOnV5bGWmvLrO LlQkH/G8y/jAH85veN8etH+QaRjW9jK4bfWpVApaStCcOZtqa88M5eObLQwrPFkoshKX Dt5R2I0ij3NHaodtu4+iw5wE1qVODE2Dc3d2zXZI4H6dVoSNQLE7huP1FDV/QtRAzx+P tpunnLTS1x817K9i4iXhJDEE4X+TqF99iiKsHc0fNAuu5SmVzdYDyiqQDe/Wntwakr6O st0A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc; bh=7VWHf3AZJPUp5itcGitEBUemPfDQSCBvk6OBJ61COBY=; b=Bcvcb3gi4OKxM9ryV9LC1e5UvLCAQkDMP1SkGC+pLkD66KjQFhaPbprkd826foIeRj nLTmtX/MI60+wGxiBwvsIcX+qaHosTdDL35LSeqEIVYKjiGwnyVFFQXEKlJP5BrVmo4y jJBzePmI26tnD4yIn9cfQ0gdwxKwy4x5WhFEkPu5Z9k9OK5Qhgnce53mYLqx25Uh/+V2 pozegfAzJn7muEp27kAkAw4L2c55i5IrY3Bqu7KI8cVqOnclqxcdaOTpH3Sns6YjEhWs VCMKbHEpB34NLwMk4fSx4tB6rMN2+bkwAVTyNTONKtHyTjCI6HeokHERsKDkf5QczX12 yLyw== X-Gm-Message-State: AD7BkJKWmfwJBMPVXNt3ofErvFeGak7V+7sCPav40eyro4NDiWfVjj02v5hKjKOSbLcUVVaKwhVv6GX/Vrh+8Q== MIME-Version: 1.0 X-Received: by 10.31.133.201 with SMTP id h192mr264851vkd.102.1456976381448; Wed, 02 Mar 2016 19:39:41 -0800 (PST) Received: by 10.31.237.129 with HTTP; Wed, 2 Mar 2016 19:39:41 -0800 (PST) In-Reply-To: References: <533710CFB86FA344BFBF2D6802E6028622F5ACB9@SHSMSX101.ccr.corp.intel.com> Date: Wed, 2 Mar 2016 22:39:41 -0500 Message-ID: From: Dhananjaya Reddy Eadala To: "De Lara Guarch, Pablo" Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH] hash: fix memcmp function pointer in multi-process X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 03 Mar 2016 03:39:42 -0000 Hi Pablo Thanks for the clarification. I just sent the patch using "git send-email". Probably that will show up as different mail thread. Thanks Dhana On Wed, Mar 2, 2016 at 9:22 AM, De Lara Guarch, Pablo < pablo.de.lara.guarch@intel.com> wrote: > Hi Dhana, > > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Dhananjaya Reddy > > Eadala > > Sent: Wednesday, March 02, 2016 2:02 PM > > To: Qiu, Michael > > Cc: dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH] hash: fix memcmp function pointer in > multi- > > process > > > > Michael > > > > Please find the attached is the patch file generated from "git > format-patch > > -1" > > What Michael means is that you should use git send-email and not attach > the patch > to the mail. > > You can use: > > git send-email -1 --to dev@dpdk.org > > or > > git send-email your-patch.patch --to dev@dpdk.org > > Take a look at dpdk.org/dev for more details and how to configure git > with your smtp server details. > > Thanks, > Pablo > > > > > > > Thanks > > Dhana > > > > > > On Tue, Mar 1, 2016 at 9:29 PM, Qiu, Michael > > wrote: > > > > > On 3/2/2016 2:57 AM, Dhananjaya Reddy Eadala wrote: > > > > Hi > > > > > > > > We found a problem in dpdk-2.2 using under multi-process environment. > > > > Here is the brief description how we are using the dpdk: > > > > > > > > We have two processes proc1, proc2 using dpdk. These proc1 and proc2 > > are > > > > two different compiled binaries. > > > > proc1 is started as primary process and proc2 as secondary process. > > > > > > > > proc1: > > > > Calls srcHash = rte_hash_create("src_hash_name") to create rte_hash > > > > structure. > > > > As part of this, this api initalized the rte_hash structure and set > the > > > > srcHash->rte_hash_cmp_eq to the address of memcmp() from proc1 > > address > > > > space. > > > > > > > > proc2: > > > > calls srcHash = rte_hash_find_existing("src_hash_name"). This > returns > > > the > > > > rte_hash created by proc1. > > > > This srcHash->rte_hash_cmp_eq still points to the address of memcmp() > > > from > > > > proc1 address space. > > > > Later proc2 calls rte_hash_lookup_with_hash(srcHash, (const void*) > > &key, > > > > key.sig); > > > > Under the hood, rte_hash_lookup_with_hash() invokes > > > > __rte_hash_lookup_with_hash(), which in turn calls > > > h->rte_hash_cmp_eq(key, > > > > k->key, h->key_len). > > > > This leads to a crash as h->rte_hash_cmp_eq is an address from proc1 > > > > address space and is invalid address in proc2 address space. > > > > > > > > We found, from dpdk documentation, that > > > > " > > > > The use of function pointers between multiple processes running > based > > of > > > > different compiled > > > > binaries is not supported, since the location of a given function > in one > > > > process may be different to > > > > its location in a second. This prevents the librte_hash library from > > > > behaving properly as in a multi- > > > > threaded instance, since it uses a pointer to the hash function > > > internally. > > > > > > > > > > > > To work around this issue, it is recommended that multi-process > > > > applications perform the hash > > > > calculations by directly calling the hashing function from the code > and > > > > then using the > > > > rte_hash_add_with_hash()/rte_hash_lookup_with_hash() functions > > instead > > > of > > > > the functions which do > > > > the hashing internally, such as rte_hash_add()/rte_hash_lookup(). > > > > " > > > > > > > > We did follow the recommended steps by invoking > > > rte_hash_lookup_with_hash(). > > > > It was no issue up to and including dpdk-2.0. In later releases > started > > > > crashing because rte_hash_cmp_eq is introduced in dpdk-2.1 > > > > > > > > We fixed it with the following patch and would like to submit the > patch > > > to > > > > dpdk.org. > > > > > > Could you send the patch in the mail? > > > > > > Learn how to send a patch: > > > > > > http://www.dpdk.org/dev > > > > > > Thanks, > > > Michael > > > > Patch is created such that, if anyone wanted to use dpdk in > multi-process > > > > environment with function pointers not shared, they need to > > > > define RTE_LIB_MP_NO_FUNC_PTR in their Makefile. Without defining > > this > > > flag > > > > in Makefile, it works as it is now. > > > > > > > > > > > > Please find here attached is the patch file. > > > > > > > > Thanks > > > > Dhana > > > > > > > > > > >