-----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.

    
    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.


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

Reply via email to