Re: [dpdk-dev] [PATCH 2/5] cxgbe: add cxgbe poll mode driver.

[Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>]

On Tue, May 26, 2015 at 10:24:37 -0700, Stephen Hemminger wrote:
> On Tue, 26 May 2015 22:32:07 +0530
> Rahul Lakkireddy <rahul.lakkireddy@chelsio.com> wrote:
> 
> > On Sat, May 23, 2015 at 11:27:56 +0530, Rahul Lakkireddy wrote:
> > > On Fri, May 22, 2015 at 09:42:50 -0700, Stephen Hemminger wrote:
> > > > On Fri, 22 May 2015 18:54:20 +0530
> > > > Rahul Lakkireddy <rahul.lakkireddy@chelsio.com> wrote:
> > > > 
[...]

> > > > 
> > > > This is not Linux kernel.
> > > > Please don't create wrappers for all the stuff in Linux to port your driver.
> > > 
> > > We actually referred several PMD's compat file including - enic_compat.h,
> > > i40e_osdep.h, ixgbe_osdep.h, fm10k_osdep.h, etc.
> > > 
> > > Most of the types above are already defined by many of existing PMD's compat
> > > file.  Can we at-least keep those which are already defined by several PMD's
> > > compat file?
> > 
> > Just to give a background - since we are new to dpdk community, we studied the
> > already merged PMD's compat files as reference to understand how things are
> > done for driver submission. And so, we wrote cxgbe compat file along similar
> > lines. However, if above wrappers are not acceptable then, we will definitely
> > remove them in V2.
> > 
> > Just trying to get a clarification so that we don't repeat the same mistake in
> > V2 submission. Reviews from you and dpdk community are more than welcome and
> > appreciated.
> 
> Does this driver share source code with other platforms?

Yes. The h/w specific code is common to Linux and FBSD.
And we will be enabling FBSD support soon after this PMD gets merged and we are
able to carry out and complete testing on FBSD.


> If it does then the compatibility wrappers make sense and reduce the
> maintenance effort.
> If the driver is a standalone port to DPDK, then it makes sense to complete
> the effort and use standard DPDK coding practices (stdint, stdbool, etc).
> 
> The other drivers in DPDK do things based on that. Many of the hardware
> drivers share code with BSD. Others like the virtual drivers were written
> or ported completely from scratch.
>

Thank you for your guidance.