On Wed, Nov 15, 2017 at 7:55 PM, Eric Blake <ebl...@redhat.com> wrote: > > On 11/15/2017 09:07 PM, Ed Swierk via Qemu-devel wrote: > > This part below... > > > > > v3: > > > > Leave net_tx_pkt_update_ip_checksums() alone since it's only computing > > a partial sum of the IP pseudo-header. > > > > Rename wrapper to net_checksum_finish_nozero() for clarity. > > > > v2: > > > > Add a wrapper net_checksum_finish_hdr() rather than duplicating the > > logic at every caller. > > ...through here, belongs... > > > > > Signed-off-by: Ed Swierk <eswi...@skyportsystems.com> > > --- > > ...here, after the --- separator (you can have more than one ---; 'git > am' stops parsing at the first). It is data that is useful to > reviewers, but will be meaningless a year from now when browsing git > history (at which point we won't care how many revisions the patch went > through on the list, but only what was checked in).
Thanks for the clue. I'll repost a corrected version. Maybe checkpatch.pl should squawk if it sees ^v[0-9]\+: before the first ---? --Ed