From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2BD60A00E6 for ; Tue, 9 Jul 2019 12:24:17 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1D7061B9AA; Tue, 9 Jul 2019 12:24:17 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id D1C8B1B959; Tue, 9 Jul 2019 12:24:12 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 09 Jul 2019 03:24:12 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.63,470,1557212400"; d="scan'208";a="165725406" Received: from aburakov-mobl1.ger.corp.intel.com (HELO [10.237.220.82]) ([10.237.220.82]) by fmsmga008.fm.intel.com with ESMTP; 09 Jul 2019 03:24:11 -0700 To: Yasufumi Ogawa Cc: dev@dpdk.org, stable@dpdk.org References: <1555379952-23517-1-git-send-email-ogawa.yasufumi@lab.ntt.co.jp> <1555386203-23776-1-git-send-email-ogawa.yasufumi@lab.ntt.co.jp> <1555386203-23776-2-git-send-email-ogawa.yasufumi@lab.ntt.co.jp> <95821e91-33f1-686c-f4c1-8ce7a07646d4@gmail.com> From: "Burakov, Anatoly" Message-ID: Date: Tue, 9 Jul 2019 11:24:10 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.7.2 MIME-Version: 1.0 In-Reply-To: <95821e91-33f1-686c-f4c1-8ce7a07646d4@gmail.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH v2 1/1] fbarray: get fbarrays from containerized secondary 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 09-Jul-19 11:22 AM, Yasufumi Ogawa wrote: > Hi Anatoly, > > On 2019/07/05 17:53, Burakov, Anatoly wrote: >> On 16-Apr-19 4:43 AM, ogawa.yasufumi@lab.ntt.co.jp wrote: >>> From: Yasufumi Ogawa >>> >>> In secondary_msl_create_walk(), it creates a file for fbarrays with its >>> PID for reserving unique name among secondary processes. However, it >>> does not work if secondary is run as app container because each of >>> containerized secondary has PID 1. To reserve unique name, use hostname >>> instead of PID if the value is 1. >>> >>> Cc: stable@dpdk.org >>> >>> Signed-off-by: Yasufumi Ogawa >>> --- >> >> I'm not too well versed in containers - is this hostname 1) always >> set, and 2) always unique? > For docker, 1) hostname is always set. 2) The hostname is decided as > short form of container ID, so it might not be unique even though very > low possibility. > > I found that we can get whole container ID in `/proc/self/cgroup` as > discussed [1]. I think using hostname is reasonable way without running > many secondary processes. However, it might be better to use 64 digits > full container ID instead of 12 digits short ID if ensure uniqueness > strongly. What do yo think? > > [1] > https://forums.docker.com/t/get-a-containers-full-id-from-inside-of-itself/37237 > I think it's better to err on the side of caution and guarantee better uniqueness. This code will get into an LTS and will be used for years to come :) -- Thanks, Anatoly