13/03/2018 10:50, Yang, Zhiyong: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 13/03/2018 09:46, Yang, Zhiyong: > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > 05/03/2018 08:43, Yang, Zhiyong: > > > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > > > I don't see why it should be exported outside of DPDK, except for > > PMDs. > > > > > > I would tend to keep it internal but I understand that it would > > > > > > mean duplicating some code, which is not ideal. > > > > > > Please could you show what would be the content of the .h in EAL? > > > > > > > > > > > > > > > > If needed to expose them in eal.h, I think that they should be the > > > > > whole fdset mechanism as followings. > > > > > > > > > > typedef void (*fd_cb)(int fd, void *dat, int *remove); > > > > > > > > > > struct fdentry { > > > > > int fd; /* -1 indicates this entry is empty */ > > > > > fd_cb rcb; /* callback when this fd is readable. */ > > > > > fd_cb wcb; /* callback when this fd is writeable.*/ > > > > > void *dat; /* fd context */ > > > > > int busy; /* whether this entry is being used in cb. */ > > > > > }; > > > > > > > > > > struct fdset { > > > > > struct pollfd rwfds[MAX_FDS]; > > > > > struct fdentry fd[MAX_FDS]; > > > > > pthread_mutex_t fd_mutex; > > > > > int num; /* current fd number of this fdset */ > > > > > }; > > > > > > > > > > void fdset_init(struct fdset *pfdset); (not used in the patchset) > > > > > > > > > > int fdset_add(struct fdset *pfdset, int fd, > > > > > fd_cb rcb, fd_cb wcb, void *dat); (used in this patchset) > > > > > > > > > > void *fdset_del(struct fdset *pfdset, int fd); (not used in the > > > > > patchset) > > > > > > > > > > void *fdset_event_dispatch(void *arg); (used in this patchset) > > > > > > > > > > seems that we have 4 options. > > > > > 1) expose them in librte_vhost > > > > > 2) expose them in other existing or new libs. for example, eal. > > > > > 3) duplicate the code lines at PMD layer. > > > > > 4) do it as the patch does that. > > > > > > > > It looks to be very close of the interrupt thread. > > > > Can we have all merged in an unique event dispatcher thread? > > > > > > If I understand right, do you mean that we can merge them in lib eal ? > > > right? > > > > Yes merge with interrupt thread in EAL. > > I didn't look at the details, but it seems the right place for such thing. > > > Ok, we have to expose them as new APIs. Expect that somebody as DPDK users > can use and like them as well. :)
I think you missed my initial question: Is it possible to merge the vhost events needs in the EAL interrupt thread?
