From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-DM3-obe.outbound.protection.outlook.com (mail-dm3nam03on0074.outbound.protection.outlook.com [104.47.41.74]) by dpdk.org (Postfix) with ESMTP id E83BB58F6 for ; Mon, 4 Sep 2017 16:18:31 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=vMtEDHhtnxKxtoZe3DukFIx/wv65lKK0JkaP2e6Qaq0=; b=CI6Vrug7b7EACGB9pW3bNbf4OvObgydH0sBYKCAcGhMOWEQpSIc8YYVZg7zSsyJ+Vn1VEt9fS6kVZgfaPMQMZfOrTXrBCflKlh5ZS87TLXVeZUZ125xHtKt9dRSApEwe3or9nMhvgaHyai6I0F4CVrWH7yFmW3CSpPS9D/c09TQ= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Santosh.Shukla@cavium.com; Received: from [10.89.1.1] (14.140.2.178) by MWHPR07MB3104.namprd07.prod.outlook.com (10.172.95.10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.13.10; Mon, 4 Sep 2017 14:18:27 +0000 To: Olivier MATZ References: <20170720134759.4680-1-santosh.shukla@caviumnetworks.com> <20170815060743.21076-1-santosh.shukla@caviumnetworks.com> <20170815060743.21076-2-santosh.shukla@caviumnetworks.com> <20170904141154.vpj3hhmzp7pkjpg3@neon> Cc: dev@dpdk.org, thomas@monjalon.net, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com From: santosh Message-ID: <4551380e-0b62-2ca6-4228-c02be8535748@caviumnetworks.com> Date: Mon, 4 Sep 2017 19:48:08 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <20170904141154.vpj3hhmzp7pkjpg3@neon> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [14.140.2.178] X-ClientProxiedBy: BM1PR01CA0109.INDPRD01.PROD.OUTLOOK.COM (10.174.208.25) To MWHPR07MB3104.namprd07.prod.outlook.com (10.172.95.10) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 414793b4-8866-4a2d-5424-08d4f39fd103 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:MWHPR07MB3104; X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3104; 3:trEwBzryz0K5gG0+LOsX0qvOw2KjGZFpbb6RcGQDUts3nPJdZzf2uoo717FZ5eJu6uNtsFVAruvBtkMeegjRUwRmJ/O+0rA0BYg6b0TwtEpqMElhe2b0TENAphvI2CW0DFXqz9v1TKMMeykKdvxOsqLo0YKLKnic+j5iholTI3Kq68jTM7HBZrshvQEp8Ml2WD1u2RIWHRG6yOoUdvX2QJWMAixWMMGoWKwNFR3FlSn+ElM8XRRcnHlKF0yGW29U; 25:8qholOyO4XHRR9yuyx0/ZDPfpRMBuijOnl1dg5HIe/1BRqEtd5SxaYVJb6ctlP4vOa72i40nYrYemK+75KZW/nnH0+H96AsSpHFiIsNEjwMzy0wGn5wCh4xemRky0RciDzTIA3bQNDF4WWBZ/s1MYIiQFO1QIRl8AsMlggbBSf/xvLukpOK1/H3pqa4cjw5G8J/5U6Djoa5ly12M/GLfSgptDmVVCzfAOEZapu12lTjMvnZg5OD3+t3Uc+RD3q+1Vk4m8ldvUP4Elwd4ayySdk/D6ijtHWOak/0F3MGZfvMht9J97enPwRg4a6GFemeYTUNQxU1BfVDzCsTTmnu7og==; 31:eDsA9Ct21bQdp7JDvAxAPYhLfVynrzdBKNFVykGVhCU1W/KwT7Kc1Kuu954lSB0iaZuw9uEqE1vpcNSVBSPZT3xysm6QASDjdeQFr8//OSFKi+Qe5NvcFtW5p559p5l4jDWHsx55GbIgcFGGcqv/l0w5wkOMtmbP83fywP4rl1E5je4134DnkInOjXPCDZ2DXrw23YPB6SDkjZEaze0Kuzkpxs9QSCUC2LbkoE6pRvE= X-MS-TrafficTypeDiagnostic: MWHPR07MB3104: X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3104; 20:ZPtUTIFXf/CcLS9PT20hf0+3QBJBkifI9KLs8y9IVwcGysNa/4MHA/ZigjnVtP57KWfMBUggBBdCCAlk3Q1VfqTuJFdul6Tsp7NYNW9DkYzRngB0OCrDNJGS32jDiMIB2Gt/8MnTfjMVOSh0ZoIQA2n8l96qOaB+0zLNyauaI+RSEvyMQYr4hQQmZkwU/vzIew/75oeJ6XeJPV9bl7t7zFhvGp1llZQmiNSMTnMSx0YJ+RFRyggLbj6AX6sC+6iO9wgRTh4FE0YgG8tCbB13b7M6yw2WmoMbjARPWSsaLyMgsVT8FKzuh2AGzYWZlFTHMDyISDVpf6voy4jSSDClWlosbQ70fnOglEELqYX1dQuAw+k37a/QFc3NU76Sp8L+wNDMKQ7NmL6V/fzon6Yp5r3Fov0shx1e/dqLcpiUOaxwRelnKYHPHB5VLbL0W3wr+fiHZ+11b6pwjNub3cZijMEoKZH4FZCvoAUTSyiMSkXMQr5NbK9S4qRc10y9KPlevE+Xf2GFrQymeZqCzgwdC3+NyBVZ6q0OIxTr0Hn4s9y7ZVhrdRS/NZFMaJu7H/SJXfbp+zwVh1yPN7FfcWtyneyR0A3nJheFfBlVC1TBcUg=; 4:RnCTEWgK6KiGOidUINJbfqYkUg5tQBIC9+s2kvhUWuD96Vtzpeblg2F2xhvdZv3rg2MwSOfH9Pl1GDKIzjpgE4xfuU88Z/bKbB2YW0nz++TRbPUQgZasEV87JGhLtP8z5cnr1mV+QxTktfKAJeroyBctGPzff3fgAhuPTRGSneQ2iURM9yPV0/aZRbuSXaOXV96WUo/OEjlmftlUakYGe0ahHxotcii6EdwtdYBev1yj/01cztAz/38P9YPJwbqh X-Exchange-Antispam-Report-Test: UriScan:; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(3002001)(10201501046)(93006095)(100000703101)(100105400095)(6041248)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(20161123562025)(20161123555025)(20161123564025)(20161123560025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:MWHPR07MB3104; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:MWHPR07MB3104; X-Forefront-PRVS: 0420213CCD X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(6049001)(24454002)(189002)(199003)(377454003)(6666003)(8676002)(5009440100003)(7736002)(3846002)(81166006)(305945005)(8936002)(230700001)(6116002)(81156014)(25786009)(93886005)(4326008)(5660300001)(31686004)(478600001)(72206003)(65826007)(83506001)(97736004)(31696002)(42882006)(2950100002)(2906002)(6916009)(66066001)(65956001)(65806001)(64126003)(33646002)(50466002)(101416001)(36756003)(47776003)(42186005)(76176999)(106356001)(4001350100001)(50986999)(54356999)(105586002)(77096006)(6486002)(90366009)(23746002)(229853002)(110136004)(53936002)(68736007)(86152003)(6246003)(189998001)(8656003)(110426004); DIR:OUT; SFP:1101; SCL:1; SRVR:MWHPR07MB3104; H:[10.89.1.1]; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1; MWHPR07MB3104; 23:VfloMa+B4LlZ6kLJAr9c6GLr+0wVUyzSpW6p0?= =?Windows-1252?Q?D7Jl52cdxXBs5hF4LMQuCpq9bVieLUf80FwFs6aHSBdY57X6Jn0N+2Z3?= =?Windows-1252?Q?PRf1yGIHwv2NUaiwlV+L6B1/87JiHRw7mN+RuWhRNEPgAKy0B1g9zj/R?= =?Windows-1252?Q?xqTmexgCqCebjNbyOYJg6ur19lxIOysI/A8r6v6nB+FDx4lPGQaSt0Jv?= =?Windows-1252?Q?uiZbQcKn2LWxnKWQFWnIV28pFCvSISyPL8aX+L+Wgq5u8bqWqjwQ5D52?= =?Windows-1252?Q?GmlFus9apC46MrUsIrt6Q80EElRei4BUAtypNG1Kq3hYQO9ksjG8zeCt?= =?Windows-1252?Q?CETv2J1vvdVaJ2Aobc+YgAWWHAXXOlKju9WmywKUISGxS6oFT8VUgVsc?= =?Windows-1252?Q?psEG/ZdzikzmyZUj+D/xV+pczKeP5JMLAhFOgnr3WFPganl8m7umQvid?= =?Windows-1252?Q?XNZBRlqE7uhLZC9Vp3jftl1Zk1h0Z6SDO0XUViQJKKFb1W0Axt8qmVSg?= =?Windows-1252?Q?IvCnrLhc/tE9LHkloneCdQ6Hau29iqTxnFhZcQHwasYBo4tNeAvMg2i4?= =?Windows-1252?Q?CiPA0ZAhzwicpTXtwkhSd3noPIH/RimNyu5TnPC9yWOZMzpzUHSf8+7P?= =?Windows-1252?Q?/+fLCoANng5vCbZVA6yoDIJNeiuxXYiq0y6AF2dOMjAAz6/zQr0mEUmX?= =?Windows-1252?Q?TOdTGiZkafzn7o7+Bb7X/JomvVnKU1nEXz+PheD446AZfSvJTDnSo+XX?= =?Windows-1252?Q?dqd4RZhanIn+LuiHX+4Yr1w0wL1W962GC8NpAZUJcN0lU6te1+LARSYu?= =?Windows-1252?Q?X3jLE/4JzKSTqOw5oGTFDTERoffXFh/+V1R+8wL1N+4vOSdhQ+SKg616?= =?Windows-1252?Q?lBvWmItWmyDRHb4wGpsPBxZ320d0uaYvnS0gn7V1EwoONq/Mjc5ThSoU?= =?Windows-1252?Q?DTzfov3qz73O+jvVffxC/mC4QS436MUTC3hoP1irnnhjjBukj3PBwdN9?= =?Windows-1252?Q?l7Uh5Q5Ofnt2LAwvEp9yZXyE3YCZ82OGldOTCYeC2eqTTSzG9dgb/q0X?= =?Windows-1252?Q?NIwQPXfayFMFV7kiuwQKqKMAnp0G4r8Cz7nF+LHZiJA6qWJmkBjZJNKf?= =?Windows-1252?Q?iZNZRCpwYeCyPEc5FOhaj6d+HfdPmKzWVOW6LXY3Ja+wzK6ZVDShjB06?= =?Windows-1252?Q?ASNpGaip2lkYSIgDrYVrp8fL4bADFOuvOr81OagQJkRMLg3NAQ/gWMNm?= =?Windows-1252?Q?kpMv7oXCzBRCDhCMrd5DlHGQjYjvRtZSs5s4Xdy6FzQij7GTXI03ZInO?= =?Windows-1252?Q?1cPTuAGGkfSADqfEgW2BxbpFd/a9UeENyd7FtxxmEfGpKr9VR+Qye1l+?= =?Windows-1252?Q?qgK8TgcKaR6i2JBlJYVR8rDHnrx77nG/fKwbUFdxbmpWkDBYhlGE9rTW?= =?Windows-1252?Q?5CTMz4Dj/kaW9oXzGFJI61noB2b8tAoiXnljxN2X2MCEg48vjPckszNE?= =?Windows-1252?Q?Jmqb88vW7Bfun4kPKDghCZxWWZB7GDdrOPCYec6fGezBD4Zm9MIMUJX9?= =?Windows-1252?Q?pOY2SNsfLVNsro=3D?= X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3104; 6:1xtjVz7H8DWdOGyDK+HRsqYWQdY5oP+ulJMvMdyJa07IzPXSpjIOcdx+wfdZSQNEUYABVo7BZoqR3zrrN7weRA5wFENVRfSzBfhBeszDa+tt51FYX8XS3dcvw9I/9pgKAeECBwA5ZedbrK9tSV7xNCNwPSPuDS6AGuMVcSFJL/B8amlDvm4CNxMYZFASVlAJabTwvnxPwnWuS78mDCsIKcS7ln69/6yx/eh3mE17ZRHpd32OhabHSBAsCuiR0SMrjEqA8foP7L9Nj3q2c7NqI+9bzTLfrzWlZ1JvHr/rfAiOPpJbPoBjHZFzSR4VXh1l8t81/rLI+t+OKTgy2oNtbA==; 5:Sou+2rzmykujnGoAxRuUSyDU9B0ukhgyixIU8yQHtoGh7dNGTitA7RjqF4wK1mJL+dW3YaAi6rWoKVe8ZuXu/cPyfuyxxUPQx1FqzfwGdxj03vvRA7oAeSawSUR6EnPKLCZiZ162V3OCHGLzrAH04w==; 24:fP3ON0IzxXbKJnPSTVkLhgEjuGbIkdnLWZB9ij2N3flA1sJ7SBO+YetNTL7y94j4vrHIzHZ6MLL1db9hy48b0BYJitFGXfGGO/YTkF8vj5A=; 7:wdCvDPQ8PCysi6Urpp6CFAvYRBLrYdQcuXQtfxJSpOEMgjQbQ7WLiY0RgxRpCYpKTMqEBRh9eKoXhtPoNzgMhQL8i1geC/tFU4JDjqEUhXysJpkX+8zkA4Z5Kaq3kXb9OL4ty1vrcl1rMSUNzHZy7hkIjEndJ5eyclaG41yirxnoU1fvEu6+iEE0vQjavd3Cv1Mc7ykTaywOd9UAMIQh/zAPfIeTKOfjSSCZtGyyN88= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Sep 2017 14:18:27.4025 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR07MB3104 Subject: Re: [dpdk-dev] [PATCH v4 1/7] mempool: fix flags data type X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 04 Sep 2017 14:18:32 -0000 On Monday 04 September 2017 07:41 PM, Olivier MATZ wrote: > On Tue, Aug 15, 2017 at 11:37:37AM +0530, Santosh Shukla wrote: >> mp->flags is int and mempool API updates unsigned int >> value in 'flags', so fix the 'flags' data type. >> >> Patch also does mp->flags cleanup like: >> * Remove redundant 'flags' API description from >> - __rte_mempool_generic_put >> - __rte_mempool_generic_get >> >> * Remove unused 'flags' param from >> - rte_mempool_generic_put >> - rte_mempool_generic_get >> >> * Fix mempool var data types int mempool.c >> - mz_flags is int, Change it to unsigned int. > This bullet list makes me think that we should have at least 2 commits: > mempool: remove unused flags argument > mempool: change flags from int to unsigned int > Ok, will split in v5.