From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrien Mazarguil Subject: Re: [dpdk-dev, 5/7] net/mlx5: add port representor awareness Date: Tue, 12 Jun 2018 15:20:22 +0200 Message-ID: <20180612132022.GX4025@6wind.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Shahaf Shuler , "dev@dpdk.org" To: "Xueming(Steven) Li" Return-path: Received: from mail-wm0-f65.google.com (mail-wm0-f65.google.com [74.125.82.65]) by dpdk.org (Postfix) with ESMTP id B88361E318 for ; Tue, 12 Jun 2018 15:20:38 +0200 (CEST) Received: by mail-wm0-f65.google.com with SMTP id j15-v6so23375888wme.0 for ; Tue, 12 Jun 2018 06:20:38 -0700 (PDT) Content-Disposition: inline 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 Mon, Jun 11, 2018 at 01:05:55PM +0000, Xueming(Steven) Li wrote: > Hi Adrien, > > Couldn't find your original email from inbox anyway, have to start a new thread here. > > +static int > > +mlx5_cmp_ibv_name(const void *a, const void *b) > > +{ > > + const char *name_a = (*(const struct ibv_device *const *)a)->name; > > + const char *name_b = (*(const struct ibv_device *const *)b)->name; > > + size_t i = 0; > > + > > + while (name_a[i] && name_a[i] == name_b[i]) > > + ++i; > > + return atoi(name_a + i) - atoi(name_b + i); > > Comparing "1" and "10" here will return 0, does this matter? Sure it does! The whole point of this function is precisely to avoid this kind of issues. I'll fix it for v2, thanks. > > + if (n > 1) { > > + /* > > + * The existence of several matching entries means port > > + * representors have been instantiated. No existing Verbs > > + * call nor /sys entries can tell them apart at this point. > > + * > > + * While definitely hackish, assume their names are numbered > > + * based on order of creation with master device first, > > + * followed by first port representor, followed by the > > + * second one and so on. > > + */ > > + DRV_LOG(WARNING, > > + "probing device with port representors involves" > > + " heuristics with uncertain outcome"); > > + qsort(ibv_match, n, sizeof(*ibv_match), mlx5_cmp_ibv_name); > > + DRV_LOG(WARNING, "assuming \"%s\" is the master device", > > + ibv_match[0]->name); > > + for (ret = 1; ret < n; ++ret) > > + DRV_LOG(WARNING, > > + "assuming \"%s\" is port representor #%d", > > + ibv_match[ret]->name, ret - 1); > > Such dump will appear when attaching each rep port, how about just > do it for PF in DEBUG level? It occurs only once when probing the master device and detecting the presence of representors, not for each of them. I prefer to leave it as a warning because this detection approach, while an undeniable improvement over not checking anything and ending up configuring the wrong netdevice, is unfortunately not 100% accurate. This will be improved, however users must be warned of possible issues in the meantime. -- Adrien Mazarguil 6WIND