[dpdk-dev] [PATCH] eal: avoid error for non-existent default PMD path

[dpdk-dev] [PATCH] eal: avoid error for non-existent default PMD path

[Bruce Richardson]

If the default location for the PMD .so files does not exist, it should
not be treated as a fatal error condition like an incorrect path on the
command line. Therefore check that the path exists and is a directory
before adding it to the list of paths to check for PMDs.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 lib/librte_eal/common/eal_common_options.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/librte_eal/common/eal_common_options.c b/lib/librte_eal/common/eal_common_options.c
index e40c04961..450f2664a 100644
--- a/lib/librte_eal/common/eal_common_options.c
+++ b/lib/librte_eal/common/eal_common_options.c
@@ -278,12 +278,13 @@ int
 eal_plugins_init(void)
 {
 	struct shared_driver *solib = NULL;
+	struct stat sb;
 
-	if (*default_solib_dir != '\0')
+	if (*default_solib_dir != '\0' && stat(solib->name, &sb) == 0 &&
+				S_ISDIR(sb.st_mode))
 		eal_plugin_add(default_solib_dir);
 
 	TAILQ_FOREACH(solib, &solib_list, next) {
-		struct stat sb;
 
 		if (stat(solib->name, &sb) == 0 && S_ISDIR(sb.st_mode)) {
 			if (eal_plugindir_init(solib->name) == -1) {
-- 
2.13.6

Re: [dpdk-dev] [PATCH] eal: avoid error for non-existent default PMD path

[Thomas Monjalon]

19/10/2017 10:49, Bruce Richardson:
> If the default location for the PMD .so files does not exist, it should
> not be treated as a fatal error condition like an incorrect path on the
> command line. Therefore check that the path exists and is a directory
> before adding it to the list of paths to check for PMDs.
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>

Applied, thanks