On 9/26/17, 8:06 AM, "[email protected] on behalf of 
[email protected]" <[email protected] on behalf of 
[email protected]> wrote:

    Log an error message when the creation of a name for a
    new mempool fails.
    
    CC: Ciara Loftus <[email protected]>
    CC: Kevin Traynor <[email protected]>
    CC: Aaron Conole <[email protected]>
    Signed-off-by: Antonio Fischetti <[email protected]>
    ---
     lib/netdev-dpdk.c | 3 +++
     1 file changed, 3 insertions(+)
    
    diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
    index f3f42ee..7c673ec 100644
    --- a/lib/netdev-dpdk.c
    +++ b/lib/netdev-dpdk.c
    @@ -502,6 +502,9 @@ dpdk_mp_name(struct dpdk_mp *dmp)
         int ret = snprintf(mp_name, RTE_MEMPOOL_NAMESIZE, "ovs_%x_%d_%u",
                            h, dmp->mtu, dmp->mp_size);
         if (ret < 0 || ret >= RTE_MEMPOOL_NAMESIZE) {
    +        VLOG_ERR("Failed to generate a mempool name for \"%s\". "
    +                "Hash:0x%x, mtu:%d, mbufs:%u",
    +                dmp->if_name, h, dmp->mtu, dmp->mp_size);

[Darrell] “Unlikely” to fail but this could be an ovs_assert(…)


             return NULL;
         }
         return mp_name;
    -- 
    2.4.11
    
    _______________________________________________
    dev mailing list
    [email protected]
    
https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=VXc6b9RFgB9FaMDL4JevWOtvj0gTpvDJYBS30fvVj8Y&s=Eeo-qAFITbCVhSWvMvxHDN4IzLsDEIYI1MmIsoUtDOE&e=
 
    

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to