From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0a-001b2d01.pphosted.com (mx0a-001b2d01.pphosted.com [148.163.156.1]) by dpdk.org (Postfix) with ESMTP id BA67F6A44 for ; Tue, 16 Aug 2016 12:35:09 +0200 (CEST) Received: from pps.filterd (m0098399.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.11/8.16.0.11) with SMTP id u7GAXkgg043545 for ; Tue, 16 Aug 2016 06:35:09 -0400 Received: from e28smtp01.in.ibm.com (e28smtp01.in.ibm.com [125.16.236.1]) by mx0a-001b2d01.pphosted.com with ESMTP id 24syq3mfdf-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Tue, 16 Aug 2016 06:35:08 -0400 Received: from localhost by e28smtp01.in.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 16 Aug 2016 16:05:05 +0530 Received: from d28dlp02.in.ibm.com (9.184.220.127) by e28smtp01.in.ibm.com (192.168.1.131) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; Tue, 16 Aug 2016 16:05:04 +0530 X-IBM-Helo: d28dlp02.in.ibm.com X-IBM-MailFrom: gowrishankar.m@linux.vnet.ibm.com X-IBM-RcptTo: dev@dpdk.org Received: from d28relay10.in.ibm.com (d28relay10.in.ibm.com [9.184.220.161]) by d28dlp02.in.ibm.com (Postfix) with ESMTP id 7A70A3940064 for ; Tue, 16 Aug 2016 16:05:03 +0530 (IST) Received: from d28av02.in.ibm.com (d28av02.in.ibm.com [9.184.220.64]) by d28relay10.in.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id u7GAZ2jZ37093606 for ; Tue, 16 Aug 2016 16:05:02 +0530 Received: from d28av02.in.ibm.com (localhost [127.0.0.1]) by d28av02.in.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id u7GAZ0pe015740 for ; Tue, 16 Aug 2016 16:05:01 +0530 Received: from [9.193.77.130] ([9.193.77.130]) by d28av02.in.ibm.com (8.14.4/8.14.4/NCO v10.0 AVin) with ESMTP id u7GAYrlM015274; Tue, 16 Aug 2016 16:04:54 +0530 To: Chao Zhu , dev@dpdk.org References: <1471003413-20685-1-git-send-email-gowrishankar.m@linux.vnet.ibm.com> <1471003413-20685-5-git-send-email-gowrishankar.m@linux.vnet.ibm.com> <000001d1f793$b93a8b90$2bafa2b0$@linux.vnet.ibm.com> Cc: "'Bruce Richardson'" , "'Konstantin Ananyev'" , "'Thomas Monjalon'" , "'Cristian Dumitrescu'" , "'Pradeep'" From: gowrishankar muthukrishnan Date: Tue, 16 Aug 2016 16:04:51 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <000001d1f793$b93a8b90$2bafa2b0$@linux.vnet.ibm.com> Content-Type: text/plain; charset=gbk; format=flowed Content-Transfer-Encoding: 8bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16081610-4789-0000-0000-0000031B2C36 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 16081610-4790-0000-0000-0000119465D5 Message-Id: X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:, , definitions=2016-08-16_07:, , signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1604210000 definitions=main-1608160137 Subject: Re: [dpdk-dev] [PATCH v5 4/8] table: enable table library for ppc64le 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, 16 Aug 2016 10:35:10 -0000 Thanks Chao for bringing up. I had fixed one of .c earlier before rebasing to master. I then dropped the patch while decoupling ppc enablement and bug fixes. Now I realize on why we need this fix as part of enabling table in ppc completely. So, added one additional patch separately fixing this in v6. After this patch, a ltable lib unit test fail as in Intel case currently and it is not related to powerpc changes. If some one already looking into broken unit test failure for table lib, that is helpful. Regards, Gowrishankar On Tuesday 16 August 2016 01:26 PM, Chao Zhu wrote: > Gowrishankar, > > The unit test of table got failed. > When I took a look at the code, it failed on this line in file > lib/librte_table/rte_table_hash_key8.c , > lib/librte_table/rte_table_hash_key16.c, > lib/librte_table/rte_table_hash_key32.c: > > if ((check_params_create_lru(p) != 0) || > ((sizeof(struct rte_table_hash) % RTE_CACHE_LINE_SIZE) != 0) > || > ((sizeof(struct rte_bucket_4_8) % RTE_CACHE_LINE_SIZE) != > 0)) { > return NULL; > } > > if ((check_params_create_lru(p) != 0) || > ((sizeof(struct rte_table_hash) % RTE_CACHE_LINE_SIZE) != 0) > || > ((sizeof(struct rte_bucket_4_16) % RTE_CACHE_LINE_SIZE) != > 0)) { > return NULL; > } > > if ((check_params_create_lru(p) != 0) || > ((sizeof(struct rte_table_hash) % RTE_CACHE_LINE_SIZE) != 0) > || > ((sizeof(struct rte_bucket_4_32) % RTE_CACHE_LINE_SIZE) != > 0)) { > return NULL; > } > > The size of rte_bucket_4_8/16/32 is not aligned to cache line size. This is > normal, because POWER has different cache line size. The change of struct > rte_bucket_4_8/16/32 may solve this problem. I didn't do further > investigation. Can you help to double check this patch? > > > -----Original Message----- > From: Gowrishankar Muthukrishnan [mailto:gowrishankar.m@linux.vnet.ibm.com] > Sent: 2016Äê8ÔÂ12ÈÕ 20:03 > To: dev@dpdk.org > Cc: Chao Zhu ; Bruce Richardson > ; Konstantin Ananyev > ; Thomas Monjalon ; > Cristian Dumitrescu ; Pradeep > ; gowrishankar > Subject: [PATCH v5 4/8] table: enable table library for ppc64le > > From: gowrishankar > > This patch enables librte_table in ppc64le. > > Signed-off-by: Gowrishankar Muthukrishnan > > --- > config/defconfig_ppc_64-power8-linuxapp-gcc | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/config/defconfig_ppc_64-power8-linuxapp-gcc > b/config/defconfig_ppc_64-power8-linuxapp-gcc > index dede34f..41f67d5 100644 > --- a/config/defconfig_ppc_64-power8-linuxapp-gcc > +++ b/config/defconfig_ppc_64-power8-linuxapp-gcc > @@ -59,5 +59,4 @@ CONFIG_RTE_LIBRTE_FM10K_PMD=n # This following libraries > are not available on Power. So they're turned off. > CONFIG_RTE_LIBRTE_SCHED=n > CONFIG_RTE_LIBRTE_PORT=n > -CONFIG_RTE_LIBRTE_TABLE=n > CONFIG_RTE_LIBRTE_PIPELINE=n > -- > 1.9.1 > > >