From: Subash Abhinov Kasiviswanathan <subas...@codeaurora.org> Date: Tue, 29 Aug 2017 18:47:55 -0600
> I have updated the locking scheme as follows - Series applied, but this is not how you write a header posting for a patch set. This posting is where you say at a high level what the patch series is doing, how it is doing it, and why it is doing it that way. You can explain what changes happened, and why, but that belongs in the changelog at the end of this posting. Here you've made an explaination for one change the entire content of the text. You not even saying what rmnet is, why we would want to add it to the kernel, and what it's all about. So now when someone tries to read the merge commit that contains this text, they will have no context about you and me talking about locking and they will thus ask themselves "what is this person talking about here? it's not explaining the patch series at all"
