diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2018-09-03 10:44:31 +0100 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2018-09-03 10:55:24 +0100 |
commit | 125eee6e981eac0a004aeb4f327439a132ceac5c (patch) | |
tree | e4f38b4ddb74cfa6ea91d55ea6478382d6f350ca /lib/drmtest.c | |
parent | 20087bf22698612a526353f022bc232e2b0dcdcc (diff) |
lib: Fix strcmp() direction for bsearch.
I fluked out as vgem was the initial mid value, hiding the worst of the
errors as i915 matched with DRIVER_ANY.
Fixes: 20087bf22698 ("lib: Use a bsearch to find the module name")
Reported-by: Petri Latvala <petri.latvala@intel.com>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Petri Latvala <petri.latvala@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'lib/drmtest.c')
-rw-r--r-- | lib/drmtest.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/drmtest.c b/lib/drmtest.c index bfb38f1e..adff1a81 100644 --- a/lib/drmtest.c +++ b/lib/drmtest.c @@ -226,9 +226,9 @@ static int open_device(const char *name, unsigned int chipset) int mid = start + (end - start) / 2; int ret = strcmp(modules[mid].module, dev_name); if (ret < 0) { - end = mid; - } else if (ret > 0) { start = mid + 1; + } else if (ret > 0) { + end = mid; } else { chip = modules[mid].bit; break; |