From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-vc0-f178.google.com (mail-vc0-f178.google.com [209.85.220.178]) by dpdk.org (Postfix) with ESMTP id 72D01AFD6 for ; Tue, 29 Apr 2014 18:45:58 +0200 (CEST) Received: by mail-vc0-f178.google.com with SMTP id hu19so638685vcb.9 for ; Tue, 29 Apr 2014 09:46:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=yWrcRnOoGpWmDPcfIxMiaFvPBlKkeovLR9pM9tBODCY=; b=HgXhsBg6ksKAxkHK4OhTCSeeVHBM/sh4ZxSC4l6Yv3iKRR5xCc39jPHu0m/ayRxmXf r3otslqtip92OhCo7RTsGn3Cmc+ix3bJxj0ZTilb7IVAdhOiCkmXUp3YAvF+LoRyVbu7 DOUmTm4NECZ4geaKlK8Sttl0lPUAo/yAB5rlgVBo5w1KK+HcbFn/ytRGRqEJm1se0mvT XbqLG//fxk2Scm4A+YE40c03vjMTcAsDhaDpyN0rq3aNCoKUzfJr9paP2TweNBo4zXD0 KwFynmYLnMLIoCE8KRmIap0HMcJkmf5VAC20d61/Smu7MdiP4mbZw95YyJ7sWjAGdvc5 R3/w== MIME-Version: 1.0 X-Received: by 10.220.175.70 with SMTP id w6mr213531vcz.72.1398789962278; Tue, 29 Apr 2014 09:46:02 -0700 (PDT) Received: by 10.58.73.72 with HTTP; Tue, 29 Apr 2014 09:46:02 -0700 (PDT) In-Reply-To: References: Date: Tue, 29 Apr 2014 20:16:02 +0330 Message-ID: From: Hamid Ramazani To: Dan Kan Content-Type: text/plain; charset=ISO-8859-1 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] Useful: Simple C++ program & Makefile X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 29 Apr 2014 16:45:58 -0000 Hi, this post might be a little old, but there is point which might be really important for C++ developers using DPDK: C++, especially C++11, doesn't support "designated initializes" of C, instead of that you can use "memset" and set the instance of a struct/class to 0/NULL/nullptr and then you can assign proper values to the members of that struct/class by . or -> (if it is pointer). Remember that even empty-initialization list in C++ (just a simple {} ) doesn't set things to 0/NULL/nullptr/flase (default values), and you may have garbage values which cause problems in your application. Hope this saves you from lots of headache. All the Best, Hamid On 1/11/14, Dan Kan wrote: > I forgot to add that you will also need to add -D__STDC_LIMIT_MACROS > because C99 standard specifies that limits such as INT8_MAX should only be > defined if explicitly requested. Also with g++, you can no longer use > non-trivial designated initializers which are used extensively throughout > dpdk sample apps. For example, > > static const struct rte_eth_rxconf rx_conf = { > .rx_thresh = { > .pthresh = RX_PTHRESH, > .hthresh = RX_HTHRESH, > .wthresh = RX_WTHRESH, > }, > .rx_free_thresh = 32, > }; > > > Basically, use gcc to build the dpdk libraries. Then, use g++ to build your > own code and link against the built libraries. > > Dan > > > On Fri, Jan 10, 2014 at 12:19 PM, Dan Kan wrote: > >> Hamid, >> I'm in the same situation as you in which I would like to write most of >> the application logic in C++. I was able to use CC=g++ by slightly >> modifying the makefiles in mk. I replaced all occurrences of "%.c" with >> "%.cc" using the following command. >> >> find . -name "*.mk" -exec sed -i 's/%\.c\([^[:alpha:]]\)/%\.cc\1/g' {} \; >> >> I also remove some warning flags as errors. >> >> Here are the diffs: >> >> diff -r mk/internal/rte.compile-pre.mk ../temp/dpdk-1.5.1r2/mk/internal/ >> rte.compile-pre.mk >> 39c39 >> < src2obj = $(strip $(patsubst %.cc,%.o,\ >> --- >> > src2obj = $(strip $(patsubst %.c,%.o,\ >> 48c48 >> < src2dep = $(strip $(call dotfile,$(patsubst %.cc,%.o.d, \ >> --- >> > src2dep = $(strip $(call dotfile,$(patsubst %.c,%.o.d, \ >> 53c53 >> < src2cmd = $(strip $(call dotfile,$(patsubst %.cc,%.o.cmd, \ >> --- >> > src2cmd = $(strip $(call dotfile,$(patsubst %.c,%.o.cmd, \ >> 125c125 >> < %.o: %.cc $$(wildcard $$(dep_$$@)) $$(DEP_$$(@)) FORCE >> --- >> > %.o: %.c $$(wildcard $$(dep_$$@)) $$(DEP_$$(@)) FORCE >> diff -r mk/rte.module.mk ../temp/dpdk-1.5.1r2/mk/rte.module.mk >> 36,37c36,37 >> < ifneq ($(MODULE),$(notdir $(SRCS-y:%.cc=%))) >> < $(MODULE)-objs += $(notdir $(SRCS-y:%.cc=%.o)) >> --- >> > ifneq ($(MODULE),$(notdir $(SRCS-y:%.c=%))) >> > $(MODULE)-objs += $(notdir $(SRCS-y:%.c=%.o)) >> diff -r >> mk/toolchain/gcc/rte.vars.mk../temp/dpdk-1.5.1r2/mk/toolchain/gcc/ >> rte.vars.mk >> 71,73c71,73 >> < WERROR_FLAGS := -W -Wall -Werror >> < WERROR_FLAGS += -Wmissing-declarations -Wpointer-arith >> < WERROR_FLAGS += -Wcast-align -Wcast-qual >> --- >> > WERROR_FLAGS := -W -Wall -Werror -Wstrict-prototypes >> > -Wmissing-prototypes >> > WERROR_FLAGS += -Wmissing-declarations -Wold-style-definition >> -Wpointer-arith >> > WERROR_FLAGS += -Wcast-align -Wnested-externs -Wcast-qual >> >> >> Dan >> >> >> On Thu, Jan 9, 2014 at 10:30 PM, Hamid Ramazani >> wrote: >> >>> > I don't exactly know what is needed for C++. Please keep us informed. >>> >>> Hey Thomas, >>> >>> I've attached a simple program (main.cpp main.h and Makefile) that has >>> a C++ class and just prints some messages in the output. >>> Despite the fact that it's working fine, I'm sure the Makefile could >>> be written much better; Maybe I made it completer in future. >>> >>> All the Best, >>> --Hamid >>> >>> On 1/3/14, Thomas Monjalon wrote: >>> > Hello, >>> > >>> > 03/01/2014 11:48, Hamid Ramazani : >>> >> eal_timer.c:(.text+0x42c): undefined reference to `clock_gettime' >>> > >>> > From "man clock_gettime": >>> > Link with -lrt (only for glibc versions before 2.17). >>> > >>> >> g++ -m64 -pthread -march=native -DRTE_MACHINE_CPUFLAG_SSE >>> >> -DRTE_MACHINE_CPUFLAG_SSE2 -DRTE_MACHINE_CPUFLAG_SSSE3 >>> >> >>> -DRTE_COMPILE_TIME_CPUFLAGS=RTE_CPUFLAG_SSE,RTE_CPUFLAG_SSE2,RTE_CPUFLAG_SS >>> >> SE3 -I/home/hamid/dpdk/dpdk-1.5.1r1/examples/sample/build/include >>> >> -I/home/hamid/dpdk/dpdk-1.5.1r1/x86_64-default-linuxapp-gcc/include >>> >> -include >>> >> >>> /home/hamid/dpdk/dpdk-1.5.1r1/x86_64-default-linuxapp-gcc/include/rte_conf >>> >> ig.h -O3 -W -Wall -Werror -Wmissing-declarations -Wpointer-arith >>> >> -Wcast-align -Wcast-qual -Wformat-nonliteral -Wformat-security >>> >> -Wundef >>> >> -Wwrite-strings -Wl,-melf_x86_64 -Wl,-export-dynamic sample.cpp -o >>> >> sample -Wl,-L/home/hamid/dpdk/dpdk-1.5.1r1/examples/sample/build/lib >>> >> -Wl,-L/home/hamid/dpdk/dpdk-1.5.1r1/x86_64-default-linuxapp-gcc/lib >>> >> -Wl,-L/home/hamid/dpdk/dpdk-1.5.1r1/x86_64-default-linuxapp-gcc/lib >>> >> -Wl,-lrte_kni -Wl,-lrte_pmd_e1000 -Wl,-lrte_pmd_ixgbe -Wl,-lrte_mbuf >>> >> -Wl,-lrte_cmdline -Wl,-lrte_timer -Wl,-lrte_hash -Wl,-lrte_lpm >>> >> -Wl,--start-group -Wl,-lethdev -Wl,-lrte_malloc -Wl,-lrte_mempool >>> >> -Wl,-lrte_ring -Wl,-lrte_eal -Wl,-ldl -Wl,--end-group >>> > >>> > Try CONFIG_RTE_BUILD_COMBINE_LIBS=y and -lintel_dpdk instead of all >>> these >>> > libraries. You can also remove the warning options if you want. >>> > >>> > You can also try to build your Makefile by including files like >>> > mk/rte.extapp.mk and defining CC=g++. >>> > I don't exactly know what is needed for C++. Please keep us informed. >>> > >>> > -- >>> > Thomas >>> > >>> >> >> >