]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
D14130: stand/fdt: Rip out FDT VA tracking
authorkevans <kevans@FreeBSD.org>
Thu, 1 Feb 2018 14:31:12 +0000 (14:31 +0000)
committerkevans <kevans@FreeBSD.org>
Thu, 1 Feb 2018 14:31:12 +0000 (14:31 +0000)
commit1bef7c29bbb4679935a5350f2c347aac2adc6b73
tree9d468cf87e836491a3a7e025e5219d6038d1d7a8
parentef01ed01d824f2fedfc7517b00ba9bfd5229311d
D14130: stand/fdt: Rip out FDT VA tracking

Whether we should be overwriting the loaded FDT module with the 'fixed up'
version or not was questionable when this was added, and now that overlays
are possible this is downright wrong.

Overlays can increase the size of the blob, so writing it back to the
original VA will generally write past the end of the block and start
clobbering other things in memory.

Rip it out- it was questionable to begin with, it's doing bad things now,
and it serves no purpose since the modified blob will be copied into place
rather than relying on this to reflect the changes.

Reviewed by: gonzo
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D14130
stand/fdt/fdt_loader_cmd.c