On Friday 30 October 2015 10:55:10 Deepa Dinamani wrote:
I looked at the spreadsheet(sheet 3). Nothing else actually needs timeval being exposed to userspace. Either whatever uses timeval, is only using it internally or uses time_t to user.
Ah, excellent!
I think there are a few that might not be on the list because I've already addressed them in another patch:
* include/uapi/linux/can/bcm.h (my patch is already merged) * include/uapi/linux/input.h (hard to fix, working on it) * include/uapi/linux/videodev2.h (also nontrivial, has been discussed at last weeks media summit, patch under work) * drivers/char/ppdev.c (patch exists, needs to get merged)
Each of the above is a bit different and needs a custom solution anyway, but it's definitely good to know that there are no others.
I submitted a v2 of the patch that doesn't require a new timeval struct accordingly.
Thanks,
Arnd