On 4/19/2018 6:16 AM, Kuusisaari, Juhamatti wrote: >>>> Why a build time config option for this? Can we make it a runtime devarg? >>> >>> Sure, we can make it a devarg. Or do we even need that? Are there a lot of >> test dependencies that would need to be fixed if we have it enabled by >> default? >> >> Not test dependencies but this may be overkill for some usecases, I prefer >> making this dynamically configurable, no strong opinion though. > > OK, I'll take a look at this and craft a new version. > >>> >>>> Overall we are trying to reduce config options already and this seems >>>> no need to be build time option at all. >>>> >>>> btw, this is a little late in release cycle, so lets target this >>>> patch for next release. >>> >>> The patch is on top of net-next, this should be just fine. >> >> Perhaps we should rename the sub-tree :) because this is not happening first >> time. next-net is not for next release, as it has been Linux, it is for this >> release but just a sub-tree for net PMDs. > > Aha, while reading the docs it says: "All sub-repositories are merged into > main repository for -rc1 and -rc2", so I kind of thought this sub-repo is > going to the next release, as you have rc2 out already.
rc2 is not out yet, we are still working for rc1, which should be soon. > >>> >>>> Thanks, >>>> ferruh >>> > > Thanks, > -- > Juhamatti >
