On 6/29/23 18:17, miter wrote: > Hi Ilya, > > Thank you for your advice. 😁 > > Can I take 2 minutes off of your time? 😉 > > Does this command work correctly? I usually use this command to send > patches. > "git send-email --to d...@openvswitch.org --cover-letter --annotate -v6 -4"
Hmm. Yes, it should. If you're using this command, then I'm not sure how your patches end up having incorrect 'In-Reply-To' headers... Weird. Looking closer, your emails have: # In-Reply-To: <0> # References: <0> And that is messing up thread tracking. I wonder if it's your mail server doing that. Best regards, Ilya Maximets. > > Best regards, Lin Huang. > > On 6/29/2023 10:51 PM, Ilya Maximets wrote: >> On 6/18/23 15:58, mit...@outlook.com wrote: >>> From: Lin Huang <linhu...@ruijie.com.cn> >>> >>> v5->v4: police pkts bps and pps at the same time. >>> Get the maximum from the number of dropped packets, and drop that. >>> >>> v4->v3: rewrite egress_policer_details_to_param func. >>> add a new pkts_policer_profile_config func. >>> >>> v2->v3: police and free pkts by bulk. >>> fix the appropriate error code. >>> >>> v1->v2: delete duplicated code. >>> >>> Lin Huang (4): >>> token-bucket: Make token-bucket timestamp updated by caller. >>> netdev-dpdk: make srtcm_policer not to free pkts. >>> netdev-dpdk: Add support for egress packet-per-second policing. >>> netdev-dpdk: Add support for ingress packet-per-second policing. >>> >>> Documentation/topics/dpdk/qos.rst | 29 +- >>> NEWS | 5 + >>> include/openvswitch/token-bucket.h | 3 +- >>> lib/netdev-dpdk.c | 233 ++++++++++++---- >>> lib/token-bucket.c | 4 +- >>> lib/vlog.c | 3 +- >>> ofproto/pinsched.c | 2 +- >>> tests/system-dpdk.at | 412 ++++++++++++++++++++++++++++- >>> vswitchd/vswitch.xml | 10 + >>> 9 files changed, 641 insertions(+), 60 deletions(-) >>> >> >> Thanks for v5! I tried this version and it seems to work fine. >> >> I posted a few small comments for the code. Please, address them >> and send a new version. >> >> BTW, v5 was sent in a strange way with patches sent in reply to >> some wrong ones. Please, send v6 without in-reply-to header. >> Patches should generally be sent as replies to a cover letter, >> but the cover letter itself should be a standalone message without >> in-reply-to header. >> >> Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
