[LTP] [PATCH v2 1/4] lib/tst_test.c: add 'needs_drivers' option with tst_check_drivers cmd

[Alexey Kodanev]

On 09/13/2018 03:27 PM, Cyril Hrubis wrote:
> Hi!
>> +2.2.26 Checking kernel for the driver support
>> +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>> +
>> +Someties test needs certain drivers to be available in the kernel and must
>> +end with TCONF if any are missing. For this task there is the '.needs_drivers'
>> +option which accepts NULL-terminated array of the drivers names.
> 
> Maybe we should explicitly say here that this works for both drivers
> that are compiled-in and modules, since otherwise this may confuse the
> users, especially since we talk about modprobe in the next paragraph.
> 
> I would write something as:
> 
> "Some tests may need specific kernel drivers, either compiled in, or
> loaded as a module. If .need_drivers points to a NULL-terminated array
> of kernel module names these are all checked and the test exits with
> TCONF on first missing driver."
> 

OK, thanks! I guess this also correlates with using the --dry-run option
of modprobe that I'm going to add in the 3rd version [1], if there are no
objections.

[1] http://lists.linux.it/pipermail/ltp/2018-August/009128.html