<div dir="ltr"><div class="gmail_extra"><div class="gmail_extra">On Mon, Apr 17, 2017 at 9:40 PM, Dmitry V. Levin wrote:</div><div class="gmail_extra">>On Wed, Mar 29, 2017 at 05:39:59PM +0800, JingPiao Chen wrote:</div><div class="gmail_extra">>> I updated the patch[1] and the proposal draft[2].</div><div class="gmail_extra">>> Please provide some feedback.</div><div class="gmail_extra">>> [1]<a href="https://github.com/ppiao/strace/tree/nl-socket">https://github.com/ppiao/strace/tree/nl-socket</a></div><div class="gmail_extra">></div><div class="gmail_extra">>As I said, every logically separate change must go to an individual commit,</div><div class="gmail_extra">>the way Fabien did in his branch.  You can rebase/fix/whatever commits</div><div class="gmail_extra">>made by Fabien, but please don't squash them -- nobody is going to spend</div><div class="gmail_extra">>time on a squashed netlink branch.</div><div class="gmail_extra">></div><div class="gmail_extra">>I've just pushed several netlink related commits to master, you can review</div><div class="gmail_extra">>them and use as examples of commit separation, too.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thank you for your reply. I have read the code.</div><div class="gmail_extra"><br></div><div class="gmail_extra">>Note that at least some of commits in netlink branch contain bugs,</div><div class="gmail_extra">>including design bugs.  For example, the first commit "Add support for</div><div class="gmail_extra">>NLMSG_ERROR and NLMSG_DONE" stops netlink processing when NLMSG_ERROR</div><div class="gmail_extra">>or NLMSG_DONE is encountered, which is incorrect.</div><div class="gmail_extra">--</div><div class="gmail_extra">JingPiao Chen</div></div></div>