[dpdk-dev] [PATCH v1 2/5] eal: add a new req event to device event
Jeff Guo
jia.guo at intel.com
Tue Aug 21 09:37:05 CEST 2018
On 8/21/2018 3:20 PM, Andrew Rybchenko wrote:
> On 21.08.2018 09:56, Jeff Guo wrote:
>> hi, andrew
>>
>>
>> On 8/20/2018 6:37 PM, Andrew Rybchenko wrote:
>>> On 17.08.2018 13:51, Jeff Guo wrote:
>>>> Add a new req event in eal device event for vfio hotplug. When the req
>>>> request send from the vfio kernel module be detected, vfio userpace
>>>> driver could use this event to notify the app to handler it.
>>>>
>>>> Signed-off-by: Jeff Guo <jia.guo at intel.com>
>>>> ---
>>>> lib/librte_eal/common/include/rte_dev.h | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/lib/librte_eal/common/include/rte_dev.h
>>>> b/lib/librte_eal/common/include/rte_dev.h
>>>> index ff580a0..0324c84 100644
>>>> --- a/lib/librte_eal/common/include/rte_dev.h
>>>> +++ b/lib/librte_eal/common/include/rte_dev.h
>>>> @@ -30,6 +30,7 @@ extern "C" {
>>>> enum rte_dev_event_type {
>>>> RTE_DEV_EVENT_ADD, /**< device being added */
>>>> RTE_DEV_EVENT_REMOVE, /**< device being removed */
>>>> + RTE_DEV_EVENT_REQ, /**< device being removed */
>>>
>>> Comment is the copy of previous one.
>>>
>>
>> You are right here, even we process these type in the same way but
>> should be considered it as the different type if we exactly want to
>> add new one.
>> so base on the interpret from the vfio kernel driver, this req event
>> is used to require the user space to release the device resources, so
>> it should be
>> interpret it by "device release request". If you object and have
>> other better idea, let me know.
>
>
> If so, may be it should be RTE_DEV_EVENT_RELEASE_REQ, since just _REQ
> is really misleading.
>
accept it , sounds better, thanks.
>>
>>>> RTE_DEV_EVENT_MAX /**< max value of this enum */
>>>> };
>>>
>
More information about the dev
mailing list