From mboxrd@z Thu Jan 1 00:00:00 1970 From: Samuel Monderer Subject: Re: could not l2fwd in DOM0 Date: Wed, 7 May 2014 12:38:20 +0000 Message-ID: <66b6b200a90f4dd2ad768bff985f1bac@MUC-SRV-MBX2.advaoptical.com> References: <7b60de2c2b904fe78a5ab2f73c08955f@MUC-SRV-MBX2.advaoptical.com> <1ED644BD7E0A5F4091CF203DAFB8E4CC01C6C07E@SHSMSX101.ccr.corp.intel.com> <9f34498d15cd400eb2db76212778c038@MUC-SRV-MBX2.advaoptical.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <9f34498d15cd400eb2db76212778c038@MUC-SRV-MBX2.advaoptical.com> Content-Language: en-US List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Liu, Jijiang" Cc: "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Liu, Jijiang > Sent: Wednesday, May 07, 2014 7:36 AM > To: dev@dpdk.org > Subject: Re: [dpdk-dev] could not l2fwd in DOM0 > = > Hi, > = > I have checked source codes of xen_create_contiguous_region function in > kernel 3.14, and found the dma_handle cannot be NULL. > = > int xen_create_contiguous_region(phys_addr_t pstart, unsigned int order, > unsigned int address_bits, > dma_addr_t *dma_handle) { > unsigned long *in_frames =3D discontig_frames, out_frame; > unsigned long flags; > int success; > unsigned long vstart =3D (unsigned long)phys_to_virt(pstart); > ... > *dma_handle =3D virt_to_machine(vstart).maddr; > return success ? 0 : -ENOMEM; > } > = > Thanks > Frank Liu > = Thanks Frank, I've changed the code as following but now the kernel module crashes. diff --git a/lib/librte_eal/linuxapp/xen_dom0/dom0_mm_misc.c=A0 b/lib/librte_eal/linuxapp/xen index 87fa3e6..8addc21 100644 --- a/lib/librte_eal/linuxapp/xen_dom0/dom0_mm_misc.c +++ b/lib/librte_eal/linuxapp/xen_dom0/dom0_mm_misc.c @@ -64,6 +64,7 @@ =A0#include =A0#include =A0#include =A0#include =A0#include @@ -309,6 +310,7 @@ dom0_prepare_memsegs(struct memory_info* meminfo, struc= t=A0 dom0_mm_data =A0=A0=A0=A0=A0=A0=A0=A0uint64_t pfn, vstart, vaddr; =A0=A0=A0=A0=A0=A0=A0=A0uint32_t i, num_block, size; =A0=A0=A0=A0=A0=A0=A0=A0int idx; +=A0=A0=A0=A0=A0=A0=A0dma_addr_t dma_handle; =A0=A0=A0=A0=A0=A0=A0=A0/* Allocate 2M memory once */ =A0=A0=A0=A0=A0=A0=A0=A0num_block =3D meminfo->size / 2; @@ -344,7 +346,7 @@ dom0_prepare_memsegs(struct memory_info* meminfo, struc= t=A0 dom0_mm_data =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0* contiguous physical ad= dresses, its maximum size is 2M. =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0*/ =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0if (xen_create_contiguous_r= egion(mm_data->block_info[i].vir_addr, -=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0= =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0DOM0_CONTIG_NUM_ORDER, 0) =3D=3D 0) { +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0= =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0DOM0_CONTIG_NUM_ORDER, 0, &dma_handle) =3D= =3D 0) { =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0mm_= data->block_info[i].exchange_flag =3D 1; =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0mm_= data->block_info[i].mfn =3D =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0= =A0=A0=A0=A0=A0=A0=A0pfn_to_mfn(mm_data->block_info[i].pfn); Samuel > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Samuel Monderer > Sent: Wednesday, April 30, 2014 1:54 AM > To: dev@dpdk.org > Cc: Shimon Zadok > Subject: [dpdk-dev] could not l2fwd in DOM0 > = > Hi, > = > First I've encountered a compiling problem when compiling for DOM0 due to > prototype change of the function xen_create_contiguous_region I made the > following changes: > = > diff --git a/lib/librte_eal/linuxapp/xen_dom0/dom0_mm_misc.c > b/lib/librte_eal/linuxapp/xen index 87fa3e6..8addc21 100644 > --- a/lib/librte_eal/linuxapp/xen_dom0/dom0_mm_misc.c > +++ b/lib/librte_eal/linuxapp/xen_dom0/dom0_mm_misc.c > @@ -64,6 +64,7 @@ > #include > #include > #include > +//#include > = > #include > #include > @@ -309,6 +310,7 @@ dom0_prepare_memsegs(struct memory_info* > meminfo, struct dom0_mm_data > uint64_t pfn, vstart, vaddr; > uint32_t i, num_block, size; > int idx; > + dma_addr_t *dma_handle =3D NULL; > = > /* Allocate 2M memory once */ > num_block =3D meminfo->size / 2; > @@ -344,7 +346,7 @@ dom0_prepare_memsegs(struct memory_info* > meminfo, struct dom0_mm_data > * contiguous physical addresses, its maximum size is 2M. > */ > if (xen_create_contiguous_region(mm_data->block_info[i].v= ir_addr, > - DOM0_CONTIG_NUM_ORDER, 0) =3D=3D 0) { > + DOM0_CONTIG_NUM_ORDER, 0, > + dma_handle) =3D=3D 0) { > mm_data->block_info[i].exchange_flag =3D 1; > mm_data->block_info[i].mfn =3D > pfn_to_mfn(mm_data->block_info[i].pfn); > = > After that I tried to run l2fwd example and got a segmentation fault > = > root@Smart:~/samuelm/dpdk/dpdk# modprobe uio > root@Smart:~/samuelm/dpdk/dpdk# insmod ./x86_64-default-linuxapp- > gcc/kmod/igb_uio.ko > root@Smart:~/samuelm/dpdk/dpdk# insmod ./x86_64-default-linuxapp- > gcc/kmod/rte_dom0_mm.ko > root@Smart:~/samuelm/dpdk/dpdk# cd examples/l2fwd/build/ > root@Smart:~/samuelm/dpdk/dpdk/examples/l2fwd/build# echo 2048 > > /sys/kernel/mm/dom0-mm/memsize-mB/memsize > root@Smart:~/samuelm/dpdk/dpdk/examples/l2fwd/build# ./l2fwd -c 3 -n 4 > --xen-dom0 -- -q 1 -p 3 > EAL: Detected lcore 0 as core 0 on socket 0 > EAL: Detected lcore 1 as core 0 on socket 0 > EAL: Setting up memory... > Segmentation fault > root@Smart:~/samuelm/dpdk/dpdk/examples/l2fwd/build# > = > Has anyone already encountered this problem? > = > Samuelm