From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0063.outbound.protection.outlook.com [104.47.42.63]) by dpdk.org (Postfix) with ESMTP id 5ED251B221 for ; Tue, 10 Oct 2017 06:48:29 +0200 (CEST) Received: from BN3PR03CA0058.namprd03.prod.outlook.com (10.167.1.146) by CY4PR03MB2695.namprd03.prod.outlook.com (10.173.43.138) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Tue, 10 Oct 2017 04:48:27 +0000 Received: from BL2FFO11OLC001.protection.gbl (2a01:111:f400:7c09::187) by BN3PR03CA0058.outlook.office365.com (2a01:111:e400:7a4d::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.13.10 via Frontend Transport; Tue, 10 Oct 2017 04:48:27 +0000 Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not designate 192.88.158.2 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.158.2; helo=az84smr01.freescale.net; Received: from az84smr01.freescale.net (192.88.158.2) by BL2FFO11OLC001.mail.protection.outlook.com (10.173.161.185) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.20.77.10 via Frontend Transport; Tue, 10 Oct 2017 04:48:27 +0000 Received: from [10.232.14.39] ([10.232.14.39]) by az84smr01.freescale.net (8.14.3/8.14.0) with ESMTP id v9A4mMZC005220; Mon, 9 Oct 2017 21:48:23 -0700 To: Don Provan , Jan Blunck , Thomas Monjalon CC: dev , Hemant Agrawal References: <20170812102220.27773-1-shreyansh.jain@nxp.com> <2075457.Vvey9mxHue@xps> <10403057.Ll0Xg1E4J1@xps> <83422f57-4c0e-5806-c741-ed5ce10891b0@nxp.com> From: Shreyansh Jain Message-ID: <83a3c6c6-8d50-8106-7c7f-9b5c8263ce96@nxp.com> Date: Tue, 10 Oct 2017 10:30:36 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-EOPAttributedMessage: 0 X-Matching-Connectors: 131520845079058930; (91ab9b29-cfa4-454e-5278-08d120cd25b8); () X-Forefront-Antispam-Report: CIP:192.88.158.2; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(7966004)(336005)(39860400002)(346002)(376002)(39380400002)(2980300002)(1110001)(1109001)(339900001)(199003)(13464003)(377454003)(24454002)(189002)(498600001)(31696002)(81156014)(2950100002)(305945005)(50986999)(83506001)(77096006)(23676002)(8656003)(54356999)(76176999)(54906003)(356003)(110136005)(316002)(4326008)(6666003)(31686004)(50466002)(64126003)(97736004)(58126008)(69596002)(229853002)(8676002)(2906002)(105606002)(6246003)(106466001)(65806001)(65956001)(53936002)(104016004)(81166006)(33646002)(93886005)(189998001)(8936002)(5660300001)(53546010)(86362001)(65826007)(47776003)(230700001)(68736007)(36756003); DIR:OUT; SFP:1101; SCL:1; SRVR:CY4PR03MB2695; H:az84smr01.freescale.net; FPR:; SPF:Fail; PTR:InfoDomainNonexistent; A:1; MX:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BL2FFO11OLC001; 1:yiauVVyqq7pvdWiHCwveva7MOEjiuamlezXCcdxiS0WoZJ2wxo09qEwHm8vC27xtn1b9YitZDPJR/tdqLlh+s2uSjPLOdi8aXtdL+FwLW3bEjPzzfc+KlL9hDbBCqDMR X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 48941731-4478-44e3-b894-08d50f9a25f3 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017052603199)(201703131430075)(201703131517081); SRVR:CY4PR03MB2695; X-Microsoft-Exchange-Diagnostics: 1; CY4PR03MB2695; 3:KB6tFOQgyjejeiIRDd0dJMVqLAD4ZBRO34szVwDF8q+ky2HiaxaE3272Tlgfgy4pznO0B+BUkFAYhzT4cHvncZ3oAWTvfb1/pfFyC2mSoguarH/+tQ1uGqFoUIGd47Mm+p8iiFHNSqwn09EavmD59XjwGzMQAB3dXoi2wUd0UtFGPwEVcLH7OdyzwNbj1+qzyWWK36/3DxMaFFoI9TvvfK6T1Bz6LiSFSjUZANQhHxRkQsFJi4JZuwRg34DiiFg4fTzN0yahOyAu5WppsBdxIGIZXyo67L5m9J0+ktQIlHsutM4M0tpM9fm6289T30Dvq9JXxt1POpYRB9tHmZulbqux/aIxLUAZqG6SiUC1vxs=; 25:c/vYqjRG+gJTTnRieOyxUl69enWGYBUJV8Ynh+2RTOuZ7DnUwB1R79CSbSAdQahCVqm2UAvtvEZ4ZfNaCZP9ug5avmUpzJ/plLFNk3eV5eZ6+De9hnlicAYemF8aBzTNAbooYdBD7wdPNvdIvIqs3BkvQfr0ZOZNUkmH9qolwJnXo1DpQ4HZ59Yc8UrVwvB6JOp5ziPu2CJDu6ukDEVpIXm7m0A4n1mALmiTqyCWVePE14LPegsCs+BDcMdgDOApMNOysOQg7eyODMZr8Tf/+LDPmmgy5AMn0ThAY0KrvYds8EaOjZU425dULPVUFh480DtR4yphtczAtcVmSO7BNQ== X-MS-TrafficTypeDiagnostic: CY4PR03MB2695: X-Microsoft-Exchange-Diagnostics: 1; CY4PR03MB2695; 31:zbmXUruAGBqgZqiahH2w2DtrWn2iauDAWsXqjgLI7hsBIMfJh448uHFC24yh+rsYXwvhItd808YLu3pfh7Y5PcWMILNP82hyCfg/YOhnlnL0bOPVgNurOORmpjRiIHTung8dSR/3g6JekPrUj1+O01SVSCHTHl6MG2FknBrvCkH/lQoxQF2vBQvpEbMXw9Lfk51VPziKxTveVGl2Cktg7n/3fpda+7XQ8EjLQV8/D9c=; 4:QoVcNnzCmnVHkvj06flJM/lcK4MNwPDeUB/yPbVSRWEcv8ZgQReD3GMIa/9WdmVpl51Xv4uZPO8UGSdjm8q9en+dywr01iVfpiLZoet6QNyL2p+PTE7gySBMOWUfcH8h/VmX9DnuYmI/FK3RhAeO9KB52MNEnornV3YTCW84L1Ppc0xVT1NHb8bJWLRhD8a4KRL7ZP+ynD1k3x/Lad17NkTfsMIG0j021imJCFRFATd3n4yXCgcT7eGl3KuZlNb+edR6Aje0ictjKglgTqOwuaShDZyuD0hSeuzV2crON5oHbgHmzcpWxr7kV9HlKUKVW1RckQ+UOi6FhImVHZt/Rg== X-Exchange-Antispam-Report-Test: UriScan:(185117386973197)(211171220733660); X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6095135)(2401047)(5005006)(8121501046)(3002001)(100000703101)(100105400095)(10201501046)(93006095)(93001095)(6055026)(6096035)(20161123565025)(201703131430075)(201703131433075)(201703131441075)(201703131448075)(201703161259150)(20161123559100)(20161123561025)(20161123563025)(20161123556025)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:CY4PR03MB2695; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(400006)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:CY4PR03MB2695; X-Forefront-PRVS: 04569283F9 X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtDWTRQUjAzTUIyNjk1OzIzOnV2VTAyU0JPc1dQUmZJZFc1dzdGWEhza3Fj?= =?utf-8?B?Smx4cTFDcjJrVldmcFZSZ2VZZWNraGZEUGtwUnRSV25EcGhycVNURTBLaWZ3?= =?utf-8?B?Qm13TkRDc1Y4WjVZc3l2ejFra0dMZmI2RFhhQmdEU3FrVWxpTDI3dXBINkFn?= =?utf-8?B?UUVpQmhBZ0tBSTFOL3c0b1ZQSzBjRVZldkxWZjg1aHdmM2NaVnFnemxtRjRp?= =?utf-8?B?QkV5czNvRXpIL1pHdDRvRTVoRFBlY3pNSk8rSE1DN0pKRFdmVjRUSEZNRzFs?= =?utf-8?B?RWdQZWc4dFE2aWN6eEN0T3YvMjJjNDllTnNvcVpRTVp1R2VXZFJFM2U2cDAv?= =?utf-8?B?WTMrQjJRTy9wcGdibW15SkpHanNScDNtekZwSHNFU3AyRVlxQ08rUlcxVUZr?= =?utf-8?B?NnBMdHZOc3FQcGl0b3o5MTRyUlJGTWlJZ0FCaThPUW52LzJwMTdhekh0T2dB?= =?utf-8?B?MW5GWEd0ZElWejBWeW01dlVFODFxcTVSVTNoUjBXbjVjY0pPb0lJK3JsVUJt?= =?utf-8?B?eXg2R3haQWVBU2JPZW5FVjN6VmFuSjkrbDg0UTZ0RGZXVWs4eFQ2bFk0YTQv?= =?utf-8?B?Y0Uxa1RZb0xicVc4cWpvaXl1TTNmZW5vdUZBaFdqdXlmTnU2bDRCZHh1cVcw?= =?utf-8?B?V2ZSQlhqWUJiOUNkNUhhRE00V0tWOTNQRGtMVkpnTDdjWXZzM1AzQVYzTGc5?= =?utf-8?B?TE94b0FFVEVNQ2pyU3ZJOXFnUmMwMlYxbFF4SDA4cnd6cmwwMmJPM1ZiRkhM?= =?utf-8?B?RnRxNWg1RkNFaU5ZL1BsaFhvQnFKcU5ZUXZKbHV0TklmOHV6VzByYzBEeXF3?= =?utf-8?B?M3lYd2tjVGg1MTNMN3EyY08rRDBlbmRoeGY5Yy9DbnJtM2c3Si9ESG03c0lj?= =?utf-8?B?aUE2dnJ6dnpzb1daejlpMkdRRkZ4enQvWHJxazAycVBWelNtWm1FY2NHOTRW?= =?utf-8?B?R2RHaGh4UXg1Y3Z2eWhyeUNFc0hXOFZJU3BxN0k0N3E1MG0zYTRlZG0xb0Fj?= =?utf-8?B?OXJhbjF5MlNWVDNTRmFNc1FYdTNJS2EzZkZXMEtCeGFUc01tdlk0cDE1S0k1?= =?utf-8?B?dFNJTWNVK0VUSDhnVVFMQVZObnJlcUl5K0FhckJwUHNpR1owT2hEQlpDT0s0?= =?utf-8?B?RTMrY0h0MGNQK21hQVVlcHFqWEpoeE5wT292dmVWaWg4dFk4Qzlmc2xhWmJK?= =?utf-8?B?MUg2VVBpbkd0UVdpdE5UTDcybkZjVCsxbUpkdm1tNzIvaHB3VW1xaEplR2J1?= =?utf-8?B?aXhYWnZRemFxSWw1NlQvWDYwd2QzVXBaR1VuOXlkVy9TL0VtTFB4NHJyNmRP?= =?utf-8?B?cGtsZUNud29sR0lkdC9hNDRudHhMYW1JdUxGc0Jldi8yNHNtaEVaN0ZqMFps?= =?utf-8?B?Qld4eVBhUjg1RDJNejFxMzNyVVF1MTVWemY4WDh3RlpHWjZEZVZqUDhqcmF1?= =?utf-8?B?SlFSV3ZtSjVCR3h4ZmpjZGdPOU0rZzQrM1BzQmdPL2grT3Q3OG9xRHlwMFdC?= =?utf-8?B?cnY2MnphZlRGM01Bei9zeSt6ejQvL1FDN20xc3pzUndZMzRtNEFTZC9ZY1pB?= =?utf-8?B?NWNxWHE4cU1ueFloWFJXbkxlaHhKT0pFU1BNbVhFbjdkTzkvMW0xc3ZRRlQv?= =?utf-8?B?QnFvRUdZZlpOWFdiMUp1Zi9yNUFjQStlRm9GYjYrK2JBSGhMbUdhb2ZvNW9D?= =?utf-8?B?dlR2Kzk5MEhtUVJkRTJJTkxNREJMMXo3dVFYSE9HRVpVdGcwQzNmUnpkSmpq?= =?utf-8?B?bFo1dEhzVC9MNjR2Y1Qydm81M2hWajdhelhReUdSODFRRzZ2K1ZCSHJwNTZz?= =?utf-8?B?L1pNeGJCTk1YcFd5N2xrQW5FdDcveWxKc0JLdWdOZXdHYW56ZTNHMXl6RTRE?= =?utf-8?B?OWIvVlZJd1lBSWc4U2ZNbTBLRkp4NGFremRpaCtiWnlINjdsWmYyVGFhU2hF?= =?utf-8?B?aCtOdUFnK0xNanJtWnF1ZEQyQ1k5b2pIZ1dwakpSTHM5bmVtVzkvd1Fwd0Ja?= =?utf-8?Q?DVvFwH?= X-Microsoft-Exchange-Diagnostics: 1; CY4PR03MB2695; 6:D67bpIYvllPdpB3i2Kx+6RFWq4g3SOgBWIxbBEcoHMIbJhVUX130fB3UxZDU+Jn+1yFBFCmRzG7pJ6XNGu3k4rXqOgbJJdwKxU9MBbw1X/qaGk76N9UA0zFO84NlrxfKttp849apIFA9GzrREGuPtFiWV8j38FwnCc0FSxKJKAaHzrLmXK67U9V3kO+MAZ99FlPZc7XI0D5H95hR3bZa3Zo0GlCBMU8US5mW3HBqtp4GB6c8Ma25I9whA2aCU/qbj4qRGPHFnSJ2Bbd6aah9fO6+7F6cdM5ufuE5PHrVrSUW2ynMe5iVdkCmD/WtQijGiAy8cVS0UsMkkPXQBlNrPg==; 5:vnApBzcUc1Vy20krMelBOZ0xfYDIkOsZLZBdLI97Fe8ofeWrsHeRrVdmzyngEIc3MDAu1pqjvgilsmvho5GbGk5rtDGuFKH2Y6asb4fJ7Cbwd76EPSJuJ1l8hlevwv6DQt2YuxzmEZPhGy2yMqwtcA==; 24:l/0Dy6UFwZBE41mLCZpAFkS3dHX5jRKYO9sezEqGhPZZ1QdviwAedfpZvurfCtRjfyNxXPQo+9YWk5tejRL2i4YaF/lk1XiSh8tpfoUF0jM=; 7:4+orr145WEaTnKHXjaoc7JWley/jse5kuWX6PDojr3KYrDTmAvGSm635/B7iylfEZkGqQPOGoVu2MR5VzJCGrbE9R8QCuSumQF5f0BFc4Kvhp7EbL5BQt2WmT132dfBgiCp4kTbGBmCEWgm49rhhnN9GgX2tmjnekoNCJ72sBAAh/ExpnB8L/lG+RSW08tbiXBOR2daQ3IcBEpRlWOFiGgX9hNqPegpJz1fyieJjvnM= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Oct 2017 04:48:27.6094 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.158.2]; Helo=[az84smr01.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR03MB2695 Subject: Re: [dpdk-dev] [PATCH] eal: bus scan and probe never fail 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: Tue, 10 Oct 2017 04:48:29 -0000 Hello Don, On Monday 09 October 2017 11:51 PM, Don Provan wrote: >> -----Original Message----- >> From: Shreyansh Jain [mailto:shreyansh.jain@nxp.com] >> Sent: Monday, October 09, 2017 4:10 AM >> To: Jan Blunck ; Thomas Monjalon >> >> Cc: dev ; Hemant Agrawal >> Subject: Re: [dpdk-dev] [PATCH] eal: bus scan and probe never fail >> >> ... >> This is where I have disagreement/doubt. >> Reporting error code from rte_bus_scan would do two things: >> >> 1. rte_eal_init is not designed to ignore/log-only these errors - it >> would quit initialization. (But, this can be changed) >> 2. What should rte_eal_init do with this error? rte_bus_scan would have >> already printed the problematic bus->scan() failure. > > These practical problems confirm to me that the failure of a bus > scan is more of a strategic issue: when asking "which devices can > I use?", "none" is a perfectly valid answer that does not seem > like an error to me even when a failed bus scan is the reason for > that answer. I agree with this. > > From the application's point of view, the potential error here > is that the device it wants to use isn't available. I don't see that > either the init function or the probe function will have enough > information to understand that application-level problem, so > they should leave it to the application to detect it. I think I understand you comment but just want to cross check again: Scan or probe error should simply be ignored by EAL layer and let the application take stance when it detects that the device it was looking for is missing. Is my understanding correct? I am trying to come a conclusion so that this patch can either be modified or pushed as it is. If the above understanding is correct, I don't see any changes required in the patch. > > -don provan > dprovan@bivio.net >