On 10/2/2018 2:27 PM, Ferruh Yigit wrote:
> On 10/2/2018 2:06 PM, Thomas Monjalon wrote:
>> Hi,
>>
>> 26/09/2018 18:21, Igor Ryzhov:
>>> Long time ago preallocation of memory for KNI was introduced in commit
>>> 0c6bc8e. It was done because of lack of ability to free previously
>>> allocated memzones, which led to memzone exhaustion. Currently memzones
>>> can be freed and this patch uses this ability for dynamic KNI memory
>>> allocation.
>>>
>>> Signed-off-by: Igor Ryzhov <iryz...@nfware.com>
>>
>> The patch does not apply on master.
>> Is there a dependency?
>
> There should not be a conflict, following commit already in tree is causing
> the
There should not be a _dependency_, ...
> conflict, I was resolving it myself when testing:
> Commit e716b639856c ("kni: fix crash with null name")
>
> Basically both do same thing:
> <<<<<<< IN REPO
> if (name == NULL || name[0] == '\0')
> return NULL;
> =======
> if (!name || !name[0])
> return NULL;
> >>>>>>> PATCH
>
>
> It seems Igor has a older version of the tree. It is OK to keep the check
> existing in the repo.
>
