From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 556642906 for ; Tue, 26 Apr 2016 06:14:42 +0200 (CEST) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga103.fm.intel.com with ESMTP; 25 Apr 2016 21:14:24 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,535,1455004800"; d="scan'208";a="940154710" Received: from yliu-dev.sh.intel.com (HELO yliu-dev) ([10.239.67.162]) by orsmga001.jf.intel.com with ESMTP; 25 Apr 2016 21:14:23 -0700 Date: Mon, 25 Apr 2016 21:16:37 -0700 From: Yuanhan Liu To: Christian Ehrhardt Cc: Aaron Conole , dev@dpdk.org, "Xie, Huawei" , Thomas Monjalon , David Marchand , Panu Matilainen Message-ID: <20160426041637.GE7832@yliu-dev.sh.intel.com> References: <1461575896-17409-1-git-send-email-christian.ehrhardt@canonical.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1461575896-17409-1-git-send-email-christian.ehrhardt@canonical.com> User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [RFC] eal: provide option to set vhost_user socket owner/permissions 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: Tue, 26 Apr 2016 04:14:42 -0000 On Mon, Apr 25, 2016 at 11:18:16AM +0200, Christian Ehrhardt wrote: > The API doesn't hold a way to specify a owner/permission set for vhost_user > created sockets. Yes, it's kind of like a known issue. So, thanks for bringing it, with a solution, for dicussion (cc'ed more people). > I don't even think an API change would make that much sense. > > Projects consuming DPDK start to do 'their own workarounds' like openvswitch > https://patchwork.ozlabs.org/patch/559043/ > https://patchwork.ozlabs.org/patch/559045/ > But for this specific example they are blocked/stalled behind a bigger > rework (https://patchwork.ozlabs.org/patch/604898/). > Also one could ask why each project would need their own workaround. > > At the same time - as I want it for existing code linking against DPDK I > wanted to avoid changing API/ABI. That way I want to provide something existing > users could utilize. So I created a DPDK EAL commandline option based ideas in > the former patches. > > For myself I consider this a nice interim solution for existing released > Openvswitch+DPDK solution. And I intend to put it as delta into the DPDK 2.2 > currently packaged in Ubuntu to get it working more smoothly with > openvswitch 2.5. > > But I'd be interested if DPDK in general would be interested in: > a) an approach like this? You were trying to add a vhost specific stuff as EAL command option, which is something we might should try to avoid. > b) would prefer a change of the API? Adding a new option to the current register API might will not work well, either. It gives you no ability to do a dynamic change later. I mean, taking OVS as an example, OVS provides you the flexible ability to do all kinds of configuration in a dynamic way, say number of rx queues. If we do the permissions setup in the register time, there would be no way to change it later, right? So, I'm thinking that we may could add a new API for that? It then would allow applications to change it at anytime. > c) consider it an issue of consuming projects and let them take care? It's not exactly an issue of consuming projects; we created the socket file after all. And I'd like to hear what others would say. Thanks. --yliu