On 15/11/2018 19:45, Philippe Mathieu-Daudé wrote: > On 15/11/18 19:41, Paolo Bonzini wrote: >> On 15/11/2018 15:55, Philippe Mathieu-Daudé wrote: >>>> >>>> if (access(path, R_OK | W_OK | X_OK)) { >>>> g_test_message("access on path (%s): %s\n", path, >>>> strerror(errno)); >>>> + abort(); >>>> return NULL; >>> >>> Can we remove the 'return NULL's now? >>> >>> Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> >> >> I can remove them, but later on the aborts will become g_test_fail and >> the test will continue without hugepagefs. > > OK, fine then!
Let's remove the returns here, and add them back together with g_test_fail already in this series, in patch 10. Paolo