> On 8/12/20 8:19 PM, Jose E. Marchesi wrote:
>> Hi Martin.
>> 
>>>> I left the changelog entry dates of the original commits untouched,
>>>> and added `(cherry-pick from commit XXX)' lines to the commit
>>>> messages.  Hope that is ok... please let me know otherwise!
>>>
>>> Hello.
>>>
>>> For creating a backport please use contrib/git-backport.py script.
>>> The script basically runs 'git cherry-pick -x' and reverts all modifications
>>> of ChangeLog files.
>>>
>>> So your line:
>>>
>>> (cherry pick of commit af30b83b50953fbbe671d93d44ea6ac2f7a50ce9)
>>>
>>> should be:
>>> (cherry picked from commit af30b83b50953fbbe671d93d44ea6ac2f7a50ce9)
>>>
>>> You can then check your commits with git gcc-verify hook (from
>>> contrib/gcc-git-customization.sh).
>> I checked each commit with git gcc-verify, but since I didn't know
>> about
>> git-backport.py I used cherry-pick -n and reverted the ChangeLog entries
>> by hand.
>
> I see!
>
> Can you please paste content of the 'git gcc-verify -p HEAD~xxx..HEAD' for
> the backport? Can you see the 'Backported from master:' lines in there?

See below.  There are no such lines in the generated ChangeLog entries.

I looked in contrib/gcc-changelog/git_commit.py, and I can guess this is
due to:

1) CHERRY_PICK_PREFIX = '(cherry picked from commit ' and I used
   a slightly differnt wording.

2) If I am not mistaken while reading the script, the CHERRY_PICK line
   should be part of the ChangeLog entries (indented, etc) and I did put
   it before the ChangeLog entries instead.


Checking d8f4f1903eac5853b51e56b8e80743bc66adbb3d: OK
------ gcc/ChangeLog ------ 
2020-08-07  Jose E. Marchesi  <jose.march...@oracle.com>

        * config/bpf/bpf.md: Remove trailing whitespaces.
        * config/bpf/constraints.md: Likewise.
        * config/bpf/predicates.md: Likewise.
------ gcc/testsuite/ChangeLog ------ 
2020-08-07  Jose E. Marchesi  <jose.march...@oracle.com>

        * gcc.target/bpf/diag-funargs-2.c: Remove trailing whitespaces.
        * gcc.target/bpf/skb-ancestor-cgroup-id.c: Likewise.
        * gcc.target/bpf/helper-xdp-adjust-meta.c: Likewise.
        * gcc.target/bpf/helper-xdp-adjust-head.c: Likewise.
        * gcc.target/bpf/helper-tcp-check-syncookie.c: Likewise.
        * gcc.target/bpf/helper-sock-ops-cb-flags-set.c
        * gcc.target/bpf/helper-sysctl-set-new-value.c: Likewise.
        * gcc.target/bpf/helper-sysctl-get-new-value.c: Likewise.
        * gcc.target/bpf/helper-sysctl-get-name.c: Likewise.
        * gcc.target/bpf/helper-sysctl-get-current-value.c: Likewise.
        * gcc.target/bpf/helper-strtoul.c: Likewise.
        * gcc.target/bpf/helper-strtol.c: Likewise.
        * gcc.target/bpf/helper-sock-map-update.c: Likewise.
        * gcc.target/bpf/helper-sk-storage-get.c: Likewise.
        * gcc.target/bpf/helper-sk-storage-delete.c: Likewise.
        * gcc.target/bpf/helper-sk-select-reuseport.c: Likewise.
        * gcc.target/bpf/helper-sk-release.c: Likewise.
        * gcc.target/bpf/helper-sk-redirect-map.c: Likewise.
        * gcc.target/bpf/helper-sk-lookup-upd.c: Likewise.
        * gcc.target/bpf/helper-sk-lookup-tcp.c: Likewise.
        * gcc.target/bpf/helper-skb-change-head.c: Likewise.
        * gcc.target/bpf/helper-skb-cgroup-id.c: Likewise.
        * gcc.target/bpf/helper-skb-adjust-room.c: Likewise.
        * gcc.target/bpf/helper-set-hash.c: Likewise.
        * gcc.target/bpf/helper-setsockopt.c: Likewise.
        * gcc.target/bpf/helper-redirect-map.c: Likewise.
        * gcc.target/bpf/helper-rc-repeat.c: Likewise.
        * gcc.target/bpf/helper-rc-keydown.c: Likewise.
        * gcc.target/bpf/helper-probe-read-str.c: Likewise.
        * gcc.target/bpf/helper-perf-prog-read-value.c: Likewise.
        * gcc.target/bpf/helper-perf-event-read-value.c: Likewise.
        * gcc.target/bpf/helper-override-return.c: Likewise.
        * gcc.target/bpf/helper-msg-redirect-map.c: Likewise.
        * gcc.target/bpf/helper-msg-pull-data.c: Likewise.
        * gcc.target/bpf/helper-msg-cork-bytes.c: Likewise.
        * gcc.target/bpf/helper-msg-apply-bytes.c: Likewise.
        * gcc.target/bpf/helper-lwt-seg6-store-bytes.c: Likewise.
        * gcc.target/bpf/helper-lwt-seg6-adjust-srh.c: Likewise.
        * gcc.target/bpf/helper-lwt-seg6-action.c: Likewise.
        * gcc.target/bpf/helper-lwt-push-encap.c: Likewise.
        * gcc.target/bpf/helper-get-socket-uid.c: Likewise.
        * gcc.target/bpf/helper-get-socket-cookie.c: Likewise.
        * gcc.target/bpf/helper-get-local-storage.c: Likewise.
        * gcc.target/bpf/helper-get-current-cgroup-id.c: Likewise.
        * gcc.target/bpf/helper-getsockopt.c: Likewise.
        * gcc.target/bpf/diag-funargs-3.c: Likewise.
Checking 9248c12b2915a07e1e258ff2226b2241663312d1: OK
------ gcc/ChangeLog ------ 
2020-08-06  Jose E. Marchesi  <jose.march...@oracle.com>

        * config/bpf/bpf-helpers.h (KERNEL_HELPER): Define.
        (KERNEL_VERSION): Remove.
        * config/bpf/bpf-helpers.def: Delete.
        * config/bpf/bpf.c (bpf_handle_fndecl_attribute): New function.
        (bpf_attribute_table): Define.
        (bpf_helper_names): Delete.
        (bpf_helper_code): Likewise.
        (enum bpf_builtins): Adjust to new helpers mechanism.
        (bpf_output_call): Likewise.
        (bpf_init_builtins): Likewise.
        (bpf_init_builtins): Likewise.
        * doc/extend.texi (BPF Function Attributes): New section.
        (BPF Kernel Helpers): Delete section.
------ gcc/testsuite/ChangeLog ------ 
2020-08-06  Jose E. Marchesi  <jose.march...@oracle.com>

        * gcc.target/bpf/helper-bind.c: Adjust to new kernel helpers
        mechanism.
        * gcc.target/bpf/helper-bpf-redirect.c: Likewise.
        * gcc.target/bpf/helper-clone-redirect.c: Likewise.
        * gcc.target/bpf/helper-csum-diff.c: Likewise.
        * gcc.target/bpf/helper-csum-update.c: Likewise.
        * gcc.target/bpf/helper-current-task-under-cgroup.c: Likewise.
        * gcc.target/bpf/helper-fib-lookup.c: Likewise.
        * gcc.target/bpf/helper-get-cgroup-classid.c: Likewise.
        * gcc.target/bpf/helper-get-current-cgroup-id.c: Likewise.
        * gcc.target/bpf/helper-get-current-comm.c: Likewise.
        * gcc.target/bpf/helper-get-current-pid-tgid.c: Likewise.
        * gcc.target/bpf/helper-get-current-task.c: Likewise.
        * gcc.target/bpf/helper-get-current-uid-gid.c: Likewise.
        * gcc.target/bpf/helper-get-hash-recalc.c: Likewise.
        * gcc.target/bpf/helper-get-listener-sock.c: Likewise.
        * gcc.target/bpf/helper-get-local-storage.c: Likewise.
        * gcc.target/bpf/helper-get-numa-node-id.c: Likewise.
        * gcc.target/bpf/helper-get-prandom-u32.c: Likewise.
        * gcc.target/bpf/helper-get-route-realm.c: Likewise.
        * gcc.target/bpf/helper-get-smp-processor-id.c: Likewise.
        * gcc.target/bpf/helper-get-socket-cookie.c: Likewise.
        * gcc.target/bpf/helper-get-socket-uid.c: Likewise.
        * gcc.target/bpf/helper-get-stack.c: Likewise.
        * gcc.target/bpf/helper-get-stackid.c: Likewise.
        * gcc.target/bpf/helper-getsockopt.c: Likewise.
        * gcc.target/bpf/helper-ktime-get-ns.c: Likewise.
        * gcc.target/bpf/helper-l3-csum-replace.c: Likewise.
        * gcc.target/bpf/helper-l4-csum-replace.c: Likewise.
        * gcc.target/bpf/helper-lwt-push-encap.c: Likewise.
        * gcc.target/bpf/helper-lwt-seg6-action.c: Likewise.
        * gcc.target/bpf/helper-lwt-seg6-adjust-srh.c: Likewise.
        * gcc.target/bpf/helper-lwt-seg6-store-bytes.c: Likewise.
        * gcc.target/bpf/helper-map-delete-elem.c: Likewise.
        * gcc.target/bpf/helper-map-lookup-elem.c: Likewise.
        * gcc.target/bpf/helper-map-peek-elem.c: Likewise.
        * gcc.target/bpf/helper-map-pop-elem.c: Likewise.
        * gcc.target/bpf/helper-map-push-elem.c: Likewise.
        * gcc.target/bpf/helper-map-update-elem.c: Likewise.
        * gcc.target/bpf/helper-msg-apply-bytes.c: Likewise.
        * gcc.target/bpf/helper-msg-cork-bytes.c: Likewise.
        * gcc.target/bpf/helper-msg-pop-data.c: Likewise.
        * gcc.target/bpf/helper-msg-pull-data.c: Likewise.
        * gcc.target/bpf/helper-msg-push-data.c: Likewise.
        * gcc.target/bpf/helper-msg-redirect-hash.c: Likewise.
        * gcc.target/bpf/helper-msg-redirect-map.c: Likewise.
        * gcc.target/bpf/helper-override-return.c: Likewise.
        * gcc.target/bpf/helper-perf-event-output.c: Likewise.
        * gcc.target/bpf/helper-perf-event-read-value.c: Likewise.
        * gcc.target/bpf/helper-perf-event-read.c: Likewise.
        * gcc.target/bpf/helper-perf-prog-read-value.c: Likewise.
        * gcc.target/bpf/helper-probe-read-str.c: Likewise.
        * gcc.target/bpf/helper-probe-read.c: Likewise.
        * gcc.target/bpf/helper-probe-write-user.c: Likewise.
        * gcc.target/bpf/helper-rc-keydown.c: Likewise.
        * gcc.target/bpf/helper-rc-pointer-rel.c: Likewise.
        * gcc.target/bpf/helper-rc-repeat.c: Likewise.
        * gcc.target/bpf/helper-redirect-map.c: Likewise.
        * gcc.target/bpf/helper-set-hash-invalid.c: Likewise.
        * gcc.target/bpf/helper-set-hash.c: Likewise.
        * gcc.target/bpf/helper-setsockopt.c: Likewise.
        * gcc.target/bpf/helper-sk-fullsock.c: Likewise.
        * gcc.target/bpf/helper-sk-lookup-tcp.c: Likewise.
        * gcc.target/bpf/helper-sk-lookup-upd.c: Likewise.
        * gcc.target/bpf/helper-sk-redirect-hash.c: Likewise.
        * gcc.target/bpf/helper-sk-redirect-map.c: Likewise.
        * gcc.target/bpf/helper-sk-release.c: Likewise.
        * gcc.target/bpf/helper-sk-select-reuseport.c: Likewise.
        * gcc.target/bpf/helper-sk-storage-delete.c: Likewise.
        * gcc.target/bpf/helper-sk-storage-get.c: Likewise.
        * gcc.target/bpf/helper-skb-adjust-room.c: Likewise.
        * gcc.target/bpf/helper-skb-cgroup-id.c: Likewise.
        * gcc.target/bpf/helper-skb-change-head.c: Likewise.
        * gcc.target/bpf/helper-skb-change-proto.c: Likewise.
        * gcc.target/bpf/helper-skb-change-tail.c: Likewise.
        * gcc.target/bpf/helper-skb-change-type.c: Likewise.
        * gcc.target/bpf/helper-skb-ecn-set-ce.c: Likewise.
        * gcc.target/bpf/helper-skb-get-tunnel-key.c: Likewise.
        * gcc.target/bpf/helper-skb-get-tunnel-opt.c: Likewise.
        * gcc.target/bpf/helper-skb-get-xfrm-state.c: Likewise.
        * gcc.target/bpf/helper-skb-load-bytes-relative.c: Likewise.
        * gcc.target/bpf/helper-skb-load-bytes.c: Likewise.
        * gcc.target/bpf/helper-skb-pull-data.c: Likewise.
        * gcc.target/bpf/helper-skb-set-tunnel-key.c: Likewise.
        * gcc.target/bpf/helper-skb-set-tunnel-opt.c: Likewise.
        * gcc.target/bpf/helper-skb-store-bytes.c: Likewise.
        * gcc.target/bpf/helper-skb-under-cgroup.c: Likewise.
        * gcc.target/bpf/helper-skb-vlan-pop.c: Likewise.
        * gcc.target/bpf/helper-skb-vlan-push.c: Likewise.
        * gcc.target/bpf/helper-skc-lookup-tcp.c: Likewise.
        * gcc.target/bpf/helper-sock-hash-update.c: Likewise.
        * gcc.target/bpf/helper-sock-map-update.c: Likewise.
        * gcc.target/bpf/helper-sock-ops-cb-flags-set.c: Likewise.
        * gcc.target/bpf/helper-spin-lock.c: Likewise.
        * gcc.target/bpf/helper-spin-unlock.c: Likewise.
        * gcc.target/bpf/helper-strtol.c: Likewise.
        * gcc.target/bpf/helper-strtoul.c: Likewise.
        * gcc.target/bpf/helper-sysctl-get-current-value.c: Likewise.
        * gcc.target/bpf/helper-sysctl-get-name.c: Likewise.
        * gcc.target/bpf/helper-sysctl-get-new-value.c: Likewise.
        * gcc.target/bpf/helper-sysctl-set-new-value.c: Likewise.
        * gcc.target/bpf/helper-tail-call.c: Likewise.
        * gcc.target/bpf/helper-tcp-check-syncookie.c: Likewise.
        * gcc.target/bpf/helper-tcp-sock.c: Likewise.
        * gcc.target/bpf/helper-trace-printk.c: Likewise.
        * gcc.target/bpf/helper-xdp-adjust-head.c: Likewise.
        * gcc.target/bpf/helper-xdp-adjust-meta.c: Likewise.
        * gcc.target/bpf/helper-xdp-adjust-tail.c: Likewise.
        * gcc.target/bpf/skb-ancestor-cgroup-id.c: Likewise.
Checking 5554630d50344dbe941c90052bbee74f96950917: OK
------ gcc/ChangeLog ------ 
2020-05-19  Jose E. Marchesi  <jose.march...@oracle.com>

        * config/bpf/bpf.c (bpf_compute_frame_layout): Include space for
        callee saved registers only in xBPF.
        (bpf_expand_prologue): Save callee saved registers only in xBPF.
        (bpf_expand_epilogue): Likewise for restoring.
        * doc/invoke.texi (eBPF Options): Document this is activated by
        -mxbpf.
------ gcc/testsuite/ChangeLog ------ 
2020-05-19  Jose E. Marchesi  <jose.march...@oracle.com>

        * gcc.target/bpf/xbpf-callee-saved-regs-1.c: New test.
        * gcc.target/bpf/xbpf-callee-saved-regs-2.c: Likewise.
Checking 148b8c8e4c1332d8d11187f0d67148b4d3f6c8bb: OK
------ gcc/ChangeLog ------ 
2020-05-19  Jose E. Marchesi  <jose.march...@oracle.com>

        * config/bpf/bpf.opt (mxbpf): New option.
        * doc/invoke.texi (Option Summary): Add -mxbpf.
        (eBPF Options): Document -mxbbpf.

Reply via email to