On 12/21/2018 12:28 PM, Andrew Rybchenko wrote:
> On 12/21/18 3:12 PM, Ferruh Yigit wrote:
>> On 10/12/2018 12:36 PM, Andrew Rybchenko wrote:
>>> From: Ivan Malov <ivan.ma...@oktetlabs.ru>
>>>
>>> This capability is reported when supported by the current emitting
>>> sub-device. Failsafe PMD itself does not excercise fast free logic.
>> I think overlay device capability reporting already discussed a few times,
>> the
>> question is if an overlay devices should claim a feature when it depends on
>> underlay devices?
>
> The capability may be reported by the failsafe since it is transparent from
> fast free logic point of view.
Why it is transparent? If one of the underlying device doesn't support/claim
this feature, application can't use this feature with failsafe, isn't it?
>
>> Given that no ack/review given to the patch, I am updating it as rejected.
>
> Is it a new policy? I thought that it was vice versa before.
Hi Andrew,
Yes policy is other-way around indeed, when there is no comment at all default
behavior is accept, but please take above paragraph as my comment to the patch.
And I was thinking it is a little controversial and there is no support to have
it, so lets don't get it. What do you think?
>
>>> Signed-off-by: Ivan Malov <ivan.ma...@oktetlabs.ru>
>>> Signed-off-by: Andrew Rybchenko <arybche...@solarflare.com>
>>> ---
>>> doc/guides/nics/features/failsafe.ini | 1 +
>>> drivers/net/failsafe/failsafe_ops.c | 1 +
>>> 2 files changed, 2 insertions(+)
>>>
>>> diff --git a/doc/guides/nics/features/failsafe.ini
>>> b/doc/guides/nics/features/failsafe.ini
>>> index e3c4c08f2..b6f3dcee6 100644
>>> --- a/doc/guides/nics/features/failsafe.ini
>>> +++ b/doc/guides/nics/features/failsafe.ini
>>> @@ -7,6 +7,7 @@
>>> Link status = Y
>>> Link status event = Y
>>> Rx interrupt = Y
>>> +Fast mbuf free = Y
>>> Queue start/stop = Y
>>> Runtime Rx queue setup = Y
>>> Runtime Tx queue setup = Y
>>> diff --git a/drivers/net/failsafe/failsafe_ops.c
>>> b/drivers/net/failsafe/failsafe_ops.c
>>> index 7f8bcd4c6..e3add404b 100644
>>> --- a/drivers/net/failsafe/failsafe_ops.c
>>> +++ b/drivers/net/failsafe/failsafe_ops.c
>>> @@ -78,6 +78,7 @@ static struct rte_eth_dev_info default_infos = {
>>> DEV_RX_OFFLOAD_SECURITY,
>>> .tx_offload_capa =
>>> DEV_TX_OFFLOAD_MULTI_SEGS |
>>> + DEV_TX_OFFLOAD_MBUF_FAST_FREE |
>>> DEV_TX_OFFLOAD_IPV4_CKSUM |
>>> DEV_TX_OFFLOAD_UDP_CKSUM |
>>> DEV_TX_OFFLOAD_TCP_CKSUM |
>>>
>
