[dpdk-dev] [PATCH V10 1/2] eal: add uevent monitor api and callback func

[Guo, Jia]

On 1/15/2018 7:32 PM, Thomas Monjalon wrote:
> 15/01/2018 11:55, Guo, Jia:
>> On 1/15/2018 7:16 AM, Thomas Monjalon wrote:
>>> Hi,
>>>
>>> 11/01/2018 15:05, Jeff Guo:
>>>> +/* A genaral callback for all registerd devices */
>>> Typos: genaral, registerd
>>>
>>> So the callback is only for registered devices?
>>> What about hotplugged devices?
>> the hotplugged devices is managed by the application, if it prior
>> registered and in the app's hotplug device list, will always be monitor
>> whenever it plug in and out.  the eal only care about the registered
>> devices.
> I disagree. The application needs the EAL service to get notified
> of a new device plugged in.
> We should find a way to register the callback for all devices,
> including new ones.
i think the current mechanism would let eal service detect all new 
device in , but would not notify all to the application. if the 
application need to monitor specific device , it will prior know the 
device name. but if notify all  in eal service and let application 
choose it in user side ,that is other story,  that is what you want ? 
but do you think about , if it is need to let user know all device 
plugin , or just let eal know all info would be better or safer?