Re: [PATCH v3 2/8] KVM: selftests: Add demand paging content to the demand paging test

On Mon, Dec 16, 2019 at 01:38:55PM -0800, Ben Gardon wrote:

[...]

+static void *uffd_handler_thread_fn(void *arg) +{

• struct uffd_handler_args *uffd_args = (struct uffd_handler_args *)arg;
• int uffd = uffd_args->uffd;
• int64_t pages = 0;
• while (!quit_uffd_thread) {

• struct uffd_msg msg;
  

• struct pollfd pollfd[1];
  

• int r;
  

• uint64_t addr;
  

• pollfd[0].fd = uffd;
  

• pollfd[0].events = POLLIN;
  

• /*
  

•  * TODO this introduces a 0.5sec delay at the end of the test.
  

•  * Reduce the timeout or eliminate it following the example in
  

•  * tools/testing/selftests/vm/userfaultfd.c
  

•  */
  

• r = poll(pollfd, 1, 500);
  

Would you mind implement it instead of adding a todo? IIUC it's as simple as a few more lines than the comment itself. Thanks,