From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM02-BL2-obe.outbound.protection.outlook.com (mail-bl2nam02on0054.outbound.protection.outlook.com [104.47.38.54]) by dpdk.org (Postfix) with ESMTP id E7C1E1B216 for ; Fri, 6 Oct 2017 05:31:28 +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=IfUdbFj3Rj2SFjqjaQUV9rsbO+Kbf0FDxmXRf3r3aLg=; b=T14M879tI+z2hX46KVBlx3VKtIvRVk7BpKFuyOf8ITe5H1tjSZzdUPBN4dFDaMamlOjtBB32fqmedr4p8zBbscpBjdETvtm26aT7fB2SkiaW3MyKXsMPV6zf6+43WZVHoEq+R+YZCUlz5MAf0zSRgPylMYJDegBTGendKi8JeWs= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Santosh.Shukla@cavium.com; Received: from [IPv6:2401:4900:1987:3a7a:c95:1bb1:8a40:6cff] (2401:4900:1987:3a7a:c95:1bb1:8a40:6cff) by DM5PR07MB3099.namprd07.prod.outlook.com (10.172.85.9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Fri, 6 Oct 2017 03:31:25 +0000 To: Thomas Monjalon References: <20170911151837.25092-1-santosh.shukla@caviumnetworks.com> <20171001091440.10232-1-santosh.shukla@caviumnetworks.com> <20171001091440.10232-2-santosh.shukla@caviumnetworks.com> <1712938.NqKmOO8lQc@xps> Cc: dev@dpdk.org, olivier.matz@6wind.com, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com From: santosh Message-ID: <838da98a-47aa-efce-fb98-304e236fa5c6@caviumnetworks.com> Date: Fri, 6 Oct 2017 09:01:07 +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: <1712938.NqKmOO8lQc@xps> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [2401:4900:1987:3a7a:c95:1bb1:8a40:6cff] X-ClientProxiedBy: MAXPR0101CA0010.INDPRD01.PROD.OUTLOOK.COM (10.174.62.20) To DM5PR07MB3099.namprd07.prod.outlook.com (10.172.85.9) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: b5029f74-8c44-42a8-3127-08d50c6aba2b X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:DM5PR07MB3099; X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3099; 3:jNsOHs6aY9QbRvIbmtrEB2yBtIGxGOYKsKruRWHRSC0qCK2M52KWF7TBzdfwEWR0+HA82M3oZGfNrLG9H1WB+krSSjGTw7waNDq096lPYwzOmBMfiYPltize7v4ucY1GJAGJtZF1uh+65KCh+HFlSBoWXaa9gVET7tap2MXcTg9v18ySAUUhuSfar1jKJ+lg/tf+0zoz2AyzeVji2MfZ1jXrHglMpVKuTiAjgWui1vLb1wWnS1T7CsP3sd8ZHzPk; 25:pw8lzbuvJij4gnpXQs+EjgklSvMqxQzAl0HgYOsrrZCI7ylKWC6yniXAKx/d0o3LwOQwiZS02lRKuX82tnJFmpnZWPbjmbmzy+sfpv08Jp/bpp85LgbNQn9qWJF9EuyiLj/D3x/F3+DtiqzRzb2eCQI70Sx5tBB6Ru+vA5EXfJJUZWQLQhL3wJxRDELKJCuJX/qUdYG9RjDRRoPspqv2ObZoeHPOOoiCO0vvuJso97h6csKs6tH6CiCi/ioesjgFWAhiNnu2FRbWI+VEHNqC6uKvBsieZmdjvXhWwWnhc8MGV+cH5GktSjs+mGvNTS8KfLb5zUbywZYsK033arrzBg==; 31:9fW4PnA5FghnaRx5C6tim/avvVe03K04NmCm58CRaQl3FURhfMTY7wjXaQqMpkk1McsmkaPVlw876h3epYq3LgckSlo4unn27AO4OGpS5wt1Z0RpCKoTtfpIaUx1fMb15AeY6/UVpOZVGAoNPrJmsv43R2PJcihWleZyJ+BuXOQSk6EOZYYCrAs8PTe0d8ZdEo5zgLcTSK1sB9ZwHZOPEdztJMYrOwzTZWq+P4EzvlE= X-MS-TrafficTypeDiagnostic: DM5PR07MB3099: X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3099; 20:q3urkr3Wvs/75zHZApn6cX1pguzb2ZiGItnxwivqJiMfZOtpzWEwBJm2FCLDUP4KkIuXilwAZOocn/WpHVJf5fj6Cb4hMnZTlQtbHmhcYg4JpSLpuYDycTTOMgIUe4F7gYzWKbKZAUdgJQXUPOolQ3QBpxRgLGQ3r5WD5cNwpbZQ4FcT7Ugk9XS+Fhc8xiEmvp61nIMxhZtsL0UflbUhNMrEZvky+W8Cs4ishwprakLkijxdcsZhUEWubYQ/4oI/nZvw1Ore9Ln7lHAFfIynBzrMKrrXNlnE9DkPsrPWpPJg8XS36mhPYqSiZhARF4NTaZRLc6HPeiP1jWpSuZHO/86IinEGuhd05uzv65oL7qUBEmD/CGDh5lO9D94wWNQVpKhp1O7J4ADTE8Gc0BbP9lbckrMWCE/IMdPdAVJiXVIqTNJ0AZ42qqO2+DKD35nwqf4w2fgA6fyZZ859sCgPi2YE/9G6/VQu4RDefFbHELuUJC2WuJXvIYJEau9nhFtEfPxsRPTO4UnNZ7DgG5Zif0S3R2yPui2CE9AL+JuiEVjeiA2hKp3Ig628QCfUD0yg6kuIC8UHsFVWklFk5m29sVHdqkdPGlRCjD7J0Ud3lH8=; 4:8H5c4ET4wisbQXiU/YZiFKqtdjFzXFipZwcIscpdOa5OvAaoZnRv4qkFMFePZYpI20nftnzI7T6arKCC3S9WdDD0oOvo52PxTvjbBoIqMFubOjI5Rq6uhnMY06P21uP4vyLXnRVhAJcwUHPL2qggAgeQJ1WMGiS5DDQWyAQF04XqJO/nsg/yBRy7fT5e72JOUFQxWDU63ZIsd+mhVqEIS3f+zO7T7IbeG0ER4AJuS+joPRqsGkUDrYZugI6xFKet 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)(5005006)(8121501046)(10201501046)(100000703101)(100105400095)(3002001)(93006095)(6041248)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123555025)(20161123560025)(20161123562025)(20161123564025)(20161123558100)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:DM5PR07MB3099; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:DM5PR07MB3099; X-Forefront-PRVS: 0452022BE1 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(376002)(346002)(199003)(189002)(24454002)(377454003)(47776003)(76176999)(54356999)(6486002)(50986999)(6916009)(229853002)(6666003)(42882006)(7736002)(2950100002)(65806001)(65956001)(31696002)(8676002)(81156014)(81166006)(189998001)(64126003)(31686004)(8656003)(101416001)(106356001)(8936002)(25786009)(58126008)(6116002)(50466002)(105586002)(33646002)(72206003)(68736007)(230700001)(23746002)(97736004)(5660300001)(1706002)(83506001)(65826007)(2906002)(36756003)(305945005)(93886005)(53936002)(86152003)(4326008)(478600001)(316002)(6246003); DIR:OUT; SFP:1101; SCL:1; SRVR:DM5PR07MB3099; H:[IPv6:2401:4900:1987:3a7a:c95:1bb1:8a40:6cff]; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX: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; DM5PR07MB3099; 23:uOj2RL+4yVuj9fgp2LQM3ZfpXjFxx6oMrrVkz?= =?Windows-1252?Q?F0WJRlPGl2vVYCJQOfk+pNmFMEFUbKGaTyYynjRuuRsVYA+hXRzZ9Gx/?= =?Windows-1252?Q?VtbrUd1WUCmQ0ho7D8CVbKpJ21UKzIBFbPj9rBzB9Wk/rJvOFpxLx8PJ?= =?Windows-1252?Q?Qo6Wtzi4IiwcTXRfd1+UBe3BAXzbTH7cZbTQRRPaia7sRibJ1tcIqVnI?= =?Windows-1252?Q?AP38MjHSfkF42ERvAZoGTmpVa9dZkTLj6Pnd3S3hADQ2lnRWpWSXESbu?= =?Windows-1252?Q?T+hBMqu3idaRdQhMsaTGL4MdUm2BBoM+57JeB2jC/ZQ4lRgUhOwQiBYZ?= =?Windows-1252?Q?yzwKnzRY/zN/mTpv52I6Ft7YP3qlktLW9ONUrfE+3+45m+BqaF4SmmhQ?= =?Windows-1252?Q?aWupab+Px1ZD5k8fIRiWu4diSOoRLMrorT4r505ZJ4BCWwATmT72J62L?= =?Windows-1252?Q?T6vyff7yHubAcwwpFM6Fa7S5YHsT5R+oN76gwM+IgkNg138J40CF51sw?= =?Windows-1252?Q?CqVth+GlEBB7hu7xz8Ma7raNCHw3iba6+8quCp+B4oR42ANbhoHhOd5z?= =?Windows-1252?Q?YpYt0jiO49sVz1p7urU08950aiQPnoqlbaBBT9wU4jaXkC+Q4GLBcWQN?= =?Windows-1252?Q?nqjRjhoa8ZtaKzsANChXSE8OppmVFsMAjoCg9Uwwfl4EnGOrrQkQyZ/5?= =?Windows-1252?Q?fG5t3tLFT5mC0B1wjNnkGPlQ51j9EWBNR2PCg13uA+Kx2BcjxkhgEyVw?= =?Windows-1252?Q?lmRWg3FoL4IyXt+G8CeXaaQ5SmxZ4ngZxTQIV1oQTDyzdJhS07pSXjUh?= =?Windows-1252?Q?IXwJbmogpYr8ghg9gNqdZOcoskFHAzuwDgiB4PCWyUaVfkbC/pNasoKC?= =?Windows-1252?Q?K1z2FI6QT5gl+uws7hHyGdyB08HCe6R8HzWwlp9apzWjmPgPhmTUno1t?= =?Windows-1252?Q?0KJSzBxOJIm1HJyHBlUJR+QmbB2drU03YwSbyaY00hOqyi2gJB/jkHxv?= =?Windows-1252?Q?zuLqXIOIpBiCjDxw4cXI9rR5dLiH6UpKqhSc3Qol+ACoGFWGKTj2Wuia?= =?Windows-1252?Q?38oqQBPL7g9BIPkLIuR4LK7CW0xmxqoJlkucDcHT8V92cUzLYDx/h3QS?= =?Windows-1252?Q?52YLn8b1BbHu+VA+orB/s76wjS4cyHtiXNJHQSWdEerxChKG9tzvlSUa?= =?Windows-1252?Q?P4SLXq39ZVvRk8+1TQGr3HQvcXfEjLqKItsXIveY6kJVAkgyn+AYn38P?= =?Windows-1252?Q?YNvSg1GMGwqh/DWUT/kewGB616zmwllIXYW8YtqpRtH7VErlXFjRZRNJ?= =?Windows-1252?Q?Uc6WzZqi1UCFPPYc+OEyr0R2KbLOs0QVbgBwaZbj50iS1hHtiCtHn6kM?= =?Windows-1252?Q?r9sVIo1hxKVbPXnLcwOZzvTnSzan7ZBuQ=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3099; 6:heb/xSmLwPdzwwbXXlhC/VXvXkF7OFESvxefdf4p+TX5GRgFJ7E0GfY1q3iqxYjbchOKqC5CGmIUu17Gr2MuTMD0xPPe129CpSTQa2Pytw7OnA3sMize01UY1HOfBNwh/oeCRJW70QIEFfcINjYn9M1VcHMKkfFM9NgrYALx+SzjUJIg++eEVlmZvQXWPH+QZd/mrJffrrF/NH3iHnzAbnd1tTXXi+09KLObhnJFa3wOMkJDygMqOH16IM28nSnFaoulGkzSp/9uHfZ2fiH2Ect+uWMTbhSemMh/5vrAr304jGI1AaQkIoFeRWLuCf7ln9PJ5y7EFaPMLkrBQBmRtg==; 5:xcF0bGuClOlhXYZB4I6dfrZz9fAZ4HeHQqUcYG7QHdpEBX6htjhT6fj4M/Hs7/yt23Cl5bL2B09FVXmAdUP0tEgK4tPcjChQWBwQPYnrRtA8qwVsQ3Pjuh4p5OnMU4X43OCS/oTnlny1zqnOPAuVvA==; 24:Hx8ruxdcO/FryrQ+pnmZlScMg0FRR5HLaM51B4wmMAyQ9CV6mKUJCDzSl2dxBpNiDNSDW+agtYPZ2idKd1wzgSKhQOEDArcFBdLF6heeP/o=; 7:U2JkJbQfOtPOegp54o07L7OBPBdIbuHIgJlkHyzPe3W1O1+YE6ms6HhOcniwJP0CmmIBmNO38FW6tm9wdexEUPfbMGughpBYjNDMt5Hk4rj0pqRr23DuZIyE8CR3P6Orf5f6RwsgD1+zdgOlJXHRFqVOC0KDqtXOsthbiELg9aEj7Un/b0KCs8K4q3W+sUOec81vw46WTDl0DDBY605sYLy+THVKFilKVERjPIQTd2Y= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Oct 2017 03:31:25.0065 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR07MB3099 Subject: Re: [dpdk-dev] [PATCH v5 1/2] eal: allow user to override default pool handle 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: Fri, 06 Oct 2017 03:31:29 -0000 On Friday 06 October 2017 05:59 AM, Thomas Monjalon wrote: > 01/10/2017 11:14, Santosh Shukla: >> --- a/lib/librte_eal/common/eal_common_options.c >> +++ b/lib/librte_eal/common/eal_common_options.c >> @@ -98,6 +98,7 @@ eal_long_options[] = { >> {OPT_VFIO_INTR, 1, NULL, OPT_VFIO_INTR_NUM }, >> {OPT_VMWARE_TSC_MAP, 0, NULL, OPT_VMWARE_TSC_MAP_NUM }, >> {OPT_XEN_DOM0, 0, NULL, OPT_XEN_DOM0_NUM }, >> + {OPT_MBUF_POOL_OPS_NAME, 1, NULL, OPT_MBUF_POOL_OPS_NAME_NUM}, >> {0, 0, NULL, 0 } >> }; > I think the options were sorted alphabetically. This is most logical comment so far I got from you. Yes' will do. posting v6. Thanks. > [...] >> --- a/lib/librte_eal/common/eal_internal_cfg.h >> +++ b/lib/librte_eal/common/eal_internal_cfg.h >> @@ -82,7 +82,7 @@ struct internal_config { >> volatile enum rte_intr_mode vfio_intr_mode; >> const char *hugefile_prefix; /**< the base filename of hugetlbfs files */ >> const char *hugepage_dir; /**< specific hugetlbfs directory to use */ >> - >> + const char *mbuf_pool_ops_name; /**< mbuf pool ops name */ > Why this config is not stored in mbuf.c? > Why the config not stored for vfio? hugepage? etc..in that case applicable too. This is correct place to keep for now, unless as discussed in dpdksummit about eal parsing abstraction approach.. plugin style approach so that each module has its own parser. till then It should sit here like other, Its blocker for external-mempool in general case: Where users are forced to hard-code their handle in _OPS_DEFAULT_=.