From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg0-f43.google.com (mail-pg0-f43.google.com [74.125.83.43]) by dpdk.org (Postfix) with ESMTP id 58F818D35 for ; Thu, 26 Apr 2018 17:42:56 +0200 (CEST) Received: by mail-pg0-f43.google.com with SMTP id i6so16213860pgv.3 for ; Thu, 26 Apr 2018 08:42:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=2mmqP/u3N2OX/mEYg9c/MtpGIJTTlzP0y62rEue7x48=; b=sFHqLov0twbcxVS2cqKFnslnY36KajFzhwQKr6JnYkb04AtVOy3z2JLJFSy67mvQYE mcsBHkyTU2PolA6y6pF+p7kjls5Za0Q7149DnJdIjkGsCtjgYnl+Cp33jofNovU1rs++ wnz+dvZntDhpgfJ47/ioH9422Hjp8ZggLJ0GPKvdgSD94wjUeyVef2/+wqYFPcDDMW8g OiAT/VLomPh79QiQb3nCZGgiArPrq8s3B51vScMJ2ONW9DNkX5FDzJmboAqJ4B8BTT7S 2bIP4UCM5BHC6xdaB/FYStf+5+jl2zHCveV1p7atHJYWjuLMcn6ufl5V8znuoPVyI/Y4 /1wg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=2mmqP/u3N2OX/mEYg9c/MtpGIJTTlzP0y62rEue7x48=; b=j1KWs7oH9Yce034KzTI8mkR/mLUNHA6S9B89aGr8By/DGFXDSvuMcl+Oyxh+rmMHF+ vsFLzwn1gr2X0QVYDEnd3R4MV4GWGdnRSQzb1v0nl1ZUPvS7gh39lR4XH0DCtH5cZOgH BdHnWVToXBDboonELnOPbWWLg1tMki3h8BqMZKWjHzALImhB68lUt6EpIq66aer12T7F fChVA3Y+gKScHUG/dKxkP8r4vBhxnprGF56tobPnO2jjsNd5oj2cK77eG/6c2NfaONI2 QTraSKfjgHeCsYmkO/sDmnsyQugMSQTZgi52o7tyyzxyyfbIqvCne+w5PNIBBx8H2l5V SQjg== X-Gm-Message-State: ALQs6tCWWiDVcldKM1njM3gRa4thWsUmSYWdGJNw204pZmQh8Iym5G/e +jyxIFTZI5Z3hPezwWoQY8jYHg== X-Google-Smtp-Source: AIpwx48wB4ucVn0JMzkuIxAC78Dm8r59Nudu4c/AL2ir2CROiwJMYYIuy35WM1qXB5u9WKosRnyZTA== X-Received: by 10.167.128.207 with SMTP id a15mr32662312pfn.116.1524757375459; Thu, 26 Apr 2018 08:42:55 -0700 (PDT) Received: from xeon-e3 (204-195-71-95.wavecable.com. [204.195.71.95]) by smtp.gmail.com with ESMTPSA id 131sm24559664pfa.103.2018.04.26.08.42.55 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 26 Apr 2018 08:42:55 -0700 (PDT) Date: Thu, 26 Apr 2018 08:42:53 -0700 From: Stephen Hemminger To: Alejandro Lucero Cc: dev Message-ID: <20180426084253.1a615c82@xeon-e3> In-Reply-To: References: <20180425154551.6795-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH 0/5] net/nfp logging fixes X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 26 Apr 2018 15:42:56 -0000 On Thu, 26 Apr 2018 13:52:53 +0100 Alejandro Lucero wrote: > Hi Stephen, > > Thanks for this patch set. > > I'm happy with it although I have some concerns regarding how the dynamic > logs work, or maybe I have a wrong understanding about it. I have tried to > read some doc about how it works, and I found the original patch from > Olivier the best source, so maybe things have changed a bit and my concerns > are unfounded. > > I think it is OK to specifically add something like > > --log-level='pmd\.i40e.*,8' > > if you want to debug a PMD, but if you are an user and you just want to > know why the app is not finding any port, finding out the right string is > not trivial. For example, with an PF, the NFP PMD goes through a process > where the NFP device (no the NIC) is accessed first through a complex > interface, then firmware is uploaded, DPDK ports created (for multiport > devices), etc. I think any error in that process should be output if the > right loglevel is there and not just if the right log type was specifically > enabled. Is this what would happen with your patchset? Most drivers set default log level to NOTICE. Then if they see something obviously wrong it will show up if the right log level is used. For the case of finding out why no drivers are found then doing something like --log-level='pmd.*:info' would be useful. Latest version makes regex optional and allows symbolic levels. > I have suffered silent configuration problems, like the NFP card being in > the wrong NUMA socket, and although I can solve that quickly because I have > the knowledge, other people using NFP with DPDK require someone to help > because they do not know what is going on. And this is usually bad because > they have another NIC card in the same host (in the right NUMA socket) and > the app just works smoothly then, leaving our NIC with a bad press. So I > think, some errors should always appear with the right loglevel configured. Driver should definitely use level > INFO for things that are wrong.