Sun, May 27, 2018 at 06:41:27PM CEST, vla...@mellanox.com wrote: [...]
>Changes from V1 to V2: >- Removed redundant actions ops lookup during delete. >- Merge action ops delete definition and implementation. >- Assume all actions have delete implemented and don't check for it > explicitly. >- Resplit action lookup/release code to prevent memory leaks in > individual patches. >- Make __tcf_idr_check function static >- Remove unique idr insertion function. Change original idr insert to do > the same thing. >- Merge changes that take reference to action when performing lookup and > changes that account for this additional reference when dumping action > to user space into single patch. >- Change convoluted commit message. >- Rename "unlocked" to "rtnl_held" for clarity. >- Remove estimator lock add patch. >- Refactor action check-alloc code into standalone function. >- Rename tcf_idr_find_delete to tcf_idr_delete_index. >- Rearrange variable definitions in tc_action_delete. >- Add patch that refactors action API code to use array of pointers to > actions instead of intrusive linked list. >- Expand cover letter. With 11 patches, I'm unable to follow the changes in this description. Could you please write the changes to each individual patch? What I do is I write changes for each patch under "---" (there are two "---" lines in each patch then. Then I also copy the changes descriptions to the cover letter. [...]