Shie-rei Huang was kind to point out that patch 2 needs to unlock udev->cmdr_lock before returning. Version 2 makes this change, and also places the || at the end of the line in the conditional, while we're at it. Sorry about that. I've made a personal note to scrutinize changes that introduce early returns more carefully in the future. -- Andy Andy Grover (3): target/user: Use sense_reason_t in tcmu_queue_cmd_ring target/user: Return an error if cmd data size is too large target/user: Fix comments to not refer to data ring drivers/target/target_core_user.c | 48 ++++++++++++++++----------------------- 1 file changed, 20 insertions(+), 28 deletions(-) -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
