From: Dexuan Cui <[email protected]>
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
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to