[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <bbe9343c-fb43-ce8d-a871-033f3556c222@deltatee.com>
Date: Thu, 30 Nov 2017 15:26:39 -0700
From: Logan Gunthorpe <logang@...tatee.com>
To: Serge Semin <fancer.lancer@...il.com>
Cc: jdmason@...zu.us, dave.jiang@...el.com, Allen.Hubbe@....com,
Shyam-sundar.S-k@....com, Xiangliang.Yu@....com,
Sergey.Semin@...latforms.ru, linux-ntb@...glegroups.com,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 01/08] NTB: ntb_test: Safely use paths with whitespace
On 30/11/17 03:12 PM, Serge Semin wrote:
> In general the intention of the patchset was to update the ntb_test.sh script
> so one would be compatible with new NTB test drivers I've also sent. It was
> asked of me by the NTB core maintainers. Sorry for not mentioning it in the
> cover letter.
I understand that. But most importantly a series needs to say what it's
based off of and given than it clearly depends on the ntb_tool changes
(which I saw later) it was a bit confusing.
I think this should be in the same series as the ntb_tool changes as the
ntb_test changes depend on ntb_tool and we shouldn't merge the ntb_tool
changes without the ntb_test changes as it would break them.
Logan
Powered by blists - more mailing lists