On Thu, Nov 14, 2019 at 12:20:13AM -0800, Dan Williams wrote:
> + if (count > 1) {
> + /* still busy */
> + return;
> + } else if (count == 0) {
> + /* only triggered by the dev_pagemap shutdown path */
> + __put_page(page);
> + return;
> + } else if (!is_device_private_page(page)) {
> + /* notify page idle for dax */
> + wake_up_var(&page->_refcount);
> + return;
> + }
No need for an else after a return.
