From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from indelg01.tcs.com (indelg01.tcs.com [203.200.109.55]) by dpdk.org (Postfix) with ESMTP id ACAB85952 for ; Tue, 10 Mar 2015 06:17:44 +0100 (CET) X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: A2AxCgAoff5U/0YgFaxchDK9FItuAoF+AQEBAQEBfIQPAQEBBHkQBQYNBAMBAi8CHy4IBgsIEYgKA6VeAQGXMA2FQAEBAQEBAQEBAQEBAQEBAQEBAQEBAReLF4JEgg8KDAUHBoMRgRYFimiDJYNngS6ERTOCYhGDF4kfglGDQoIiH4FYZ4JDAQEB X-IPAS-Result: A2AxCgAoff5U/0YgFaxchDK9FItuAoF+AQEBAQEBfIQPAQEBBHkQBQYNBAMBAi8CHy4IBgsIEYgKA6VeAQGXMA2FQAEBAQEBAQEBAQEBAQEBAQEBAQEBAReLF4JEgg8KDAUHBoMRgRYFimiDJYNngS6ERTOCYhGDF4kfglGDQoIiH4FYZ4JDAQEB X-IronPort-AV: E=Sophos;i="5.11,372,1422901800"; d="xml'?rdf'?odt'72,48?png'72,48,150?scan'72,48,150,72,48,208,150"; a="61730545" X-DISCLAIMER: FALSE MIME-Version: 1.0 Importance: Normal X-Priority: 3 (Normal) In-Reply-To: References: , From: Arkajit Ghosh To: Ngo Doan Lap Message-ID: Date: Tue, 10 Mar 2015 10:47:39 +0530 X-Mailer: Lotus Domino Web Server Release 9.0.1FP2HF609 December 16, 2014 X-MIMETrack: Serialize by Notes Server on InDelM04/TCS(Release 9.0.1FP2HF609 | December 16, 2014) at 03/10/2015 10:47:39, Serialize complete at 03/10/2015 10:47:40, Itemize by Notes Server on InDelM04/TCS(Release 9.0.1FP2HF609 | December 16, 2014) at 03/10/2015 10:47:40, Serialize by Router on InDelM04/TCS(Release 9.0.1FP2HF609 | December 16, 2014) at 03/10/2015 10:47:41 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: dev@dpdk.org Subject: Re: [dpdk-dev] #PERSONAL# Guest Machine is not Pingable from Host Machine 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, 10 Mar 2015 05:17:46 -0000 =20 Hi, Please find the attachement. Thanks & Regards Arkajit Ghosh =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F= =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F -----Ngo Doan Lap wrote: ----- To: Arkajit Ghosh From: Ngo Doan Lap Date: 03/10/2015 10:25AM Cc: dev@dpdk.org Subject: Re: #PERSONAL#[dpdk-dev] Guest Machine is not Pingable from Host M= achine Hi=A0Arkajit, I can't find your attachment. On Tue, Mar 10, 2015 at 10:23 AM, Arkajit Ghosh wro= te: =20 Hi, Can any one please suggest how to proceed now.=20 Thanks & Regards Arkajit Ghosh =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F= =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F -----Arkajit Ghosh/DEL/TCS wrote: ----- To: Ngo Doan Lap From: Arkajit Ghosh/DEL/TCS Date: 03/09/2015 09:39AM Cc: dev@dpdk.org Subject: Re: [dpdk-dev] Guest Machine is not Pingable from Host Machine Hi,=20 I have executed below commands: ./utilities/ovs-vsctl add-br ovsbr0 ./utilities/ovs-vsctl set bridge datapath=5Ftype=3Dnetdev ./utilities/ovs-vsctl add-port ovsbr0 dpdk0 -- set Interface dpdk0 type=3Dd= pdk ./utilities/ovs-vsctl add-port ovsbr0 dpdk1 -- set Interface dpdk1 type=3Dd= pdk ./utilities/ovs-vsctl show Please find the attachement for bridge and dpdk-port creation.. Now, If i do "ifconfig -a" the created bridge ovsbr0 is showing Broadcast R= unning and Multicast but ip address was not given. So i had assign an IP-ad= dress to ovsbr0 which previously Eth1 had.=20 So, now I try to ping this guest machine ip address from its host.=A0 Here = is the output.=20 ping 192.168.1.7 PING 192.168.1.7 (192.168.1.7) 56(84) bytes of data. >>From 192.168.1.40 icmp=5Fseq=3D1 Destination Host Unreachable >>From 192.168.1.40 icmp=5Fseq=3D5 Destination Host Unreachable >>From 192.168.1.40 icmp=5Fseq=3D6 Destination Host Unreachable >>From 192.168.1.40 icmp=5Fseq=3D7 Destination Host Unreachable >>From 192.168.1.40 icmp=5Fseq=3D8 Destination Host Unreachable >>From 192.168.1.40 icmp=5Fseq=3D9 Destination Host Unreachable I believe it should be pinged from its host machine. Can please let=A0 me k= now what is the issue and how to proceed. Thanks & Regards Arkajit Ghosh =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F= =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F -----Ngo Doan Lap wrote: ----- To: Arkajit Ghosh From: Ngo Doan Lap Date: 03/06/2015 08:52PM Cc: dev@dpdk.org Subject: Re: [dpdk-dev] Guest Machine is not Pingable from Host Machine Hi, Can you show the log output of OVS? and the ouput of following command cd path/to/ovs/utilities/ =A0./ovs-ofctl show br0 =A0./ovs-ofctl dump-flows br0 (Assum that br0 is a bridge name that you had created) On Fri, Mar 6, 2015 at 10:14 PM, Arkajit Ghosh wrot= e: Hi, =20 Can anyone please suggest how to proceed. =20 =20 Thanks & Regards Arkajit Ghosh =20 -----Arkajit Ghosh/DEL/TCS wrote: ----- To: dev@dpdk.org From: Arkajit Ghosh/DEL/TCS Date: 03/04/2015 12:39PM Subject: Guest Machine is not Pingable from Host Machine =20 Hi Team, =20 Guest machine is not pingable from Host machine after creating a bridge wi= th datapath=5Ftype "netdev" in the configuration database and adding=A0 dpd= k ports. Can anyone please let me know what is the issue. =20 Thanks in advance. =20 Thanks & Regards Arkajit Ghosh =20 =20 =3D=3D=3D=3D=3D-----=3D=3D=3D=3D=3D-----=3D=3D=3D=3D=3D Notice: The information contained in this e-mail message and/or attachments to it may contain confidential or privileged information. If you are not the intended recipient, any dissemination, use, review, distribution, printing or copying of the information contained in this e-mail message and/or attachments to it are strictly prohibited. If you have received this communication in error, please notify us by reply e-mail or telephone and immediately and permanently delete the message and any attachments. Thank you =20 =20 =20 --=20 Thanks and Best Regards, Ngo Doan Lap Mobile: 0977.833.757 =20 [attachment "DPDK=5FOutput=5FResult.odt" removed by Arkajit Ghosh/DEL/TCS] = --=20 Thanks and Best Regards, Ngo Doan Lap Mobile: 0977.833.757 =20 >From jijiang.liu@intel.com Tue Mar 10 08:03:19 2015 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 6C43B58E4 for ; Tue, 10 Mar 2015 08:03:18 +0100 (CET) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga103.jf.intel.com with ESMTP; 10 Mar 2015 00:00:28 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,373,1422950400"; d="scan'208";a="662921048" Received: from kmsmsx151.gar.corp.intel.com ([172.21.73.86]) by orsmga001.jf.intel.com with ESMTP; 10 Mar 2015 00:03:15 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by KMSMSX151.gar.corp.intel.com (172.21.73.86) with Microsoft SMTP Server (TLS) id 14.3.195.1; Tue, 10 Mar 2015 15:02:33 +0800 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.150]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.45]) with mapi id 14.03.0224.002; Tue, 10 Mar 2015 15:02:32 +0800 From: "Liu, Jijiang" To: Stephen Hemminger , "dev@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH 1/5] xen: allow choosing dom0 support at runtime Thread-Index: AQHQSTOswcupOJhAwU2665gohh/CS50VbY0A Date: Tue, 10 Mar 2015 07:02:31 +0000 Message-ID: <1ED644BD7E0A5F4091CF203DAFB8E4CC01DF0E01@SHSMSX101.ccr.corp.intel.com> References: <1424013889-2226-1-git-send-email-shemming@brocade.com> In-Reply-To: <1424013889-2226-1-git-send-email-shemming@brocade.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: Stephen Hemminger Subject: Re: [dpdk-dev] [PATCH 1/5] xen: allow choosing dom0 support at runtime 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, 10 Mar 2015 07:03:19 -0000 > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Stephen Hemminger > Sent: Sunday, February 15, 2015 11:25 PM > To: dev@dpdk.org > Cc: Stephen Hemminger > Subject: [dpdk-dev] [PATCH 1/5] xen: allow choosing dom0 support at runti= me >=20 > The previous code would only allow building library and application so th= at it > ran on Xen DOM0 or not on DOM0. This changes that to a runtime flag. >=20 > Signed-off-by: Stephen Hemminger > --- > v2 -- fix i40e as well >=20 > lib/librte_eal/common/include/rte_memory.h | 4 +++ > lib/librte_eal/linuxapp/eal/eal_memory.c | 7 ++++ > lib/librte_ether/rte_ethdev.c | 22 ++++++++++++ > lib/librte_ether/rte_ethdev.h | 23 ++++++++++++ > lib/librte_mempool/rte_mempool.c | 26 +++++++------- > lib/librte_pmd_e1000/em_rxtx.c | 30 +++------------- > lib/librte_pmd_e1000/igb_rxtx.c | 52 +++++++++---------------= --- > lib/librte_pmd_i40e/i40e_ethdev.c | 16 +++++---- > lib/librte_pmd_i40e/i40e_fdir.c | 8 +++-- > lib/librte_pmd_i40e/i40e_rxtx.c | 57 +++++++++++++++++-------= ----- > lib/librte_pmd_ixgbe/ixgbe_rxtx.c | 58 +++++++++---------------= ------ > 11 files changed, 156 insertions(+), 147 deletions(-) >=20 > diff --git a/lib/librte_eal/common/include/rte_memory.h > b/lib/librte_eal/common/include/rte_memory.h > index 7f8103f..ab6c1ff 100644 > --- a/lib/librte_eal/common/include/rte_memory.h > +++ b/lib/librte_eal/common/include/rte_memory.h > @@ -176,6 +176,10 @@ unsigned rte_memory_get_nchannel(void); unsigned > rte_memory_get_nrank(void); >=20 > #ifdef RTE_LIBRTE_XEN_DOM0 > + > +/**< Internal use only - should DOM0 memory mapping be used */ extern > +int is_xen_dom0_supported(void); > + > /** > * Return the physical address of elt, which is an element of the pool m= p. > * > diff --git a/lib/librte_eal/linuxapp/eal/eal_memory.c > b/lib/librte_eal/linuxapp/eal/eal_memory.c > index a67a1b0..4afda2a 100644 > --- a/lib/librte_eal/linuxapp/eal/eal_memory.c > +++ b/lib/librte_eal/linuxapp/eal/eal_memory.c > @@ -98,6 +98,13 @@ > #include "eal_filesystem.h" > #include "eal_hugepages.h" >=20 > +#ifdef RTE_LIBRTE_XEN_DOM0 > +int is_xen_dom0_supported(void) > +{ > + return internal_config.xen_dom0_support; } #endif > + > /** > * @file > * Huge page mapping under linux > diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.= c index > ea3a1fb..457e0bc 100644 > --- a/lib/librte_ether/rte_ethdev.c > +++ b/lib/librte_ether/rte_ethdev.c > @@ -2825,6 +2825,27 @@ _rte_eth_dev_callback_process(struct > rte_eth_dev *dev, > } > rte_spinlock_unlock(&rte_eth_dev_cb_lock); > } > + > +const struct rte_memzone * > +rte_eth_dma_zone_reserve(const struct rte_eth_dev *dev, const char > *ring_name, > + uint16_t queue_id, size_t size, unsigned align, > + int socket_id) > +{ > + char z_name[RTE_MEMZONE_NAMESIZE]; > + const struct rte_memzone *mz; > + > + snprintf(z_name, sizeof(z_name), "%s_%s_%d_%d", > + dev->driver->pci_drv.name, ring_name, > + dev->data->port_id, queue_id); > + > + mz =3D rte_memzone_lookup(z_name); > + if (mz) > + return mz; > + > + return rte_memzone_reserve_bounded(z_name, size, > + socket_id, 0, align, > RTE_PGSIZE_2M); } > + > #ifdef RTE_NIC_BYPASS > int rte_eth_dev_bypass_init(uint8_t port_id) { @@ -3003,6 +3024,7 @@ > rte_eth_dev_bypass_wd_reset(uint8_t port_id) > (*dev->dev_ops->bypass_wd_reset)(dev); > return 0; > } > + > #endif >=20 > int > diff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.= h index > 1200c1c..747acb5 100644 > --- a/lib/librte_ether/rte_ethdev.h > +++ b/lib/librte_ether/rte_ethdev.h > @@ -3664,6 +3664,29 @@ int rte_eth_dev_filter_supported(uint8_t port_id, > enum rte_filter_type filter_ty int rte_eth_dev_filter_ctrl(uint8_t port_= id, enum > rte_filter_type filter_type, > enum rte_filter_op filter_op, void *arg); >=20 > +/** > + * Create memzone for HW rings. > + * malloc can't be used as the physical address is needed. > + * If the memzone is already created, then this function returns a ptr > + * to the old one. > + * > + * @param eth_dev > + * The *eth_dev* pointer is the address of the *rte_eth_dev* structure > + * @param name > + * The name of the memory zone > + * @param queue_id > + * The index of the queue to add to name > + * @param size > + * The sizeof of the memory area > + * @param align > + * Alignment for resulting memzone. Must be a power of 2. > + * @param socket_id > + * The *socket_id* argument is the socket identifier in case of NUMA. > + */ > +const struct rte_memzone * > +rte_eth_dma_zone_reserve(const struct rte_eth_dev *eth_dev, const char > *name, > + uint16_t queue_id, size_t size, > + unsigned align, int socket_id); > #ifdef __cplusplus > } > #endif > diff --git a/lib/librte_mempool/rte_mempool.c > b/lib/librte_mempool/rte_mempool.c > index 4cf6c25..5056a4f 100644 > --- a/lib/librte_mempool/rte_mempool.c > +++ b/lib/librte_mempool/rte_mempool.c > @@ -372,19 +372,21 @@ rte_mempool_create(const char *name, unsigned n, > unsigned elt_size, > int socket_id, unsigned flags) > { > #ifdef RTE_LIBRTE_XEN_DOM0 > - return (rte_dom0_mempool_create(name, n, elt_size, > - cache_size, private_data_size, > - mp_init, mp_init_arg, > - obj_init, obj_init_arg, > - socket_id, flags)); > -#else > - return (rte_mempool_xmem_create(name, n, elt_size, > - cache_size, private_data_size, > - mp_init, mp_init_arg, > - obj_init, obj_init_arg, > - socket_id, flags, > - NULL, NULL, MEMPOOL_PG_NUM_DEFAULT, > MEMPOOL_PG_SHIFT_MAX)); > + if (is_xen_dom0_supported()) > + return (rte_dom0_mempool_create(name, n, elt_size, > + cache_size, private_data_size, > + mp_init, mp_init_arg, > + obj_init, obj_init_arg, > + socket_id, flags)); > + else > #endif > + return (rte_mempool_xmem_create(name, n, elt_size, > + cache_size, private_data_size, > + mp_init, mp_init_arg, > + obj_init, obj_init_arg, > + socket_id, flags, > + NULL, NULL, > MEMPOOL_PG_NUM_DEFAULT, > + MEMPOOL_PG_SHIFT_MAX)); > } >=20 > /* > diff --git a/lib/librte_pmd_e1000/em_rxtx.c b/lib/librte_pmd_e1000/em_rxt= x.c > index aa0b88c..9e09cfa 100644 > --- a/lib/librte_pmd_e1000/em_rxtx.c > +++ b/lib/librte_pmd_e1000/em_rxtx.c > @@ -1104,28 +1104,6 @@ eth_em_recv_scattered_pkts(void *rx_queue, > struct rte_mbuf **rx_pkts, > #define EM_MAX_BUF_SIZE 16384 > #define EM_RCTL_FLXBUF_STEP 1024 >=20 > -static const struct rte_memzone * > -ring_dma_zone_reserve(struct rte_eth_dev *dev, const char *ring_name, > - uint16_t queue_id, uint32_t ring_size, int socket_id) > -{ > - const struct rte_memzone *mz; > - char z_name[RTE_MEMZONE_NAMESIZE]; > - > - snprintf(z_name, sizeof(z_name), "%s_%s_%d_%d", > - dev->driver->pci_drv.name, ring_name, dev->data->port_id, > - queue_id); > - > - if ((mz =3D rte_memzone_lookup(z_name)) !=3D 0) > - return (mz); > - > -#ifdef RTE_LIBRTE_XEN_DOM0 > - return rte_memzone_reserve_bounded(z_name, ring_size, > - socket_id, 0, RTE_CACHE_LINE_SIZE, RTE_PGSIZE_2M); > -#else > - return rte_memzone_reserve(z_name, ring_size, socket_id, 0); > -#endif > -} > - > static void > em_tx_queue_release_mbufs(struct em_tx_queue *txq) { @@ -1273,8 > +1251,8 @@ eth_em_tx_queue_setup(struct rte_eth_dev *dev, > * resizing in later calls to the queue setup function. > */ > tsize =3D sizeof (txq->tx_ring[0]) * EM_MAX_RING_DESC; > - if ((tz =3D ring_dma_zone_reserve(dev, "tx_ring", queue_idx, tsize, > - socket_id)) =3D=3D NULL) > + if ((tz =3D rte_eth_dma_zone_reserve(dev, "tx_ring", queue_idx, tsize, > + RTE_CACHE_LINE_SIZE, socket_id)) > =3D=3D NULL) > return (-ENOMEM); >=20 > /* Allocate the tx queue data structure. */ @@ -1400,8 +1378,8 @@ > eth_em_rx_queue_setup(struct rte_eth_dev *dev, >=20 > /* Allocate RX ring for max possible mumber of hardware descriptors. > */ > rsize =3D sizeof (rxq->rx_ring[0]) * EM_MAX_RING_DESC; > - if ((rz =3D ring_dma_zone_reserve(dev, "rx_ring", queue_idx, rsize, > - socket_id)) =3D=3D NULL) > + if ((rz =3D rte_eth_dma_zone_reserve(dev, "rx_ring", queue_idx, rsize, > + RTE_CACHE_LINE_SIZE, socket_id)) > =3D=3D NULL) > return (-ENOMEM); >=20 > /* Allocate the RX queue data structure. */ diff --git > a/lib/librte_pmd_e1000/igb_rxtx.c b/lib/librte_pmd_e1000/igb_rxtx.c index > 5c394a9..d36469b 100644 > --- a/lib/librte_pmd_e1000/igb_rxtx.c > +++ b/lib/librte_pmd_e1000/igb_rxtx.c > @@ -1109,29 +1109,6 @@ eth_igb_recv_scattered_pkts(void *rx_queue, > struct rte_mbuf **rx_pkts, #define IGB_MIN_RING_DESC 32 #define > IGB_MAX_RING_DESC 4096 >=20 > -static const struct rte_memzone * > -ring_dma_zone_reserve(struct rte_eth_dev *dev, const char *ring_name, > - uint16_t queue_id, uint32_t ring_size, int socket_id) > -{ > - char z_name[RTE_MEMZONE_NAMESIZE]; > - const struct rte_memzone *mz; > - > - snprintf(z_name, sizeof(z_name), "%s_%s_%d_%d", > - dev->driver->pci_drv.name, ring_name, > - dev->data->port_id, queue_id); > - mz =3D rte_memzone_lookup(z_name); > - if (mz) > - return mz; > - > -#ifdef RTE_LIBRTE_XEN_DOM0 > - return rte_memzone_reserve_bounded(z_name, ring_size, > - socket_id, 0, IGB_ALIGN, RTE_PGSIZE_2M); > -#else > - return rte_memzone_reserve_aligned(z_name, ring_size, > - socket_id, 0, IGB_ALIGN); > -#endif > -} > - > static void > igb_tx_queue_release_mbufs(struct igb_tx_queue *txq) { @@ -1265,8 > +1242,8 @@ eth_igb_tx_queue_setup(struct rte_eth_dev *dev, > * resizing in later calls to the queue setup function. > */ > size =3D sizeof(union e1000_adv_tx_desc) * IGB_MAX_RING_DESC; > - tz =3D ring_dma_zone_reserve(dev, "tx_ring", queue_idx, > - size, socket_id); > + tz =3D rte_eth_dma_zone_reserve(dev, "tx_ring", queue_idx, size, > + IGB_ALIGN, socket_id); > if (tz =3D=3D NULL) { > igb_tx_queue_release(txq); > return (-ENOMEM); > @@ -1284,12 +1261,14 @@ eth_igb_tx_queue_setup(struct rte_eth_dev > *dev, > txq->port_id =3D dev->data->port_id; >=20 > txq->tdt_reg_addr =3D E1000_PCI_REG_ADDR(hw, E1000_TDT(txq- > >reg_idx)); -#ifndef RTE_LIBRTE_XEN_DOM0 > - txq->tx_ring_phys_addr =3D (uint64_t) tz->phys_addr; > -#else > - txq->tx_ring_phys_addr =3D rte_mem_phy2mch(tz->memseg_id, tz- > >phys_addr); > +#ifdef RTE_LIBRTE_XEN_DOM0 > + if (is_xen_dom0_supported()) > + txq->tx_ring_phys_addr =3D rte_mem_phy2mch(tz->memseg_id, > tz->phys_addr); > + else > #endif > - txq->tx_ring =3D (union e1000_adv_tx_desc *) tz->addr; > + txq->tx_ring_phys_addr =3D (uint64_t) tz->phys_addr; > + > + txq->tx_ring =3D (union e1000_adv_tx_desc *) tz->addr; > /* Allocate software ring */ > txq->sw_ring =3D rte_zmalloc("txq->sw_ring", > sizeof(struct igb_tx_entry) * nb_desc, @@ - > 1414,18 +1393,21 @@ eth_igb_rx_queue_setup(struct rte_eth_dev *dev, > * resizing in later calls to the queue setup function. > */ > size =3D sizeof(union e1000_adv_rx_desc) * IGB_MAX_RING_DESC; > - rz =3D ring_dma_zone_reserve(dev, "rx_ring", queue_idx, size, socket_id= ); > + rz =3D rte_eth_dma_zone_reserve(dev, "rx_ring", queue_idx, size, > + IGB_ALIGN, socket_id); > if (rz =3D=3D NULL) { > igb_rx_queue_release(rxq); > return (-ENOMEM); > } > rxq->rdt_reg_addr =3D E1000_PCI_REG_ADDR(hw, E1000_RDT(rxq- > >reg_idx)); > rxq->rdh_reg_addr =3D E1000_PCI_REG_ADDR(hw, E1000_RDH(rxq- > >reg_idx)); -#ifndef RTE_LIBRTE_XEN_DOM0 > - rxq->rx_ring_phys_addr =3D (uint64_t) rz->phys_addr; > -#else > - rxq->rx_ring_phys_addr =3D rte_mem_phy2mch(rz->memseg_id, rz- > >phys_addr); > +#ifdef RTE_LIBRTE_XEN_DOM0 > + if (is_xen_dom0_supported()) > + rxq->rx_ring_phys_addr =3D > + rte_mem_phy2mch(rz->memseg_id, rz->phys_addr); > + else > #endif > + rxq->rx_ring_phys_addr =3D (uint64_t) rz->phys_addr; > rxq->rx_ring =3D (union e1000_adv_rx_desc *) rz->addr; >=20 > /* Allocate software ring. */ > diff --git a/lib/librte_pmd_i40e/i40e_ethdev.c > b/lib/librte_pmd_i40e/i40e_ethdev.c > index 9fa6bec..44a012f 100644 > --- a/lib/librte_pmd_i40e/i40e_ethdev.c > +++ b/lib/librte_pmd_i40e/i40e_ethdev.c > @@ -1979,11 +1979,12 @@ > i40e_allocate_dma_mem_d(__attribute__((unused)) struct i40e_hw *hw, > id++; > snprintf(z_name, sizeof(z_name), "i40e_dma_%"PRIu64, id); #ifdef > RTE_LIBRTE_XEN_DOM0 > - mz =3D rte_memzone_reserve_bounded(z_name, size, 0, 0, alignment, > - RTE_PGSIZE_2M); > -#else > - mz =3D rte_memzone_reserve_aligned(z_name, size, 0, 0, alignment); > + if (is_xen_dom0_supported()) > + mz =3D rte_memzone_reserve_bounded(z_name, size, 0, 0, > alignment, > + RTE_PGSIZE_2M); > + else > #endif > + mz =3D rte_memzone_reserve_aligned(z_name, size, 0, 0, > alignment); > if (!mz) > return I40E_ERR_NO_MEMORY; >=20 > @@ -1991,10 +1992,11 @@ > i40e_allocate_dma_mem_d(__attribute__((unused)) struct i40e_hw *hw, > mem->size =3D size; > mem->va =3D mz->addr; > #ifdef RTE_LIBRTE_XEN_DOM0 > - mem->pa =3D rte_mem_phy2mch(mz->memseg_id, mz->phys_addr); > -#else > - mem->pa =3D mz->phys_addr; > + if (is_xen_dom0_supported()) > + mem->pa =3D rte_mem_phy2mch(mz->memseg_id, mz- > >phys_addr); > + else > #endif > + mem->pa =3D mz->phys_addr; >=20 > return I40E_SUCCESS; > } > diff --git a/lib/librte_pmd_i40e/i40e_fdir.c b/lib/librte_pmd_i40e/i40e_f= dir.c > index 68511c8..4c5b185 100644 > --- a/lib/librte_pmd_i40e/i40e_fdir.c > +++ b/lib/librte_pmd_i40e/i40e_fdir.c > @@ -274,10 +274,12 @@ i40e_fdir_setup(struct i40e_pf *pf) > } > pf->fdir.prg_pkt =3D mz->addr; > #ifdef RTE_LIBRTE_XEN_DOM0 > - pf->fdir.dma_addr =3D rte_mem_phy2mch(mz->memseg_id, mz- > >phys_addr); > -#else > - pf->fdir.dma_addr =3D (uint64_t)mz->phys_addr; > + if (is_xen_dom0_supported()) > + pf->fdir.dma_addr =3D rte_mem_phy2mch(mz->memseg_id, mz- > >phys_addr); > + else > #endif > + pf->fdir.dma_addr =3D (uint64_t)mz->phys_addr; > + > pf->fdir.match_counter_index =3D I40E_COUNTER_INDEX_FDIR(hw- > >pf_id); > PMD_DRV_LOG(INFO, "FDIR setup successfully, with programming > queue %u.", > vsi->base_queue); > diff --git a/lib/librte_pmd_i40e/i40e_rxtx.c b/lib/librte_pmd_i40e/i40e_r= xtx.c > index 2beae3c..bd01dc0 100644 > --- a/lib/librte_pmd_i40e/i40e_rxtx.c > +++ b/lib/librte_pmd_i40e/i40e_rxtx.c > @@ -1796,10 +1796,11 @@ i40e_dev_rx_queue_setup(struct rte_eth_dev > *dev, > memset(rz->addr, 0, ring_size); >=20 > #ifdef RTE_LIBRTE_XEN_DOM0 > - rxq->rx_ring_phys_addr =3D rte_mem_phy2mch(rz->memseg_id, rz- > >phys_addr); > -#else > - rxq->rx_ring_phys_addr =3D (uint64_t)rz->phys_addr; > + if (is_xen_dom0_supported()) > + rxq->rx_ring_phys_addr =3D rte_mem_phy2mch(rz->memseg_id, > rz->phys_addr); > + else > #endif > + rxq->rx_ring_phys_addr =3D (uint64_t)rz->phys_addr; >=20 > rxq->rx_ring =3D (union i40e_rx_desc *)rz->addr; >=20 > @@ -2079,10 +2080,11 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev > *dev, > txq->tx_deferred_start =3D tx_conf->tx_deferred_start; >=20 > #ifdef RTE_LIBRTE_XEN_DOM0 > - txq->tx_ring_phys_addr =3D rte_mem_phy2mch(tz->memseg_id, tz- > >phys_addr); > -#else > - txq->tx_ring_phys_addr =3D (uint64_t)tz->phys_addr; > + if (is_xen_dom0_supported()) > + txq->tx_ring_phys_addr =3D rte_mem_phy2mch(tz->memseg_id, > tz->phys_addr); > + else > #endif > + txq->tx_ring_phys_addr =3D (uint64_t)tz->phys_addr; > txq->tx_ring =3D (struct i40e_tx_desc *)tz->addr; >=20 > /* Allocate software ring */ > @@ -2147,12 +2149,13 @@ i40e_ring_dma_zone_reserve(struct rte_eth_dev > *dev, > return mz; >=20 > #ifdef RTE_LIBRTE_XEN_DOM0 > - return rte_memzone_reserve_bounded(z_name, ring_size, > - socket_id, 0, I40E_ALIGN, RTE_PGSIZE_2M); > -#else > - return rte_memzone_reserve_aligned(z_name, ring_size, > - socket_id, 0, I40E_ALIGN); > -#endif > + if (is_xen_dom0_supported()) > + return rte_memzone_reserve_bounded(z_name, ring_size, > + socket_id, 0, I40E_ALIGN, RTE_PGSIZE_2M); > + else > +#endif > + return rte_memzone_reserve_aligned(z_name, ring_size, > + socket_id, 0, I40E_ALIGN); > } >=20 > const struct rte_memzone * > @@ -2164,12 +2167,14 @@ i40e_memzone_reserve(const char *name, > uint32_t len, int socket_id) > if (mz) > return mz; > #ifdef RTE_LIBRTE_XEN_DOM0 > - mz =3D rte_memzone_reserve_bounded(name, len, > - socket_id, 0, I40E_ALIGN, RTE_PGSIZE_2M); > -#else > - mz =3D rte_memzone_reserve_aligned(name, len, > - socket_id, 0, I40E_ALIGN); > + if (is_xen_dom0_supported()) > + mz =3D rte_memzone_reserve_bounded(name, len, > + socket_id, 0, I40E_ALIGN, RTE_PGSIZE_2M); > + else > #endif > + mz =3D rte_memzone_reserve_aligned(name, len, > + socket_id, 0, I40E_ALIGN); > + > return mz; > } >=20 > @@ -2573,10 +2578,13 @@ i40e_fdir_setup_tx_resources(struct i40e_pf *pf) > txq->vsi =3D pf->fdir.fdir_vsi; >=20 > #ifdef RTE_LIBRTE_XEN_DOM0 > - txq->tx_ring_phys_addr =3D rte_mem_phy2mch(tz->memseg_id, tz- > >phys_addr); > -#else > - txq->tx_ring_phys_addr =3D (uint64_t)tz->phys_addr; > + if (is_xen_dom0_supported()) > + txq->tx_ring_phys_addr =3D rte_mem_phy2mch(tz->memseg_id, > + tz->phys_addr); > + else > #endif > + txq->tx_ring_phys_addr =3D (uint64_t)tz->phys_addr; > + > txq->tx_ring =3D (struct i40e_tx_desc *)tz->addr; > /* > * don't need to allocate software ring and reset for the fdir @@ - > 2633,10 +2641,13 @@ i40e_fdir_setup_rx_resources(struct i40e_pf *pf) > rxq->vsi =3D pf->fdir.fdir_vsi; >=20 > #ifdef RTE_LIBRTE_XEN_DOM0 > - rxq->rx_ring_phys_addr =3D rte_mem_phy2mch(rz->memseg_id, rz- > >phys_addr); > -#else > - rxq->rx_ring_phys_addr =3D (uint64_t)rz->phys_addr; > + if (is_xen_dom0_supported()) > + rxq->rx_ring_phys_addr =3D rte_mem_phy2mch(rz->memseg_id, > + rz->phys_addr); > + else > #endif > + rxq->rx_ring_phys_addr =3D (uint64_t)rz->phys_addr; > + > rxq->rx_ring =3D (union i40e_rx_desc *)rz->addr; >=20 > /* > diff --git a/lib/librte_pmd_ixgbe/ixgbe_rxtx.c > b/lib/librte_pmd_ixgbe/ixgbe_rxtx.c > index e6766b3..303144d 100644 > --- a/lib/librte_pmd_ixgbe/ixgbe_rxtx.c > +++ b/lib/librte_pmd_ixgbe/ixgbe_rxtx.c > @@ -1656,35 +1656,6 @@ ixgbe_recv_scattered_pkts(void *rx_queue, struct > rte_mbuf **rx_pkts, #define IXGBE_MIN_RING_DESC 32 #define > IXGBE_MAX_RING_DESC 4096 >=20 > -/* > - * Create memzone for HW rings. malloc can't be used as the physical add= ress is > - * needed. If the memzone is already created, then this function returns= a ptr > - * to the old one. > - */ > -static const struct rte_memzone * > -ring_dma_zone_reserve(struct rte_eth_dev *dev, const char *ring_name, > - uint16_t queue_id, uint32_t ring_size, int socket_id) > -{ > - char z_name[RTE_MEMZONE_NAMESIZE]; > - const struct rte_memzone *mz; > - > - snprintf(z_name, sizeof(z_name), "%s_%s_%d_%d", > - dev->driver->pci_drv.name, ring_name, > - dev->data->port_id, queue_id); > - > - mz =3D rte_memzone_lookup(z_name); > - if (mz) > - return mz; > - > -#ifdef RTE_LIBRTE_XEN_DOM0 > - return rte_memzone_reserve_bounded(z_name, ring_size, > - socket_id, 0, IXGBE_ALIGN, RTE_PGSIZE_2M); > -#else > - return rte_memzone_reserve_aligned(z_name, ring_size, > - socket_id, 0, IXGBE_ALIGN); > -#endif > -} > - > static void > ixgbe_tx_queue_release_mbufs(struct igb_tx_queue *txq) { @@ -1920,9 > +1891,9 @@ ixgbe_dev_tx_queue_setup(struct rte_eth_dev *dev, > * handle the maximum ring size is allocated in order to allow for > * resizing in later calls to the queue setup function. > */ > - tz =3D ring_dma_zone_reserve(dev, "tx_ring", queue_idx, > + tz =3D rte_eth_dma_zone_reserve(dev, "tx_ring", queue_idx, > sizeof(union ixgbe_adv_tx_desc) * > IXGBE_MAX_RING_DESC, > - socket_id); > + IXGBE_ALIGN, socket_id); > if (tz =3D=3D NULL) { > ixgbe_tx_queue_release(txq); > return (-ENOMEM); > @@ -1950,11 +1921,14 @@ ixgbe_dev_tx_queue_setup(struct rte_eth_dev > *dev, > txq->tdt_reg_addr =3D IXGBE_PCI_REG_ADDR(hw, > IXGBE_VFTDT(queue_idx)); > else > txq->tdt_reg_addr =3D IXGBE_PCI_REG_ADDR(hw, > IXGBE_TDT(txq->reg_idx)); > -#ifndef RTE_LIBRTE_XEN_DOM0 > - txq->tx_ring_phys_addr =3D (uint64_t) tz->phys_addr; > -#else > - txq->tx_ring_phys_addr =3D rte_mem_phy2mch(tz->memseg_id, tz- > >phys_addr); > + > +#ifdef RTE_LIBRTE_XEN_DOM0 > + if (is_xen_dom0_supported()) > + txq->tx_ring_phys_addr =3D > + rte_mem_phy2mch(tz->memseg_id, tz->phys_addr); > + else > #endif > + txq->tx_ring_phys_addr =3D (uint64_t) tz->phys_addr; > txq->tx_ring =3D (union ixgbe_adv_tx_desc *) tz->addr; >=20 > /* Allocate software ring */ > @@ -2195,8 +2169,8 @@ ixgbe_dev_rx_queue_setup(struct rte_eth_dev > *dev, > * handle the maximum ring size is allocated in order to allow for > * resizing in later calls to the queue setup function. > */ > - rz =3D ring_dma_zone_reserve(dev, "rx_ring", queue_idx, > - RX_RING_SZ, socket_id); > + rz =3D rte_eth_dma_zone_reserve(dev, "rx_ring", queue_idx, > + RX_RING_SZ, IXGBE_ALIGN, socket_id); > if (rz =3D=3D NULL) { > ixgbe_rx_queue_release(rxq); > return (-ENOMEM); > @@ -2223,11 +2197,13 @@ ixgbe_dev_rx_queue_setup(struct rte_eth_dev > *dev, > rxq->rdh_reg_addr =3D > IXGBE_PCI_REG_ADDR(hw, IXGBE_RDH(rxq->reg_idx)); > } > -#ifndef RTE_LIBRTE_XEN_DOM0 > - rxq->rx_ring_phys_addr =3D (uint64_t) rz->phys_addr; > -#else > - rxq->rx_ring_phys_addr =3D rte_mem_phy2mch(rz->memseg_id, rz- > >phys_addr); > +#ifdef RTE_LIBRTE_XEN_DOM0 > + if (is_xen_dom0_supported()) > + rxq->rx_ring_phys_addr =3D > + rte_mem_phy2mch(rz->memseg_id, rz->phys_addr); > + else > #endif > + rxq->rx_ring_phys_addr =3D (uint64_t) rz->phys_addr; > rxq->rx_ring =3D (union ixgbe_adv_rx_desc *) rz->addr; >=20 > /* > -- > 2.1.4 Acked-by: Jijiang Liu