From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1972BC433EF for ; Thu, 21 Oct 2021 12:06:21 +0000 (UTC) Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by mail.kernel.org (Postfix) with ESMTP id 693E460F9F for ; Thu, 21 Oct 2021 12:06:20 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 693E460F9F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=huawei.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=dpdk.org Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8FC524118E; Thu, 21 Oct 2021 14:06:18 +0200 (CEST) Received: from szxga02-in.huawei.com (szxga02-in.huawei.com [45.249.212.188]) by mails.dpdk.org (Postfix) with ESMTP id 22E6F4117A for ; Thu, 21 Oct 2021 14:06:16 +0200 (CEST) Received: from dggemv711-chm.china.huawei.com (unknown [172.30.72.56]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4HZmKv0Rh0zbh5R; Thu, 21 Oct 2021 20:01:39 +0800 (CST) Received: from dggpeml500024.china.huawei.com (7.185.36.10) by dggemv711-chm.china.huawei.com (10.1.198.66) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.15; Thu, 21 Oct 2021 20:06:13 +0800 Received: from [127.0.0.1] (10.67.100.224) by dggpeml500024.china.huawei.com (7.185.36.10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.15; Thu, 21 Oct 2021 20:06:13 +0800 To: Thomas Monjalon , , , Bruce Richardson CC: , References: <20210924102942.2878051-1-bruce.richardson@intel.com> <20211013151736.762378-1-bruce.richardson@intel.com> <3092116.XSkZisFCfb@thomas> From: fengchengwen Message-ID: Date: Thu, 21 Oct 2021 20:06:13 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 MIME-Version: 1.0 In-Reply-To: <3092116.XSkZisFCfb@thomas> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.67.100.224] X-ClientProxiedBy: dggems704-chm.china.huawei.com (10.3.19.181) To dggpeml500024.china.huawei.com (7.185.36.10) X-CFilter-Loop: Reflected Subject: Re: [dpdk-dev] [PATCH v7 00/13] add test suite for DMA drivers X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Bruce, I observed a large number of checkpatch errors [1] when synchronizing to our inner CI, almost all of them are over 80 lines, and many are not LONG LOG. The DPDK coding style recommends to be not more than 80 characters unless rarest situations (which LONG LOG belongs to this one I think). I don't know which to follow: just ignore or should fix it ? [1]: lib/dmadev/rte_dmadev.c:56: WARNING:LONG_LINE: line length of 95 exceeds 80 columns lib/dmadev/rte_dmadev.c:696: WARNING:LONG_LINE: line length of 87 exceeds 80 columns lib/dmadev/rte_dmadev.c:704: WARNING:LONG_LINE: line length of 85 exceeds 80 columns lib/dmadev/rte_dmadev.h:267: WARNING:LONG_LINE_COMMENT: line length of 84 exceeds 80 columns lib/dmadev/rte_dmadev.h:269: WARNING:LONG_LINE_COMMENT: line length of 84 exceeds 80 columns lib/dmadev/rte_dmadev.h:677: WARNING:LONG_LINE_COMMENT: line length of 91 exceeds 80 columns lib/dmadev/rte_dmadev.h:683: WARNING:LONG_LINE_COMMENT: line length of 95 exceeds 80 columns lib/dmadev/rte_dmadev.h:691: WARNING:LONG_LINE_COMMENT: line length of 90 exceeds 80 columns lib/dmadev/rte_dmadev.h:692: WARNING:LONG_LINE_COMMENT: line length of 82 exceeds 80 columns lib/dmadev/rte_dmadev.h:706: WARNING:LONG_LINE: line length of 88 exceeds 80 columns lib/dmadev/rte_dmadev_core.h:51: WARNING:LONG_LINE: line length of 86 exceeds 80 columns lib/dmadev/rte_dmadev_pmd.h:58: WARNING:LONG_LINE: line length of 84 exceeds 80 columns total: 1 errors, 12 warnings, 235 lines checked app/test/test_dmadev.c:19: WARNING:LONG_LINE: line length of 95 exceeds 80 columns app/test/test_dmadev.c:19: WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided app/test/test_dmadev.c:39: WARNING:LONG_LINE: line length of 94 exceeds 80 columns app/test/test_dmadev.c:72: WARNING:LONG_LINE_COMMENT: line length of 90 exceeds 80 columns app/test/test_dmadev.c:77: WARNING:LONG_LINE_COMMENT: line length of 96 exceeds 80 columns app/test/test_dmadev.c:79: WARNING:LONG_LINE: line length of 83 exceeds 80 columns app/test/test_dmadev.c:85: WARNING:LONG_LINE_COMMENT: line length of 95 exceeds 80 columns app/test/test_dmadev.c:90: WARNING:LONG_LINE_COMMENT: line length of 90 exceeds 80 columns app/test/test_dmadev.c:113: WARNING:LONG_LINE: line length of 86 exceeds 80 columns app/test/test_dmadev.c:114: WARNING:LONG_LINE: line length of 98 exceeds 80 columns app/test/test_dmadev.c:115: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:124: WARNING:LONG_LINE: line length of 85 exceeds 80 columns app/test/test_dmadev.c:129: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:138: WARNING:LONG_LINE: line length of 96 exceeds 80 columns app/test/test_dmadev.c:143: WARNING:LONG_LINE: line length of 97 exceeds 80 columns app/test/test_dmadev.c:157: WARNING:LONG_LINE: line length of 90 exceeds 80 columns app/test/test_dmadev.c:158: WARNING:LONG_LINE: line length of 88 exceeds 80 columns app/test/test_dmadev.c:169: WARNING:LONG_LINE: line length of 92 exceeds 80 columns app/test/test_dmadev.c:196: WARNING:LONG_LINE: line length of 94 exceeds 80 columns app/test/test_dmadev.c:207: WARNING:LONG_LINE: line length of 95 exceeds 80 columns app/test/test_dmadev.c:249: WARNING:LONG_LINE: line length of 88 exceeds 80 columns app/test/test_dmadev.c:254: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:272: WARNING:LONG_LINE_COMMENT: line length of 99 exceeds 80 columns app/test/test_dmadev.c:277: WARNING:LONG_LINE_COMMENT: line length of 91 exceeds 80 columns app/test/test_dmadev.c:287: WARNING:LONG_LINE: line length of 86 exceeds 80 columns app/test/test_dmadev.c:299: WARNING:LONG_LINE_COMMENT: line length of 86 exceeds 80 columns app/test/test_dmadev.c:302: WARNING:LONG_LINE: line length of 94 exceeds 80 columns app/test/test_dmadev.c:306: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:313: WARNING:LONG_LINE: line length of 94 exceeds 80 columns app/test/test_dmadev.c:314: WARNING:LONG_LINE: line length of 85 exceeds 80 columns app/test/test_dmadev.c:329: WARNING:LONG_LINE_COMMENT: line length of 82 exceeds 80 columns app/test/test_dmadev.c:331: WARNING:LONG_LINE: line length of 86 exceeds 80 columns app/test/test_dmadev.c:332: WARNING:LONG_LINE: line length of 82 exceeds 80 columns app/test/test_dmadev.c:338: WARNING:LONG_LINE_COMMENT: line length of 97 exceeds 80 columns app/test/test_dmadev.c:339: WARNING:LONG_LINE_COMMENT: line length of 96 exceeds 80 columns app/test/test_dmadev.c:344: WARNING:LONG_LINE: line length of 90 exceeds 80 columns app/test/test_dmadev.c:351: WARNING:LONG_LINE_COMMENT: line length of 92 exceeds 80 columns app/test/test_dmadev.c:357: WARNING:LONG_LINE_COMMENT: line length of 85 exceeds 80 columns app/test/test_dmadev.c:367: WARNING:LONG_LINE: line length of 94 exceeds 80 columns app/test/test_dmadev.c:368: WARNING:LONG_LINE: line length of 85 exceeds 80 columns app/test/test_dmadev.c:372: WARNING:LONG_LINE: line length of 93 exceeds 80 columns app/test/test_dmadev.c:379: WARNING:LONG_LINE: line length of 85 exceeds 80 columns app/test/test_dmadev.c:380: WARNING:LONG_LINE: line length of 86 exceeds 80 columns app/test/test_dmadev.c:392: WARNING:LONG_LINE: line length of 94 exceeds 80 columns app/test/test_dmadev.c:396: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:421: WARNING:LONG_LINE_COMMENT: line length of 87 exceeds 80 columns app/test/test_dmadev.c:441: WARNING:LONG_LINE: line length of 86 exceeds 80 columns app/test/test_dmadev.c:443: WARNING:LONG_LINE_COMMENT: line length of 83 exceeds 80 columns app/test/test_dmadev.c:455: WARNING:LONG_LINE: line length of 94 exceeds 80 columns app/test/test_dmadev.c:459: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:479: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:503: WARNING:LONG_LINE: line length of 98 exceeds 80 columns app/test/test_dmadev.c:515: WARNING:LONG_LINE_COMMENT: line length of 92 exceeds 80 columns app/test/test_dmadev.c:524: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:529: WARNING:LONG_LINE: line length of 85 exceeds 80 columns app/test/test_dmadev.c:533: WARNING:LONG_LINE: line length of 93 exceeds 80 columns app/test/test_dmadev.c:548: WARNING:LONG_LINE_COMMENT: line length of 88 exceeds 80 columns app/test/test_dmadev.c:551: WARNING:LONG_LINE_COMMENT: line length of 92 exceeds 80 columns app/test/test_dmadev.c:560: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:568: WARNING:LONG_LINE: line length of 87 exceeds 80 columns app/test/test_dmadev.c:570: WARNING:LONG_LINE: line length of 81 exceeds 80 columns app/test/test_dmadev.c:598: WARNING:LONG_LINE: line length of 94 exceeds 80 columns app/test/test_dmadev.c:601: WARNING:LONG_LINE: line length of 83 exceeds 80 columns app/test/test_dmadev.c:605: WARNING:LONG_LINE_COMMENT: line length of 95 exceeds 80 columns app/test/test_dmadev.c:611: WARNING:LONG_LINE: line length of 83 exceeds 80 columns app/test/test_dmadev.c:624: WARNING:LONG_LINE_COMMENT: line length of 93 exceeds 80 columns app/test/test_dmadev.c:625: WARNING:LONG_LINE_COMMENT: line length of 90 exceeds 80 columns app/test/test_dmadev.c:648: WARNING:LONG_LINE: line length of 91 exceeds 80 columns app/test/test_dmadev.c:654: WARNING:LONG_LINE: line length of 94 exceeds 80 columns app/test/test_dmadev.c:660: WARNING:LONG_LINE: line length of 83 exceeds 80 columns app/test/test_dmadev.c:662: WARNING:LONG_LINE_COMMENT: line length of 85 exceeds 80 columns app/test/test_dmadev.c:693: WARNING:LONG_LINE: line length of 84 exceeds 80 columns app/test/test_dmadev.c:697: WARNING:LONG_LINE: line length of 88 exceeds 80 columns app/test/test_dmadev.c:698: WARNING:LONG_LINE: line length of 84 exceeds 80 columns app/test/test_dmadev.c:702: WARNING:LONG_LINE: line length of 91 exceeds 80 columns app/test/test_dmadev.c:713: WARNING:LONG_LINE: line length of 84 exceeds 80 columns app/test/test_dmadev.c:719: WARNING:LONG_LINE: line length of 90 exceeds 80 columns app/test/test_dmadev.c:721: WARNING:LONG_LINE: line length of 100 exceeds 80 columns app/test/test_dmadev.c:755: WARNING:LONG_LINE: line length of 85 exceeds 80 columns app/test/test_dmadev.c:765: WARNING:LONG_LINE: line length of 93 exceeds 80 columns app/test/test_dmadev.c:774: WARNING:LONG_LINE: line length of 89 exceeds 80 columns app/test/test_dmadev.c:789: WARNING:LONG_LINE_COMMENT: line length of 88 exceeds 80 columns app/test/test_dmadev.c:790: WARNING:LONG_LINE: line length of 85 exceeds 80 columns app/test/test_dmadev.c:794: WARNING:LONG_LINE: line length of 93 exceeds 80 columns app/test/test_dmadev.c:797: WARNING:LONG_LINE_COMMENT: line length of 97 exceeds 80 columns app/test/test_dmadev.c:802: WARNING:LONG_LINE: line length of 99 exceeds 80 columns app/test/test_dmadev.c:811: WARNING:LONG_LINE: line length of 92 exceeds 80 columns app/test/test_dmadev.c:812: WARNING:LONG_LINE: line length of 86 exceeds 80 columns app/test/test_dmadev.c:833: WARNING:LONG_LINE_COMMENT: line length of 98 exceeds 80 columns total: 0 errors, 89 warnings, 861 lines checked On 2021/10/18 17:20, Thomas Monjalon wrote: > 13/10/2021 17:17, Bruce Richardson: >> Bruce Richardson (10): >> dmadev: add channel status check for testing use >> dma/skeleton: add channel status function >> dma/skeleton: add burst capacity function >> dmadev: add device iterator >> app/test: add basic dmadev instance tests >> app/test: add basic dmadev copy tests >> app/test: run test suite on skeleton driver >> app/test: add more comprehensive dmadev copy tests >> dmadev: add flag for error handling support >> app/test: test dmadev instance failure handling >> >> Kevin Laatz (3): >> dmadev: add burst capacity API >> app/test: add dmadev fill tests >> app/test: add dmadev burst capacity API test > > Applied, thanks. > > > > > . >