On 8/25/17 5:44 PM, Andre McCurdy wrote: > On Fri, Aug 25, 2017 at 5:16 PM, Khem Raj <[email protected]> wrote: >> On Fri, Aug 25, 2017 at 4:37 PM, Andre McCurdy <[email protected]> wrote: >>> On Fri, Aug 25, 2017 at 3:42 PM, Khem Raj <[email protected]> wrote: >>>> OE-core minimal kernel has been bumped to 4.4 >>>> so at this point I think its fine to remove it since there >>>> is no reliable way for OE-Core to test out validity of >>>> this patch >>> >>> If that's the official policy then let's bump OLDEST_KERNEL and make >>> the point clear to everyone. >> >> its not official policy. However we do try to minimise patches we >> carry as much as we can moreover I was thinking from testability point >> of view, we don't know what harm it can cause when you dont need this >> patch. > > Yes we do. The case where we don't need the patch (ie kernel 4.4) is > the case we *can* test. > > If the patch is found to cause problems with newer kernels then > certainly it should be removed (or updated as Martin suggested), but I > haven't seen anyone suggest that it's causing any issues. >
Just being there it could have side-effects too, I dont think we have enough testing scenario to establish the effect. so removing is best unless someone with 4.1 kernel or older can test it out with newer kernels and establish that its not causing any undesired effects on kernels where its not required. Until then it should stay removed. -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
