From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] devargs: restore empty devargs as "" Date: Tue, 24 Feb 2015 20:24:20 +0100 Message-ID: <1836720.DS2V1117TQ@xps13> References: <54E5ED96.9040402@igel.co.jp> <1424770891-19243-1-git-send-email-david.marchand@6wind.com> <54EC4F7B.6080907@igel.co.jp> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: David Marchand Return-path: In-Reply-To: <54EC4F7B.6080907-AlSX/UN32fvPDbFq/vQRIQ@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 2015-02-24 19:16, Tetsuya Mukawa: > On 2015/02/24 18:41, David Marchand wrote: > > Following commit c07691ae1089, an implicit change has been done in the devargs > > api. > > This triggers problem in virtual pmds that did not check for parameters validity > > as it was implicitely valid. > > > > Fix this by restoring the empty argument as "" and add a note in the api. > > Restore associated tests. > > > > Fixes: c07691ae1089 ("devargs: remove limit on parameters length") > > Reported-by: Tetsuya Mukawa > > Signed-off-by: David Marchand > > Hi David, > > I've confirmed this patch fixes the issue descried in above comment. Applied, thanks