Code review comment for ~williamhsu/plainbox-provider-checkbox/+git/plainbox-provide-checkbox:bug/lp-1884235/exchanged-the-sequence-of-graphics-log-parsing

Revision history for this message
William Hsu (williamhsu) wrote :

@Pierre, I fine-turned the MR. The latest change has been verified. Results are as follows. May I have a few minutes of your time to do a double-check. Thank you! :)

### NV modeset enabled ###
u@u-ThinkStationP520c:~/workspace$ date
Thu 13 Aug 2020 11:09:21 AM EDT
u@u-ThinkStationP520c:~/Desktop$ ./graphics_driver.py
------------- VIDEO DRIVER INFORMATION -------------
Video Driver: nvidia
Driver Version: 440.64

------------- HYBRID GRAPHICS CHECK ----------------
Graphics Chipset: NVIDIA (10de:1cb6)
Loaded DDX Drivers: nouveau, modesetting, fbdev, vesa
Hybrid Graphics: no

### NV modeset disabled (UMA) ###
whsu@thinkpad-x:~/workspace/plainbox-provider-checkbox/bin$ date
Thu Aug 13 23:09:38 CST 2020
whsu@thinkpad-x:~/workspace/plainbox-provider-checkbox/bin$ ./graphics_driver.py
------------- VIDEO DRIVER INFORMATION -------------
Video Driver: modesetting
Driver Version: 1.20.5

------------- HYBRID GRAPHICS CHECK ----------------
Graphics Chipset: Intel (8086:0a16)
Loaded DDX Drivers: modesetting, fbdev, vesa
Hybrid Graphics: no

« Back to merge proposal