Hi Anton, On Tue, Nov 29, 2022 at 12:25:11PM +0100, Anton Khirnov wrote: > I'm not the maintainer for this, but I wonder if it wouldn't be cleaner > to instead add something like a 'kernel_driver_blacklist' option that > would contain a comma-separated list of ignored drivers, and default to > 'vgem' (with maybe more added later).
I could go for a list of some kind to make this more extensible, if it's worthwhile. But is it important to make it an "option" (as in, CLI option presumably)? Comma-separated is especially tricky, as I believe the existing dict/arg parsing uses commas to delimit separate key/value pairs. I suppose I could use "+", as there's precedent in vulkan (instance_extensions, etc.). It also feels like "user-configurable" and "default list" are somewhat at odds, because the default list has good reason to be there, and yet might change underneath the user across versions (if there are new vgem-like drivers, etc.). So any CLI user would have to keep up to date with the defaults. I suppose maybe that's not a horrible problem; I'm just trying to think it through, as I wouldn't personally have any need for it if the default list is good. Thanks for your thoughts. Brian _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
