On Sun, Jan 31, 2016 at 10:05:09AM +0530, Parav Pandit wrote: > Changes from v4: > * Fixed compilation errors for lockdep_assert_held reported by kbuild > test robot > * Fixed compilation warning reported by coccinelle for extra semicolon. > * Fixed compilation error for inclusion of linux/parser.h which cannot > be included in any header file, as that triggers multiple inclusion > error. Instead parser.h is included in C files which intent to use it. > * Removed unused header file inclusion in cgroup_rdma.c > > Changes from v3: > * (To address comments from Tejun) > 1. Renamed cg_resource to rdmacg_resource > 2. Merged dealloc_cg_rpool and _dealloc_cg_rpool to single function > 3. Renamed _find_cg_rpool to find_cg_rpool_locked() > 5. Removed RDMACG_MAX_RESOURCE_INDEX limitation > 6. Fixed few alignments. > 7. Improved description for RDMA cgroup configuration menu > 8. Renamed cg_list_lock to rpool_list_lock to reflect the lock > is for rpools. > 9. Renamed _get_cg_rpool to find_cg_rpool. > 10. Made creator as int variable, instead of atomic as its not > required to be atomic. > * Fixed freeing right rpool during query_limit error path > * Added copywrite for cgroup.c > * Removed including parser.h from cgroup.c as its included by cgroup_rdma.h > * Reduced try_charge functions to single function and removed duplicate > comments.
Please don't send out patchsets in rapid succession while design discussion is still in progress. It's tiring for everyone involved. Thanks. -- tejun
