From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2 0/2] 10G PMD: vectorized RX and TX functions Date: Thu, 12 Jun 2014 00:33:51 +0200 Message-ID: <5836928.Hv09yK42VY@xps13> References: <1399888587-21068-1-git-send-email-bruce.richardson@intel.com> <3079600.trYA8PKFWl@xps13> <59AF69C657FD0841A61C55336867B5B01AA35278@IRSMSX103.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: "Richardson, Bruce" Return-path: In-Reply-To: <59AF69C657FD0841A61C55336867B5B01AA35278-kPTMFJFq+rELt2AQoY/u9bfspsVTdybXVpNB7YpNyf8@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 2014-06-11 22:27, Richardson, Bruce: > > From: Thomas Monjalon [mailto:thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org] > > You are introducing a new option CONFIG_RTE_IXGBE_RX_OLFLAGS_DISABLE. > > I feel a comment in the code would be helpful here. > > Comment in the code, or in the config file where the option is defined? There are several config files so I think it's more appropriated in the code. And it's not in DPDK habit to add comments in config files. We are used to grep in code ;) -- Thomas