kernel/memremap: Remove stale devres_free() call
devm_memremap_pages() was re-worked ine8d5134833
"memremap: change devm_memremap_pages interface to use struct dev_pagemap" to take a caller allocated struct dev_pagemap as a function parameter. A call to devres_free() was left in the error cleanup path which results in a kernel panic if the remap fails for some reason. Remove it to fix the panic and let devm_memremap_pages() fail gracefully. Fixes:e8d5134833
("memremap: change devm_memremap_pages interface...") Signed-off-by: Oliver O'Halloran <oohall@gmail.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Logan Gunthorpe <logang@deltatee.com> Signed-off-by: Dan Williams <dan.j.williams@intel.com>
This commit is contained in:
parent
661e50bc85
commit
167f5594b5
|
@ -427,7 +427,6 @@ void *devm_memremap_pages(struct device *dev, struct dev_pagemap *pgmap)
|
||||||
err_pfn_remap:
|
err_pfn_remap:
|
||||||
err_radix:
|
err_radix:
|
||||||
pgmap_radix_release(res, pgoff);
|
pgmap_radix_release(res, pgoff);
|
||||||
devres_free(pgmap);
|
|
||||||
return ERR_PTR(error);
|
return ERR_PTR(error);
|
||||||
}
|
}
|
||||||
EXPORT_SYMBOL(devm_memremap_pages);
|
EXPORT_SYMBOL(devm_memremap_pages);
|
||||||
|
|
Loading…
Reference in New Issue