Re: [dpdk-dev] [PATCH] net/*/base: allow experimental APIs in base of avf and qede

[Ilya Maximets <i.maximets@samsung.com>]

On 04.10.2018 17:51, Ferruh Yigit wrote:
> On 10/4/2018 2:18 PM, Ilya Maximets wrote:
>> This functionality was missed while adding new drivers to
>> the meson build.
> 
> Is there any call from base code to experimental APIs? What do you think adding
> this when we need it, since I have feeling that this may not be needed.

The patch itself does not enable the experimental APIs, it only
adds the ability to enable them in the future. It's actually
pretty hard for the person that not much familiar with meson to
find out why your code provides deprecation warnings. I spent
few hours trying to make it work but failed. After that Bruce found
that 'base' code parts ignores 'allow_experimental_apis' flag and
prepared the patch. Thank you, Bruce!
This patch possibly will save a few hours for someone like me, who
will try to enable experimental APIs for avf and qede drivers.

And I already need avf part for my patch set with nanosleep
based delay:
	http://patches.dpdk.org/cover/46088/

> 
>>
>> Fixes: 1d75caf81474 ("net/avf: add meson support")
>> Fixes: 30d3d0168301 ("net/qede: add in meson build")
>>
>> Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
>> ---
>>  drivers/net/avf/base/meson.build  | 3 +++
>>  drivers/net/qede/base/meson.build | 3 +++
>>  2 files changed, 6 insertions(+)
>>
>> diff --git a/drivers/net/avf/base/meson.build b/drivers/net/avf/base/meson.build
>> index 90fd6b445..6f3d7192e 100644
>> --- a/drivers/net/avf/base/meson.build
>> +++ b/drivers/net/avf/base/meson.build
>> @@ -8,6 +8,9 @@ sources = [
>>  
>>  error_cflags = ['-Wno-pointer-to-int-cast']
>>  c_args = cflags
>> +if allow_experimental_apis
>> +	c_args += '-DALLOW_EXPERIMENTAL_API'
>> +endif
>>  foreach flag: error_cflags
>>  	if cc.has_argument(flag)
>>  		c_args += flag
>> diff --git a/drivers/net/qede/base/meson.build b/drivers/net/qede/base/meson.build
>> index 59b41c895..71b89737d 100644
>> --- a/drivers/net/qede/base/meson.build
>> +++ b/drivers/net/qede/base/meson.build
>> @@ -45,6 +45,9 @@ error_cflags = [
>>  	'-Wno-pointer-bool-conversion',
>>  ]
>>  c_args = cflags
>> +if allow_experimental_apis
>> +	c_args += '-DALLOW_EXPERIMENTAL_API'
>> +endif
>>  foreach flag: error_cflags
>>          if cc.has_argument(flag)
>>                  c_args += flag
>>
> 
> 
>