On Mon, Jun 2, 2025 at 4:08 PM Alex Williamson
<alex.william...@redhat.com> wrote:
> On Fri, 23 May 2025 23:30:15 +0000
> > +function add_id() {
> > +     if echo $(echo ${1} | tr : ' ') > ${2}/new_id 2> /dev/null; then
> > +             echo "Added ${1} to ${2}"
> > +             return 0
> > +     fi
> > +
> > +     return 1
> > +}
>
> I'd suggest using the newer driver_override mechanism rather than
> new_id/remove_id.

Thanks for the tip, I'll include that in the next version.

> I appreciate the work here, I think this could be really useful.  As
> Jason notes a few times, I'd also like to see this automatically
> iterate through all the combinations of options.

Will do.

> Maybe we could also
> think about whether we can define some build environment options to
> specify devices to use and this could all be run automatically from
> 'make kselftest'.

That's an interesting idea. Maybe an environment variable that tests
will look for and use if they aren't provided with an explicit BDF
string on the command line?

    export VFIO_SELFTESTS_DEFAULT_BDF=XXXX:YY:ZZ.AA

Thanks for taking a look!

Reply via email to