Hi Megha, Marcin,
> Yes, I agree this would be the ideal way to drive this change, but the > problem is that modifying the existing subport parameter structure would > represent an API change. This would require a deprecation notice, and the > patch would be blocked until 22.11 release. Are you willing to wait until > 22.11? If not, then adding the configuration function for this flag is the > next best thing. Are we making any plans for that? > > Also the name of the feature should not be abbreviated in the patch title. > > In original code, this feature has always been disabled as it impacts >> performance. > > So, in my opinion we should keep it disabled by default and let user enable > > it > > when required. > > > In the original code, yes, it had to be explicitly enabled through a > build-time flag. This was not the best option, and this is precisely what we > are trying to fix with this patch. > But on the other hand all the users of these library that I know use it with > the TC oversubscription turned on. Functionality is more important for them > than performance. Hence my vote now is to enable it by default; those users > that prefer performance over functionality can easily turn this feature off > with no issues. That was plan for this patch and yet another change to remove API change is awaiting. But when we publish this change. BR, /Marcin -------------------------------------------------------------- Intel Research and Development Ireland Limited Registered in Ireland Registered Office: Collinstown Industrial Park, Leixlip, County Kildare Registered Number: 308263 This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.
