According to the coding conventions and best practice no function should return a struct. However, vtimer_now() returns a struct of the type timex_t. This should be replaced by adding an output parameter.