Hi, On 19-12-17 14:22, Justin Forbes wrote:
On Tue, Dec 19, 2017 at 7:13 AM, Josh Boyer <jwboyer@xxxxxxxxxxxxxxxxx <mailto:jwboyer@xxxxxxxxxxxxxxxxx>> wrote: On Tue, Dec 19, 2017 at 6:03 AM, Hans de Goede <hdegoede@xxxxxxxxxx <mailto:hdegoede@xxxxxxxxxx>> wrote: > Hi All, > > Good news, the vboxguest driver has been queued for > upstream merging in char-misc-next. This just happened > so I want to wait for a couple of days to make sure > they stick and they do not get reverted for some reason. Nice job! That's been needed a long time and I'm really happy to see your success in getting them upstream. > Then I would like to add them as downstream patches > to the rawhide kernels for now, they can be dropped > once we rebase to 4.16. > > So as always when making non trivial changes, my > question to the Fedora kernel team is, is adding > these as downstream patches ok? I have no strong opinion, but I'm curious why we wouldn't just wait for 4.16. That seems like a natural sync point so that everyone that provides this driver as a kmod simply stops when the first 4.16 merge window kernel lands. Adding them now means more coordination for users and kmod builders and I wonder if that will cause confusion. Also worth noting, 4.16 will be the Fedora 28 release kernel, so it will make F28 either way.
To answer you both, I would like to get this into Rawhide now, to get as much testing as possible, esp. surrounding the integration issues with 3th party repos providing kmods, that needs some time to all get figured out. I've been discussing upgrade paths with the current 3th party repo kmod maintainers here: https://bugzilla.redhat.com/show_bug.cgi?id=1481630 My current line of thinking is to only enable this in Fedora 28+ and only have a VirtualBox-guest-additions package (which will need this) in the F28+ repos. The downside of this is that we will need to remember to disable this once 4.15 with downstream patches or 4.16 hits F26 / F27, I'm not sure if there is any other such feature, we could do something like this in the .spec (and drop it in a year from now): # TODO: Drop this once we no longer support Fedora 27 %if 0%{?fedora} < 28 sed 's/CONFIG_VBOXGUEST=m/# CONFIG_VBOXGUEST is not set' .config %endif Regards, Hans _______________________________________________ kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
