Darrell Ball <db...@vmware.com> writes:

> -----Original Message-----
> From: Aaron Conole <acon...@redhat.com>
> Date: Wednesday, August 9, 2017 at 12:51 PM
> To: Darrell Ball <db...@vmware.com>
> Cc: Joe Stringer <j...@ovn.org>, Darrell Ball <dlu...@gmail.com>, ovs
> dev <d...@openvswitch.org>
> Subject: Re: [ovs-dev] [patch_v4] dp-packet: Reset DPDK HWOL checksum flags 
> on init.
>
>     Darrell Ball <db...@vmware.com> writes:
>     
>     > Thanks Joe
>     > I forgot to add your Tested-by to V5; I have been testing this myself;
>     > but let me know if you would like it added – I can send a V6.
>     
>     It will automatically be added by patchwork.  It is sufficient to
>     download (ex: 
> https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.ozlabs.org_patch_799499_mbox&d=DwIFaQ&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=LK8s59ajLx5MbmH5Ng8SG1F1nAgKZJ_0JBs7phbl6C8&s=HRG5wrQrEXCl_AA8uAGbroVSvYGUIQjQkF8HrMRvNYI&e=
>  )
>     
>     I usually use patchwork when working on a series - if I download it from
>     patchwork, I can be confident that all the tags are applied properly, so
>     I won't forget.  Plus, all the discussion happens there, so I can
>     quickly browse it.
>
> Thanks Aaron
> In this case, the Tested-by was applied to V4 and I sent a V5 which did not 
> have the Tested-by.
> The easiest solution would have been to ask Joe to respond to V5 – well, it 
> is moot now anyways with V6 on the way.

Yep, that happens.

>     The full list is available at:
>     
> https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.ozlabs.org_project_openvswitch_list_&d=DwIFaQ&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=LK8s59ajLx5MbmH5Ng8SG1F1nAgKZJ_0JBs7phbl6C8&s=rqaCq5Jr-Vy-cfQ_t5px-zGtdb55CQn9cIAZCC1PvfE&e=
>  
>
> Thanks, I am on this page most of the day (
>     
>     It would actually be cool to have a few more admins to troll patchwork
>     and do things like ping for status (ex:
>     
> https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.ozlabs.org_patch_719492_&d=DwIFaQ&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=LK8s59ajLx5MbmH5Ng8SG1F1nAgKZJ_0JBs7phbl6C8&s=y0RPF47IRVw8BujWHzB9oc7IUihgLuCenLfteZ2vUr4&e=
>   
>
> I don’t have any context on this patch.
> Aaron, would you like to resurrect this patch with rebase ?; it is RHEL 
> related.

It doesn't need a rebase; it applies cleanly.  The version of patchwork
running doesn't recognize my ack, so someone will have to add it
manually (or omit it... doesn't matter too much, since Ben ack'd as
well).

But it would be cool to get it applied;  maybe even to get back as far
as branch-2.7 (I think it was posted in time for 2.7 branch).

> is this still needed?  it
>     does seem to mostly apply).  Then we could make sure we don't miss
>     things.
>     
>     > Darrell
>     >
>     > -----Original Message-----
>     > From: <ovs-dev-boun...@openvswitch.org> on behalf of Joe Stringer 
> <j...@ovn.org>
>     > Date: Tuesday, August 8, 2017 at 4:51 PM
>     > To: Darrell Ball <dlu...@gmail.com>
>     > Cc: ovs dev <d...@openvswitch.org>
>     > Subject: Re: [ovs-dev] [patch_v4] dp-packet: Reset DPDK HWOL checksum
>     > flags on init.
>     >
>     >     On 8 August 2017 at 16:39, Darrell Ball <dlu...@gmail.com> wrote:
>     >     > Reset the DPDK HWOL checksum flags in dp_packet_init_.
>     >     > The new HWOL bad checksum flag is uninitialized on non-dpdk ports 
> and
>     >     > this is noticed as test failures using netdev-dummy ports where 
> the bad
>     >     > checksum flag is checked.
>     >     >
>     >     > Fixes: 7451af618e0d ("dp-packet : Update DPDK rx checksum
>     >     > validation functions.")
>     >     > CC: Sugesh Chandran <sugesh.chand...@intel.com>
>     >     > Signed-off-by: Darrell Ball <dlu...@gmail.com>
>     >     > ---
>     >     
>     >     Tested-by: Joe Stringer <j...@ovn.org>
>     >     Tested-at: 
> https://urldefense.proofpoint.com/v2/url?u=https-3A__travis-2Dci.org_joestringer_openvswitch_jobs_262464859&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=jXM8RI10bFYBt8zzW6xxX0MRb4YxJ_uCcOdGTE0sfJo&s=d664GG1tgARiN5uEJxebCnczlUnMDtHdWluYN0dRc5g&e=
>  
>     >     
>     >     I'll let those more familiar with this code provide the review.
>     >     _______________________________________________
>     >     dev mailing list
>     >     d...@openvswitch.org
>     >     
> https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=jXM8RI10bFYBt8zzW6xxX0MRb4YxJ_uCcOdGTE0sfJo&s=kCTOzFrO_3CdwW2mM_wcQZXMHDtzpo8cAuKlHSOcxTw&e=
>  
>     >     
>     >
>     > _______________________________________________
>     > dev mailing list
>     > d...@openvswitch.org
>     > 
> https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwIFaQ&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=LK8s59ajLx5MbmH5Ng8SG1F1nAgKZJ_0JBs7phbl6C8&s=nnp778JNhzpZBELfmseffsohxJbAQgpKL0BLRS-vlmc&e=
>  
>     
>
> _______________________________________________
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to