From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 2A614C2FE for ; Wed, 22 Apr 2015 13:59:48 +0200 (CEST) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga102.fm.intel.com with ESMTP; 22 Apr 2015 04:59:46 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,623,1422950400"; d="scan'208";a="699115842" Received: from irsmsx103.ger.corp.intel.com ([163.33.3.157]) by fmsmga001.fm.intel.com with ESMTP; 22 Apr 2015 04:59:46 -0700 Received: from irsmsx105.ger.corp.intel.com ([169.254.7.178]) by IRSMSX103.ger.corp.intel.com ([169.254.3.76]) with mapi id 14.03.0224.002; Wed, 22 Apr 2015 12:59:40 +0100 From: "Ananyev, Konstantin" To: Olivier Matz , "dev@dpdk.org" Thread-Topic: [PATCH v6 00/13] mbuf: enhancements of mbuf clones Thread-Index: AQHQfOLUX9xM4FuzP0yKuS5qsFU5u51Y7Yug Date: Wed, 22 Apr 2015 11:59:39 +0000 Message-ID: <2601191342CEEE43887BDE71AB97725821420BD6@irsmsx105.ger.corp.intel.com> References: <1429610122-30943-1-git-send-email-olivier.matz@6wind.com> <1429696650-9043-1-git-send-email-olivier.matz@6wind.com> In-Reply-To: <1429696650-9043-1-git-send-email-olivier.matz@6wind.com> Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v6 00/13] mbuf: enhancements of mbuf clones 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: Wed, 22 Apr 2015 11:59:49 -0000 > -----Original Message----- > From: Olivier Matz [mailto:olivier.matz@6wind.com] > Sent: Wednesday, April 22, 2015 10:57 AM > To: dev@dpdk.org > Cc: Ananyev, Konstantin; zoltan.kiss@linaro.org; Richardson, Bruce; nhorm= an@tuxdriver.com; olivier.matz@6wind.com > Subject: [PATCH v6 00/13] mbuf: enhancements of mbuf clones >=20 > The first objective of this series is to fix the support of indirect > mbufs when the application reserves a private area in mbufs. It also > removes the limitation that rte_pktmbuf_clone() is only allowed on > direct (non-cloned) mbufs. The series also contains some enhancements > and fixes in the mbuf area that makes the implementation of the > last patches easier. >=20 > Changes in v6: > - restore the priv_size in mbuf structure, version 4 broke the > attachment between mbufs having different private size > - add a test case to ensure it won't be broken again > - replace 0xffff by UINT16_MAX > - fix some minor checkpatch issues >=20 > Changes in v5: > - update rte_mbuf_version.map to fix compilation with shared libraries >=20 > Changes in v4: > - do not add a priv_size in mbuf structure, having a proper accessor > to read it from the pool private area is clearer > - prepend some reworks in the mbuf area to simplify the implementation > (fix mbuf initialization by not using a hardcoded mbuf size, add > accessors for mbuf pool private area, add a helper to create a > mbuf pool) >=20 > Changes in v3: > - a mbuf can now attach to another one that have a different private > size. In this case, the m->priv_size corresponds to the size of the > private area of the direct mbuf. > - add comments to reflect these changes > - minor style modifications >=20 > Changes in v2: > - do not change the use of MBUF_EXT_MEM() in vhost > - change rte_mbuf_from_baddr() to rte_mbuf_from_indirect(), removing > one parameter > - fix and rework rte_pktmbuf_detach() > - move m->priv_size in second mbuf cache line > - fix mbuf free in test error case >=20 > Olivier Matz (13): > mbuf: fix mbuf data room size calculation rte_pktmbuf_pool_init > examples: always initialize mbuf_pool private area > mbuf: add accessors to get data room size and private size > mbuf: fix rte_pktmbuf_init when mbuf private size is not zero > testpmd: use standard functions to initialize mbufs and mbuf pool > mbuf: introduce a new helper to create a mbuf pool > apps: use rte_pktmbuf_pool_create to create mbuf pools > mbuf: fix clone support when application uses private mbuf data > mbuf: allow to clone an indirect mbuf > test/mbuf: rename mc variable in m > test/mbuf: enhance mbuf refcnt test > test/mbuf: verify that cloning a clone works properly > test/mbuf: add a test case for clone with different priv size >=20 > app/test-pipeline/init.c | 15 +- > app/test-pmd/testpmd.c | 84 +------- > app/test/test_distributor.c | 10 +- > app/test/test_distributor_perf.c | 10 +- > app/test/test_kni.c | 16 +- > app/test/test_link_bonding.c | 10 +- > app/test/test_link_bonding_mode4.c | 12 +- > app/test/test_mbuf.c | 238 +++++++++++++++= +++--- > app/test/test_pmd_perf.c | 11 +- > app/test/test_pmd_ring.c | 10 +- > app/test/test_reorder.c | 10 +- > app/test/test_sched.c | 16 +- > app/test/test_table.c | 9 +- > app/test/test_table.h | 3 +- > doc/guides/rel_notes/updating_apps.rst | 16 ++ > examples/bond/main.c | 10 +- > examples/distributor/main.c | 11 +- > examples/dpdk_qat/main.c | 10 +- > examples/exception_path/main.c | 14 +- > examples/ip_fragmentation/main.c | 18 +- > examples/ip_pipeline/init.c | 28 +-- > examples/ipv4_multicast/main.c | 21 +- > examples/kni/main.c | 12 +- > examples/l2fwd-ivshmem/host/host.c | 10 +- > examples/l2fwd-jobstats/main.c | 10 +- > examples/l2fwd/main.c | 11 +- > examples/l3fwd-acl/main.c | 11 +- > examples/l3fwd-power/main.c | 11 +- > examples/l3fwd-vf/main.c | 12 +- > examples/l3fwd/main.c | 10 +- > examples/link_status_interrupt/main.c | 10 +- > examples/load_balancer/init.c | 12 +- > examples/load_balancer/main.h | 4 +- > .../client_server_mp/mp_server/init.c | 10 +- > examples/multi_process/symmetric_mp/main.c | 10 +- > examples/netmap_compat/bridge/bridge.c | 12 +- > examples/packet_ordering/main.c | 11 +- > examples/qos_meter/main.c | 7 +- > examples/qos_sched/init.c | 10 +- > examples/qos_sched/main.h | 2 +- > examples/quota_watermark/include/conf.h | 2 +- > examples/quota_watermark/qw/main.c | 7 +- > examples/rxtx_callbacks/main.c | 11 +- > examples/skeleton/basicfwd.c | 13 +- > examples/vhost/main.c | 31 +-- > examples/vhost_xen/main.c | 11 +- > examples/vmdq/main.c | 11 +- > examples/vmdq_dcb/main.c | 10 +- > lib/librte_ether/rte_ethdev.c | 4 +- > lib/librte_mbuf/rte_mbuf.c | 65 ++++-- > lib/librte_mbuf/rte_mbuf.h | 200 +++++++++++++--= -- > lib/librte_mbuf/rte_mbuf_version.map | 8 + > lib/librte_pmd_af_packet/rte_eth_af_packet.c | 6 +- > lib/librte_pmd_bond/rte_eth_bond_alb.c | 16 +- > lib/librte_pmd_e1000/em_rxtx.c | 5 +- > lib/librte_pmd_e1000/igb_rxtx.c | 12 +- > lib/librte_pmd_fm10k/fm10k_ethdev.c | 6 +- > lib/librte_pmd_i40e/i40e_ethdev_vf.c | 6 +- > lib/librte_pmd_i40e/i40e_rxtx.c | 15 +- > lib/librte_pmd_ixgbe/ixgbe_rxtx.c | 12 +- > lib/librte_pmd_pcap/rte_eth_pcap.c | 5 +- > lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c | 7 +- > 62 files changed, 650 insertions(+), 570 deletions(-) >=20 Acked-by: Konstantin Ananyev > -- > 2.1.4