> -----Original Message----- > From: dev [mailto:[email protected]] On Behalf Of Herakliusz Lipiec > Sent: Tuesday, April 23, 2019 7:43 PM > Cc: [email protected]; Lipiec, Herakliusz <[email protected]>; Zhang, > Qi Z <[email protected]>; [email protected] > Subject: [dpdk-dev] [PATCH v2 2/8] ipc: fix hotplug memleak > > When sending synchronous IPC requests, the caller must free the response > buffer even if the request returned failure. Fix the code to correctly > use the IPC API. > > Bugzilla ID: 228 > Fixes: ac9e4a17370f ("eal: support attach/detach shared device from > secondary") > Cc: [email protected] > Cc: [email protected] > Signed-off-by: Herakliusz Lipiec <[email protected]> > ---
Shouldn't the commit title be something like "hotplug: fix ipc memleak"? Or even more specifically "hotplug: fix memleak on ipc failure". For the patch itself: Reviewed-by: Darek Stojaczyk <[email protected]>
