From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH 01/10] examples/tep_termination:initialize the VXLAN sample Date: Fri, 15 May 2015 16:55:48 -0700 Message-ID: <20150515165548.4e88b744@urahara> References: <1431670141-7835-1-git-send-email-jijiang.liu@intel.com> <1431670141-7835-2-git-send-email-jijiang.liu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Jijiang Liu Return-path: Received: from mail-ig0-f174.google.com (mail-ig0-f174.google.com [209.85.213.174]) by dpdk.org (Postfix) with ESMTP id E99B75697 for ; Sat, 16 May 2015 01:55:46 +0200 (CEST) Received: by igbyr2 with SMTP id yr2so10058737igb.0 for ; Fri, 15 May 2015 16:55:46 -0700 (PDT) In-Reply-To: <1431670141-7835-2-git-send-email-jijiang.liu@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, 15 May 2015 14:08:52 +0800 Jijiang Liu wrote: > + while (dev_ll != NULL) { > + /*get virtio device ID*/ Really minor style nit. Please put whitespace in comments. Do this instead. /* get virtio device ID */ Also, the name virtio is confusing since it can be confused with KVM virtio.