Re: [dpdk-dev] [PATCH] vhost: return error message for mbuf allocation failure

[Maxime Coquelin <maxime.coquelin@redhat.com>]

On 10/24/19 11:39 AM, Maxime Coquelin wrote:
> 
> 
> On 10/16/19 4:31 PM, Ilya Maximets wrote:
>> mbuf allocation failure is a hard failure that highlights some
>> significant issues with memory pool size or a mbuf leak.
>>
>> We still have the message for subsequent chained mbufs, but not
>> for the first one.  It was removed while introducing extbuf
>> support for large buffers.  But it was useful for catching
>> mempool issues and needs to be returned back.
>>
>> Cc: Flavio Leitner <fbl@sysclose.org>
>>
>> Fixes: 5005bcda7123 ("vhost: add support for large buffers")
>> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
>> ---
>>  lib/librte_vhost/virtio_net.c | 5 ++++-
>>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> 
> Thanks!
> Maxime
> 


Applied to dpdk-next-virtio/master.

Thanks,
Maxime