Re: [PATCH v2] cxl: Fix error path on bad ioctl

2017-06-09 Thread christophe lombard
Reviewed-by: christophe lombard Le 06/06/2017 à 11:43, Frederic Barrat a écrit : Fix error path if we can't copy user structure on CXL_IOCTL_START_WORK ioctl. We shouldn't unlock the context status mutex as it was not locked (yet). Signed-off-by: Frederic Barrat Cc: sta...@vger.kernel.org Fi

Re: [PATCH v2] cxl: Fix error path on bad ioctl

2017-06-07 Thread Andrew Donnellan
Reviewed-by: Andrew Donnellan On 06/06/17 19:43, Frederic Barrat wrote: Fix error path if we can't copy user structure on CXL_IOCTL_START_WORK ioctl. We shouldn't unlock the context status mutex as it was not locked (yet). Signed-off-by: Frederic Barrat Cc: sta...@vger.kernel.org Fixes: 0712d

Re: [PATCH v2] cxl: Fix error path on bad ioctl

2017-06-07 Thread Vaibhav Jain
Hi Fred, Thanks for addressing the review comments. Frederic Barrat writes: > Fix error path if we can't copy user structure on CXL_IOCTL_START_WORK > ioctl. We shouldn't unlock the context status mutex as it was not > locked (yet). > > Signed-off-by: Frederic Barrat > Cc: sta...@vger.kernel.o

[PATCH v2] cxl: Fix error path on bad ioctl

2017-06-06 Thread Frederic Barrat
Fix error path if we can't copy user structure on CXL_IOCTL_START_WORK ioctl. We shouldn't unlock the context status mutex as it was not locked (yet). Signed-off-by: Frederic Barrat Cc: sta...@vger.kernel.org Fixes: 0712dc7e73e5 ("cxl: Fix issues when unmapping contexts") --- Changelog: v2: chang