From: Dexuan Cui <de...@microsoft.com>
Date: Fri, 24 Jun 2016 07:45:24 +0000

> +     while ((ret = vmalloc(size)) == NULL)
> +             ssleep(1);

This is completely, and entirely, unacceptable.

If the allocation fails, you return an error and release
your resources.

You don't just loop forever waiting for it to succeed.
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to