Google Git
Sign in
llvm / scudo / ac4db770356cc5b9a2a9df1bc4a1c3d4c99f5bef
commitac4db770356cc5b9a2a9df1bc4a1c3d4c99f5bef[log] [tgz]
authorPeter Collingbourne <peter@pcc.me.uk>Thu Apr 22 22:41:01 2021 -0700
committerCopybara-Service <copybara-worker@google.com>Sun May 09 19:09:17 2021 -0700
treeba95e34624a76d84eb5a5858ca1b916efa2353d2
parenteff1f9f51091e86c205f917e64169eac18f4f4ec [diff]
scudo: Store header on deallocation before retagging memory.

From a cache perspective it's better to store the header immediately
after loading it. If we delay this operation until after we've
retagged it's more likely that our header will have been evicted from
the cache and we'll need to fetch it again in order to perform the
compare-exchange operation.

For similar reasons, store the deallocation stack before retagging
instead of afterwards.

Differential Revision: https://reviews.llvm.org/D101137

GitOrigin-RevId: 0a5576ecf05f810065b6562605f1cbe78fee2988
1 file changed
tree: ba95e34624a76d84eb5a5858ca1b916efa2353d2
  1. benchmarks/
  2. fuzz/
  3. include/
  4. tests/
  5. tools/
  6. allocator_config.h
  7. atomic_helpers.h
  8. bytemap.h
  9. checksum.cpp
  10. checksum.h
  11. chunk.h
  12. combined.h
  13. common.cpp
  14. common.h
  15. crc32_hw.cpp
  16. flags.cpp
  17. flags.h
  18. flags.inc
  19. flags_parser.cpp
  20. flags_parser.h
  21. fuchsia.cpp
  22. fuchsia.h
  23. internal_defs.h
  24. LICENSE.TXT
  25. linux.cpp
  26. linux.h
  27. list.h
  28. local_cache.h
  29. memtag.h
  30. mutex.h
  31. options.h
  32. platform.h
  33. primary32.h
  34. primary64.h
  35. quarantine.h
  36. release.cpp
  37. release.h
  38. report.cpp
  39. report.h
  40. secondary.h
  41. size_class_map.h
  42. stack_depot.h
  43. stats.h
  44. string_utils.cpp
  45. string_utils.h
  46. tsd.h
  47. tsd_exclusive.h
  48. tsd_shared.h
  49. vector.h
  50. wrappers_c.cpp
  51. wrappers_c.h
  52. wrappers_c.inc
  53. wrappers_c_bionic.cpp
  54. wrappers_c_checks.h
  55. wrappers_cpp.cpp