> if (PageHWPoison(hpage)) { > pr_info("soft offline: %#lx hugepage already poisoned\n", pfn); > - return -EBUSY; > + ret = -EBUSY; > + goto out;
Doesn't look like a code improvement to me. Single return is easier and simpler. -Andi > +out: > return ret; > } > -- > 1.7.1 > > -- a...@linux.intel.com -- Speaking for myself only. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/