DPDK patches and discussions
 help / color / mirror / Atom feed
From: Ferruh Yigit <ferruh.yigit@intel.com>
To: Yong Wang <wang.yong19@zte.com.cn>, shepard.siegel@atomicrules.com
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH] net/ark:add null point check
Date: Thu, 14 Sep 2017 10:33:54 +0100	[thread overview]
Message-ID: <89a345e1-43ba-2a2a-4cd2-55e2bad1817d@intel.com> (raw)
In-Reply-To: <1505373770-24737-1-git-send-email-wang.yong19@zte.com.cn>

On 9/14/2017 8:22 AM, Yong Wang wrote:
> In function ark_config_device(), there are several malloc without null
> point check. Fix it by adding null point check.
> 
> Signed-off-by: Yong Wang <wang.yong19@zte.com.cn>

Hi Yong,

Thank you for the patch.

I think you have sent new version of the patch because previous one was
giving checkpatch warning, but having two patches with same title is
being confusing.

Next time would you mind sending next version as version 2 (v2) and as a
reply to previous version of the patch? So this makes clear which one is
the one to take into account.
Also adding a changelog to the commit log makes life easy. Thanks.

DPDK Contributor's Guide has more details:
http://dpdk.org/doc/guides/contributing/patches.html

Thanks,
ferruh

  reply	other threads:[~2017-09-14  9:33 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-14  7:22 Yong Wang
2017-09-14  9:33 ` Ferruh Yigit [this message]
2017-09-14  9:58   ` [dpdk-dev] 答复: " wang.yong19
  -- strict thread matches above, loose matches on Subject: below --
2017-09-14  6:59 [dpdk-dev] " Yong Wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=89a345e1-43ba-2a2a-4cd2-55e2bad1817d@intel.com \
    --to=ferruh.yigit@intel.com \
    --cc=dev@dpdk.org \
    --cc=shepard.siegel@atomicrules.com \
    --cc=wang.yong19@zte.com.cn \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).