From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Tan, Jianfeng" Subject: Re: [PATCH 2/4] bus/vdev: bus scan by multi-process channel Date: Mon, 12 Mar 2018 11:22:09 +0800 Message-ID: References: <1520177405-59091-1-git-send-email-jianfeng.tan@intel.com> <1520177405-59091-3-git-send-email-jianfeng.tan@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: bruce.richardson@intel.com, konstantin.ananyev@intel.com, thomas@monjalon.net, maxime.coquelin@redhat.com, ferruh.yigit@intel.com To: "Burakov, Anatoly" , dev@dpdk.org Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id CC3CE19F5 for ; Mon, 12 Mar 2018 04:22:14 +0100 (CET) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 3/7/2018 10:00 PM, Burakov, Anatoly wrote: > On 04-Mar-18 3:30 PM, Jianfeng Tan wrote: >> To scan the vdevs in primary, we send request to primary process >> to obtain the names for vdevs. >> >> Only the name is shared from the primary. In probe(), the device >> driver is supposed to locate (or request more) the detail >> information from the primary. >> >> Signed-off-by: Jianfeng Tan >> --- > > General note - you probably want to syncrhonize access to the tailq. > Multiple secondaries may initialize, a vdev hotplug event may be in > process, etc. > Make sense, will change it in next version. Thanks, Jianfeng