)]}' { "log": [ { "commit": "a7e928b8bb8d79aa2feb809c1bd4752eecc68802", "tree": "c68d910d9444de4faca0a216d8c2e0162f41cbee", "parents": [ "66b0df34a721e0f7fe09d1606a92b5e4d909c4a9" ], "author": { "name": "Benoît Lizé", "email": "lizeb@chromium.org", "time": "Thu Jun 04 16:03:51 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jun 04 16:05:58 2026" }, "message": "[tools] Add a tool to report IOAccelerator memory locally.\n\nThis is used to investigate IOAccelerator memory usage locally.\nFor instance, on a mac laptop showing a simple rotating cube:\n\n ✗ tools/memory/mac/parse_mac_vmmap_ioaccelerator.py --pid 75120\nRunning vmmap on PID 75120...\nTotal IOAccelerator entries: 196\nTotal memory size: 687.60 MB\n\nHistogram of total memory size per bucket:\n 0.1 KB | 1 entries | 0.00 MB |\n 0.5 KB | 1 entries | 0.00 MB |\n 0.6 KB | 1 entries | 0.00 MB |\n 2.3 KB | 1 entries | 0.00 MB |\n 16.0 KB | 48 entries | 0.75 MB |\n 32.0 KB | 35 entries | 1.09 MB |\n 40.0 KB | 1 entries | 0.04 MB |\n 48.0 KB | 35 entries | 1.64 MB |\n 64.0 KB | 13 entries | 0.81 MB |\n 128.0 KB | 5 entries | 0.62 MB |\n 256.0 KB | 2 entries | 0.50 MB |\n 416.0 KB | 2 entries | 0.81 MB |\n 464.0 KB | 1 entries | 0.45 MB |\n 496.0 KB | 2 entries | 0.97 MB |\n 768.0 KB | 2 entries | 1.50 MB |\n 1024.0 KB | 2 entries | 2.00 MB |\n 1616.0 KB | 2 entries | 3.16 MB |\n 2176.0 KB | 2 entries | 4.25 MB |\n 8192.0 KB | 36 entries | 288.00 MB | ************************************************************\n 16384.0 KB | 2 entries | 32.00 MB | ******\n154828.8 KB | 1 entries | 151.20 MB | *******************************\n202547.2 KB | 1 entries | 197.80 MB | *****************************************\n\nBug: 514593915\nChange-Id: I9dd87866a5daafe19a2ab3bf5c85da299efd9f5b\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7903258\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1641686}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 65f8573eb4ded78c50b5feefb71e2bc7d22460ff\n" }, { "commit": "66b0df34a721e0f7fe09d1606a92b5e4d909c4a9", "tree": "87d6f15125575c714f26c9d5b2e6ab46782175a6", "parents": [ "5d7cdd26348635470b1f9c5765e8ab37251d70af" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Thu May 14 23:56:32 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri May 15 00:00:20 2026" }, "message": "Reland \"Split PartitionAlloc headers into internal ones and public ones\"\n\nThis is reland of crrev.com/c/7795856. The reason why 7795856 was\nreverted is that allocator_shim_default_dispatch_to_partition_alloc.cc\ndepends on address_pool_manager.h directly, but doesn\u0027t include it\n(indirectly include). So modified\nallocator_shim_default_dispatch_to_partition_alloc.cc to include it.\n\nThe reason why No-Try is that linux-chromeos-rel\u0027s ash_unittests fail\nbecause of timeout. The timeouts seemed to be caused by llvm_profiler\n(code coverage check). If this CL causes the same timeouts after\nlanding, I will revert this CL.\n\nBypass-Check-License: page_allocator_internal.h is just moved, not newly added. No need to check its license.\nNo-Try: true\nChange-Id: I3c21220d90956210110095e70d6ea74931cbddd4\nCq-Include-Trybots: luci.chromium.try:android-cast-arm-rel,android-cast-arm64-rel\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7847692\nReviewed-by: Keishi Hattori \u003ckeishi@chromium.org\u003e\nReviewed-by: Giovanni Ortuno Urquidi \u003cortuno@chromium.org\u003e\nReviewed-by: Lei Zhang \u003cthestig@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1630913}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 7c766eb88a59f73ca3cdf80f4e1bd4ef992ed310\n" }, { "commit": "5d7cdd26348635470b1f9c5765e8ab37251d70af", "tree": "1fbb30df861ebe24a94fcacfa97458765437dbe2", "parents": [ "0c01f49c688a32cd7317728953e2cdac17c53f07" ], "author": { "name": "Andrea Orru", "email": "andreaorru@chromium.org", "time": "Thu May 14 00:05:21 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu May 14 00:09:02 2026" }, "message": "Revert \"Split PartitionAlloc headers into internal ones and public ones\"\n\nThis reverts commit cd836a819c4401c7a455400ac2112d83764060c3.\n\nReason for revert: suspected for tree closure https://ci.chromium.org/ui/p/chromium/builders/ci/android-cast-arm-rel/32082/overview\n\nOriginal change\u0027s description:\n\u003e Split PartitionAlloc headers into internal ones and public ones\n\u003e\n\u003e PS1 \u003d baseline (copy headers into internal/)\n\u003e\n\u003e The reason why No-Try is that linux-chromeos-rel\u0027s ash_unittests fail\n\u003e because of timeout. The timeouts seemed to be caused by llvm_profiler\n\u003e (code coverage check). If this CL causes the same timeouts after\n\u003e landing, I will revert this CL.\n\u003e\n\u003e Bypass-Check-License: page_allocator_internal.h is just moved, not newly added. No need to check its license.\n\u003e No-Try: true\n\u003e Change-Id: I3481802fd95e229b1e287dabd67b4e2490d763e1\n\u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7795856\n\u003e Commit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\n\u003e Reviewed-by: Lei Zhang \u003cthestig@chromium.org\u003e\n\u003e Reviewed-by: Sergei Glazunov \u003cglazunov@google.com\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#1630318}\n\nNo-Presubmit: true\nNo-Tree-Checks: true\nNo-Try: true\nChange-Id: Ie744a441026645b582d9994878bf67426c37cbe6\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7846961\nCommit-Queue: Andrea Orru \u003candreaorru@chromium.org\u003e\nBot-Commit: rubber-stamper@appspot.gserviceaccount.com \u003crubber-stamper@appspot.gserviceaccount.com\u003e\nOwners-Override: Andrea Orru \u003candreaorru@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1630322}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 5e8bd89900b807519c27e9fe24bdf1fc0d86e2fd\n" }, { "commit": "0c01f49c688a32cd7317728953e2cdac17c53f07", "tree": "87d6f15125575c714f26c9d5b2e6ab46782175a6", "parents": [ "b635f27e932356a2e29450e5cfa544cdcc9ea6bb" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Wed May 13 23:51:04 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed May 13 23:54:15 2026" }, "message": "Split PartitionAlloc headers into internal ones and public ones\n\nPS1 \u003d baseline (copy headers into internal/)\n\nThe reason why No-Try is that linux-chromeos-rel\u0027s ash_unittests fail\nbecause of timeout. The timeouts seemed to be caused by llvm_profiler\n(code coverage check). If this CL causes the same timeouts after\nlanding, I will revert this CL.\n\nBypass-Check-License: page_allocator_internal.h is just moved, not newly added. No need to check its license.\nNo-Try: true\nChange-Id: I3481802fd95e229b1e287dabd67b4e2490d763e1\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7795856\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nReviewed-by: Lei Zhang \u003cthestig@chromium.org\u003e\nReviewed-by: Sergei Glazunov \u003cglazunov@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1630318}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: cd836a819c4401c7a455400ac2112d83764060c3\n" }, { "commit": "b635f27e932356a2e29450e5cfa544cdcc9ea6bb", "tree": "1fbb30df861ebe24a94fcacfa97458765437dbe2", "parents": [ "08a72b752a1d37509fcecec692fb1c5ec66b8eaf" ], "author": { "name": "Avi Drissman", "email": "avi@chromium.org", "time": "Wed Jan 28 22:28:28 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jan 28 22:34:19 2026" }, "message": "Use modern names for base::DictValue and base::ListValue\n\nThese classes were hoisted and renamed; switch all usage of these\nclasses to use the new names.\n\nThis is a strictly mechanical change, except for the presubmit.\n\nBug: 478100525\nChange-Id: I1088c6f6f274eac48ce06b766be405d5e326f46e\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7529153\nCommit-Queue: Avi Drissman \u003cavi@chromium.org\u003e\nOwners-Override: Daniel Cheng \u003cdcheng@chromium.org\u003e\nReviewed-by: Daniel Cheng \u003cdcheng@chromium.org\u003e\nCommit-Queue: Daniel Cheng \u003cdcheng@chromium.org\u003e\nAuto-Submit: Avi Drissman \u003cavi@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1576210}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: fbffe55c2d7d156ce726a5850381664e33c1232e\n" }, { "commit": "08a72b752a1d37509fcecec692fb1c5ec66b8eaf", "tree": "cf46197041ac91f67f6201c6a630f3ce05454770", "parents": [ "e028ac649856478983fdca599522f242de568826" ], "author": { "name": "mikt", "email": "mikt@google.com", "time": "Tue Jan 27 08:41:50 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 27 08:48:14 2026" }, "message": "Reland \"[PA] Make PartitionRoot a class\"\n\nThis reverts commit e260315ba2f64b827705e4355891f92920ce3e7d.\n\nReason for revert: Failure fixed; see diff from PS1\n\nOriginal change\u0027s description:\n\u003e Revert \"[PA] Make PartitionRoot a class\"\n\u003e\n\u003e This reverts commit 51c088b1846e62d2709c62139ecb402cf476ed46.\n\u003e\n\u003e Reason for revert: Change is suspected to cause build failures.\n\u003e https://ci.chromium.org/ui/p/chromium/builders/ci/linux-extended-tracing-rel/100068/blamelist\n\u003e\n\u003e Original change\u0027s description:\n\u003e \u003e [PA] Make PartitionRoot a class\n\u003e \u003e\n\u003e \u003e `PartitionRoot` is one of the most complex type within PartitionAlloc,\n\u003e \u003e with non-trivial constructors and a destructor. It is not simple enough\n\u003e \u003e to be a struct, hence changing to a class.\n\u003e \u003e\n\u003e \u003e This change also introduces trailing underscores to its member variable\n\u003e \u003e names.\n\u003e \u003e\n\u003e \u003e Fuchsia-Binary-Size: Pure refactoring.\n\u003e \u003e Change-Id: Id344859356522e7bec6b9207b9cc94819a67eabe\n\u003e \u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7364660\n\u003e \u003e Reviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\n\u003e \u003e Commit-Queue: Mikihito Matsuura \u003cmikt@google.com\u003e\n\u003e \u003e Cr-Commit-Position: refs/heads/main@{#1568957}\n\u003e\n\u003e No-Presubmit: true\n\u003e No-Tree-Checks: true\n\u003e No-Try: true\n\u003e Change-Id: I22c22203ed6f26fdc7fac6fd662d7b7de3c82a82\n\u003e Bug: 7364660\n\u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7461033\n\u003e Reviewed-by: Matthias Körber \u003ckoerber@google.com\u003e\n\u003e Owners-Override: Matthias Körber \u003ckoerber@google.com\u003e\n\u003e Auto-Submit: Matthias Körber \u003ckoerber@google.com\u003e\n\u003e Commit-Queue: Matthias Körber \u003ckoerber@google.com\u003e\n\u003e Bot-Commit: Rubber Stamper \u003crubber-stamper@appspot.gserviceaccount.com\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#1568964}\n\nCq-Include-Trybots: luci.chromium.try:linux-extended-tracing-rel\nBug: 475729821\nChange-Id: I08bfaa4bbfcce9586177d7116ab6c8cdc8b8b48b\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7501136\nCommit-Queue: Mikihito Matsuura \u003cmikt@google.com\u003e\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1575048}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 4c9f31db6b9d8d8ca9d3036be16bd00425e5a2cb\n" }, { "commit": "e028ac649856478983fdca599522f242de568826", "tree": "de956482f0f4bfb7b7d86ca00dc802a061a65a6a", "parents": [ "16429092be4bb5160bed0ad763bd4c13f1f52990" ], "author": { "name": "Avi Drissman", "email": "avi@chromium.org", "time": "Sun Jan 25 20:39:26 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Jan 25 20:44:20 2026" }, "message": "Use modern names for base::DictValue and base::ListValue\n\nThese classes were hoisted and renamed; switch all usage of these\nclasses to use the new names.\n\nThis is a strictly mechanical change.\n\nBug: 478100525\nChange-Id: Ib24d1f98695415ff9b8d2d6eefe69323d8473b12\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7509300\nReviewed-by: Daniel Cheng \u003cdcheng@chromium.org\u003e\nAuto-Submit: Avi Drissman \u003cavi@chromium.org\u003e\nCommit-Queue: Daniel Cheng \u003cdcheng@chromium.org\u003e\nOwners-Override: Daniel Cheng \u003cdcheng@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1574260}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: e99429c7513db8b3f278f15b5ae11d9f402f0d52\n" }, { "commit": "16429092be4bb5160bed0ad763bd4c13f1f52990", "tree": "3335d7f6fceaee7584cc674fd7efcc9c9c1bc512", "parents": [ "fadcf286230c8d51283c330603f5ffaadbc86a0b" ], "author": { "name": "Matthias Körber", "email": "koerber@google.com", "time": "Wed Jan 14 09:19:42 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jan 14 09:26:38 2026" }, "message": "Revert \"[PA] Make PartitionRoot a class\"\n\nThis reverts commit 51c088b1846e62d2709c62139ecb402cf476ed46.\n\nReason for revert: Change is suspected to cause build failures.\nhttps://ci.chromium.org/ui/p/chromium/builders/ci/linux-extended-tracing-rel/100068/blamelist\n\nOriginal change\u0027s description:\n\u003e [PA] Make PartitionRoot a class\n\u003e\n\u003e `PartitionRoot` is one of the most complex type within PartitionAlloc,\n\u003e with non-trivial constructors and a destructor. It is not simple enough\n\u003e to be a struct, hence changing to a class.\n\u003e\n\u003e This change also introduces trailing underscores to its member variable\n\u003e names.\n\u003e\n\u003e Fuchsia-Binary-Size: Pure refactoring.\n\u003e Change-Id: Id344859356522e7bec6b9207b9cc94819a67eabe\n\u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7364660\n\u003e Reviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\n\u003e Commit-Queue: Mikihito Matsuura \u003cmikt@google.com\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#1568957}\n\nNo-Presubmit: true\nNo-Tree-Checks: true\nNo-Try: true\nChange-Id: I22c22203ed6f26fdc7fac6fd662d7b7de3c82a82\nBug: 7364660\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7461033\nReviewed-by: Matthias Körber \u003ckoerber@google.com\u003e\nOwners-Override: Matthias Körber \u003ckoerber@google.com\u003e\nAuto-Submit: Matthias Körber \u003ckoerber@google.com\u003e\nCommit-Queue: Matthias Körber \u003ckoerber@google.com\u003e\nBot-Commit: Rubber Stamper \u003crubber-stamper@appspot.gserviceaccount.com\u003e\nCr-Commit-Position: refs/heads/main@{#1568964}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: e260315ba2f64b827705e4355891f92920ce3e7d\n" }, { "commit": "fadcf286230c8d51283c330603f5ffaadbc86a0b", "tree": "18b9a2225e6dd3b5ac64aee0bc072d971d417b46", "parents": [ "60d6f6fb67336f5b5060c6e1d0a72a32432423b5" ], "author": { "name": "mikt", "email": "mikt@google.com", "time": "Wed Jan 14 08:42:23 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jan 14 08:50:55 2026" }, "message": "[PA] Make PartitionRoot a class\n\n`PartitionRoot` is one of the most complex type within PartitionAlloc,\nwith non-trivial constructors and a destructor. It is not simple enough\nto be a struct, hence changing to a class.\n\nThis change also introduces trailing underscores to its member variable\nnames.\n\nFuchsia-Binary-Size: Pure refactoring.\nChange-Id: Id344859356522e7bec6b9207b9cc94819a67eabe\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7364660\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nCommit-Queue: Mikihito Matsuura \u003cmikt@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1568957}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 51c088b1846e62d2709c62139ecb402cf476ed46\n" }, { "commit": "60d6f6fb67336f5b5060c6e1d0a72a32432423b5", "tree": "3335d7f6fceaee7584cc674fd7efcc9c9c1bc512", "parents": [ "d500250668368512cf2b0873c7727f034c00ce7a" ], "author": { "name": "Jan Keitel", "email": "jkeitel@google.com", "time": "Fri Jan 09 05:34:30 2026" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Jan 09 05:42:08 2026" }, "message": "Update file API calls to use span-based versions in pa_tcache_inspect\n\nNo functionality change.\n\nBug: 435317390\nChange-Id: Iefeb76a43d35d72adc40c8f46f7549bfa7f2d655\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7212362\nAuto-Submit: Jan Keitel \u003cjkeitel@google.com\u003e\nCommit-Queue: Keishi Hattori \u003ckeishi@chromium.org\u003e\nReviewed-by: Keishi Hattori \u003ckeishi@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1566735}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 2291f6db764157f58abd3a4f83c6213b0fcf8c9d\n" }, { "commit": "d500250668368512cf2b0873c7727f034c00ce7a", "tree": "120c6621ce1b489fe44f84888a379f51b19bf3a8", "parents": [ "299412a3264f857e795082bd8c9aa3eb1d706e38" ], "author": { "name": "Jan Keitel", "email": "jkeitel@google.com", "time": "Tue Dec 02 18:45:40 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Dec 02 18:51:06 2025" }, "message": "Use span-based file API in inspect_utils_linux.cc\n\nNo functionality change.\n\nBug: 435317390\nChange-Id: I133a61d17ea1aa86874b77f67ce30452965dc63e\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7212363\nCommit-Queue: Lei Zhang \u003cthestig@chromium.org\u003e\nAuto-Submit: Jan Keitel \u003cjkeitel@google.com\u003e\nReviewed-by: Lei Zhang \u003cthestig@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1552946}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 4f69e2c14be55c4f603bd87996aa1ee6319d4234\n" }, { "commit": "299412a3264f857e795082bd8c9aa3eb1d706e38", "tree": "81de3032b131936a5c0d63ab77b48affe59118c6", "parents": [ "27e942fcc0c46109be1cf02d1257784115974c9f" ], "author": { "name": "Lei Zhang", "email": "thestig@chromium.org", "time": "Wed Nov 26 01:32:26 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Nov 26 01:37:29 2025" }, "message": "Remove includes of callback_forward.h from .cc files\n\nGenerally, .cc files should not be forward declaring base::OnceCallback\nand related classes. If they are using Callbacks, they should include\ncallback.h instead. So treat the base/functional/callback_forward.h\nincludes as superfluous and remove them.\n\nBug: 40318405\nChange-Id: I487c907e520d03be3db5b03a8757c7f2559c8278\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7201005\nCommit-Queue: Lei Zhang \u003cthestig@chromium.org\u003e\nOwners-Override: Kyle Charbonneau \u003ckylechar@chromium.org\u003e\nReviewed-by: Kyle Charbonneau \u003ckylechar@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1550173}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 16723c112a9691d3819d7117e2faf2aeecf1891a\n" }, { "commit": "27e942fcc0c46109be1cf02d1257784115974c9f", "tree": "f2f43ab6fe32be2b47f43c878ddf28e759627989", "parents": [ "5ca6db8bfa616814f6b66b2413fdb54779180d9f" ], "author": { "name": "Lei Zhang", "email": "thestig@chromium.org", "time": "Fri Nov 21 03:40:04 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Nov 21 03:46:04 2025" }, "message": "Reland \"Move logging::LoggingSettings to base/logging/logging_settings.h\"\n\nThis is a reland of commit 6b79baf89a1d87407d4bafdc5d871061a81f80fc\n\nThe reland is the same CL. The difference is the problematic code in a\nseparate repo that did not do IWYU has been fixed.\n\nOriginal change\u0027s description:\n\u003e Move logging::LoggingSettings to base/logging/logging_settings.h\n\u003e\n\u003e Move struct LoggingSettings, which is only used in a relatively small\n\u003e number of files, out of base/logging.h and into its own header. Update\n\u003e direct users and then do IWYU to fix the build. This effectively moves\n\u003e base/files/file_path.h out of base/logging.h.\n\u003e\n\u003e Change-Id: I04209f1063051b7399e335d7e30cb7abb9442c94\n\u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7160150\n\u003e Reviewed-by: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\n\u003e Commit-Queue: Lei Zhang \u003cthestig@chromium.org\u003e\n\u003e Owners-Override: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#1546655}\n\nChange-Id: Ie877c3c85cc3e1983fb8b2b065b0d09607442ce4\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7173024\nOwners-Override: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\nReviewed-by: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\nCommit-Queue: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1548242}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: d7fb9d0cd0b6e1589ee974c1ca33de984df513e7\n" }, { "commit": "5ca6db8bfa616814f6b66b2413fdb54779180d9f", "tree": "e0a60f8db164582b4c901434786a090fcfc6d3cf", "parents": [ "6fa90aee8c8352bd29689262ddb2420fe85e1f2c" ], "author": { "name": "Lei Zhang", "email": "thestig@chromium.org", "time": "Wed Nov 19 02:46:22 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Nov 19 02:54:15 2025" }, "message": "Revert \"Move logging::LoggingSettings to base/logging/logging_settings.h\"\n\nThis reverts commit 6b79baf89a1d87407d4bafdc5d871061a81f80fc.\n\nReason for revert: Broke win-chrome\n\nOriginal change\u0027s description:\n\u003e Move logging::LoggingSettings to base/logging/logging_settings.h\n\u003e\n\u003e Move struct LoggingSettings, which is only used in a relatively small\n\u003e number of files, out of base/logging.h and into its own header. Update\n\u003e direct users and then do IWYU to fix the build. This effectively moves\n\u003e base/files/file_path.h out of base/logging.h.\n\u003e\n\u003e Change-Id: I04209f1063051b7399e335d7e30cb7abb9442c94\n\u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7160150\n\u003e Reviewed-by: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\n\u003e Commit-Queue: Lei Zhang \u003cthestig@chromium.org\u003e\n\u003e Owners-Override: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#1546655}\n\nNo-Presubmit: true\nNo-Tree-Checks: true\nNo-Try: true\nChange-Id: I7cf4167499823ba368fcfab88c930d1d98455fa0\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7170750\nBot-Commit: Rubber Stamper \u003crubber-stamper@appspot.gserviceaccount.com\u003e\nCommit-Queue: Lei Zhang \u003cthestig@chromium.org\u003e\nAuto-Submit: Lei Zhang \u003cthestig@chromium.org\u003e\nOwners-Override: Lei Zhang \u003cthestig@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1546935}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 6201eb11311d419ae1f44e85fff590ec69156cf5\n" }, { "commit": "6fa90aee8c8352bd29689262ddb2420fe85e1f2c", "tree": "f2f43ab6fe32be2b47f43c878ddf28e759627989", "parents": [ "3c7b1f4daab1520239cb172059e2e16684fd3128" ], "author": { "name": "Lei Zhang", "email": "thestig@chromium.org", "time": "Tue Nov 18 18:20:35 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Nov 18 18:27:27 2025" }, "message": "Move logging::LoggingSettings to base/logging/logging_settings.h\n\nMove struct LoggingSettings, which is only used in a relatively small\nnumber of files, out of base/logging.h and into its own header. Update\ndirect users and then do IWYU to fix the build. This effectively moves\nbase/files/file_path.h out of base/logging.h.\n\nChange-Id: I04209f1063051b7399e335d7e30cb7abb9442c94\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7160150\nReviewed-by: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\nCommit-Queue: Lei Zhang \u003cthestig@chromium.org\u003e\nOwners-Override: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1546655}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 6b79baf89a1d87407d4bafdc5d871061a81f80fc\n" }, { "commit": "3c7b1f4daab1520239cb172059e2e16684fd3128", "tree": "e0a60f8db164582b4c901434786a090fcfc6d3cf", "parents": [ "14058fc042c2dca428d7e65c537c91a4a0adda9b" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Mon Jul 28 05:47:05 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Jul 28 05:52:37 2025" }, "message": "[PA] Move Metadata outside of GigaCage (6/7)\n\nProvide `root` or `metadata offset` for the methods to obtain metadata,\ne.g. SlotSpanMetadata::FromObject(), FromAddr(), and ToSlotStart(), and\nremove default parameter: `root \u003d nullptr` from the methods.\n\nRemove unused `root` from SlotSpanMetadata::SetFreelistHead(), and\nPartitionBucket::SetNewActiveSlotSpan(),\n\nDesign Doc:\nhttps://docs.google.com/document/d/1Mn-qRAWuDhS_gRL_OG2TYL-EW5D8yljaCyM40S0ocVc/edit?tab\u003dt.0\n\nBug: crbug.com/40238514\nChange-Id: Ie8857e47fe55b53ff7c3e0f64300fed26cbf06f7\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6758538\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1492598}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 000352b61a6701c4014ec823d0099bb3d1be8953\n" }, { "commit": "14058fc042c2dca428d7e65c537c91a4a0adda9b", "tree": "01c4fccf78370626efb2eb8035aa84f8b3fa710d", "parents": [ "2ea414c50e21048ff88faff869b41df2bbf9ab3c" ], "author": { "name": "Lei Zhang", "email": "thestig@chromium.org", "time": "Fri Jul 25 17:30:45 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Jul 25 17:37:22 2025" }, "message": "Auto-format //tools GN files\n\nRun \"gn format\" and automatically shorten target references, now that GN\nknows how to do this.\n\nBug: 40760278\nChange-Id: Idf7709e04ed1787e9c1246251402d406291763c7\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6785953\nCommit-Queue: Jesse McKenna \u003cjessemckenna@google.com\u003e\nAuto-Submit: Lei Zhang \u003cthestig@chromium.org\u003e\nReviewed-by: Jesse McKenna \u003cjessemckenna@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1492161}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 1631e78d6d308eb025afc432c1569b32b9a7387f\n" }, { "commit": "2ea414c50e21048ff88faff869b41df2bbf9ab3c", "tree": "5faf392b955f2afa9092a0fb7a32afdbcbb486cb", "parents": [ "7f4bdaac14afec7c4fff3c9c88194c4a8f49ab25" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Wed Jul 23 01:23:01 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jul 23 01:29:41 2025" }, "message": "[PA] Move Metadata outside of GigaCage (3/5)\n\nRevert `template \u003cMetadataKind\u003e SlotSpanMetadata`.\n\nDesignDoc:\nhttps://docs.google.com/document/d/1Mn-qRAWuDhS_gRL_OG2TYL-EW5D8yljaCyM40S0ocVc/edit?tab\u003dt.0\n\nBug: crbug.com/40238514\nChange-Id: I2d5d4642d754bca97edf2d04d263938ce620cf8e\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6758537\nReviewed-by: Sergei Glazunov \u003cglazunov@google.com\u003e\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nReviewed-by: Keishi Hattori \u003ckeishi@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1490549}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 133753cd6a1c5a028f8fad9834204260975a859a\n" }, { "commit": "7f4bdaac14afec7c4fff3c9c88194c4a8f49ab25", "tree": "bb2220c08a458a0fa5f6caefd4e6c2cdf57d58a0", "parents": [ "cc38b4b04fbf942926366f420c43afb9e9ab364b" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Fri Jul 18 07:58:14 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Jul 18 08:18:30 2025" }, "message": "[PA] Move Metadata outside of GigaCage (2/5)\n\nRemove MetadataKind from SuperPageExtentEntry, DirectMapExtentEntry and\nPartitionPageMetadata. Make SlotSpanMetadata\u003cMetadataKind\u003e an alias of\nSlotSpanMetadataBase.\n\nDesignDoc:\nhttps://docs.google.com/document/d/1Mn-qRAWuDhS_gRL_OG2TYL-EW5D8yljaCyM40S0ocVc/edit?tab\u003dt.0\n\nBug: crbug.com/40238514\nChange-Id: I4c0dc91fc7d9bc1751427041724db60103267762\nValidate-Test-Flakiness: skip\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6758536\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1488691}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 1f1efda4e2c325753c786acbb77ee7b9d9f6e6ec\n" }, { "commit": "cc38b4b04fbf942926366f420c43afb9e9ab364b", "tree": "5aef0d54617670368685d5afc648d8613e441247", "parents": [ "14089a7f57fa80c58e8604efef2134d542e7134b" ], "author": { "name": "mikt", "email": "mikt@google.com", "time": "Fri May 02 04:35:46 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri May 02 04:42:30 2025" }, "message": "[PA] Simplify bucket lookup\n\nCurrent implementation requires to construct a lookup table, and memory\naccess happens every time. This CL rewrites it into calculation within\nregisters.\n\nAs side-effect, this allowed to eliminate some unused buckets in\n`PartitionRoot`, simplifying its logic.\n\nThis CL also removes some unused configuration constants. The other\nconstants are relocated into `BucketIndexLookup` class, to make it as a\npart of PartitionAlloc public API.\n\nSee buckets.md for the new design details.\n\nThis CL should have no behavior change, and should not regress\nperformance.\n\nChange-Id: Ib6c9d7c95918397941d27f43b1813cbe0348079f\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6490617\nCommit-Queue: Mikihito Matsuura \u003cmikt@google.com\u003e\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nReviewed-by: Stephen Nusko \u003cnuskos@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1454826}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 169e9acf5be3b9c6cf3bb0af5fdd11ea444d073b\n" }, { "commit": "14089a7f57fa80c58e8604efef2134d542e7134b", "tree": "d155058a693ebe6cfad06244a375abccb501021c", "parents": [ "8e9b58419b41fe0d9dfe2e870f3164f446a0d7ab" ], "author": { "name": "Leszek Swirski", "email": "leszeks@google.com", "time": "Thu Feb 06 12:16:57 2025" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Feb 06 12:35:34 2025" }, "message": "[base] Use wrapper type for PlatformThreadId\n\nIntroduce a wrapper class (similar to base::StrongAlias) for\nPlatformThreadId, whose API assumes that the value can be 64-bit. It\nforces explicit conversion to integers with various bitness, and makes\nconversion to 32-bit be explicit about truncation.\n\nThe values are currently all still 32-bit -- follow-up work will make\nthread id values on macOS 64-bit.\n\nBug: 40187449\nChange-Id: Ic635c346be1c3a0d62fd573572f4fcdc642a734f\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6206469\nReviewed-by: Peter McNeeley \u003cpetermcneeley@google.com\u003e\nReviewed-by: Mark Mentovai \u003cmark@chromium.org\u003e\nOwners-Override: Mark Mentovai \u003cmark@chromium.org\u003e\nAuto-Submit: Leszek Swirski \u003cleszeks@chromium.org\u003e\nReviewed-by: Victor Vianna \u003cvictorvianna@google.com\u003e\nCommit-Queue: Leszek Swirski \u003cleszeks@chromium.org\u003e\nReviewed-by: Mikhail Khokhlov \u003ckhokhlov@google.com\u003e\nReviewed-by: Ahmed Fakhry \u003cafakhry@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1416712}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: b3fb42320c2b4354477bf76aab232336f2961eee\n" }, { "commit": "8e9b58419b41fe0d9dfe2e870f3164f446a0d7ab", "tree": "dbc3e64e745842380c7d03b3c65a783ea5c806ba", "parents": [ "8385f10496532b425c4036d7e938393f683e5be9" ], "author": { "name": "Alex Attar", "email": "aattar@google.com", "time": "Fri Dec 06 13:20:57 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Dec 06 13:27:32 2024" }, "message": "Use using enum for JsonOptions (C++20)\n\nThis CL updates the JsonWriter class to use the `using enum` declaration\nfor JsonOptions, which is a C++20 feature. This removes the need for the\nprevious workaround that manually defined static constexpr variables for\neach enum value.\n\nChange-Id: Id9a567f5e52cefca217cd14c03ceb5a53143aa18\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6072783\nReviewed-by: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\nCommit-Queue: Alex Attar \u003caattar@google.com\u003e\nReviewed-by: Kentaro Hara \u003charaken@chromium.org\u003e\nReviewed-by: Greg Thompson \u003cgrt@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1392849}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 88cdc6d086bdcf6a6fb1f14de4af7a0f72356706\n" }, { "commit": "8385f10496532b425c4036d7e938393f683e5be9", "tree": "3d7fd8a887dbed7d76a858b8a0ab0f0534e004c4", "parents": [ "a7bb45e874cd595c082fdbf9697afe3b49dcb9ae" ], "author": { "name": "Tom Sepez", "email": "tsepez@chromium.org", "time": "Tue Sep 17 17:54:33 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Sep 17 18:04:38 2024" }, "message": "Avoid unsafe forms for base::File::WriteAtCurrentPos()\n\nUse the span based forms instead. Adjust callers to accommodate\nan optional\u003csize_t\u003e return.\n\nThis CL was uploaded by git cl split.\n\nR\u003dlizeb@chromium.org, olivierli@chromium.org, pasko@chromium.org\n\nBug: 42271176\nChange-Id: Ia97077b38293bd6713b5afa69641c771b74db260\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5819672\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nReviewed-by: Egor Pasko \u003cpasko@chromium.org\u003e\nAuto-Submit: Tom Sepez \u003ctsepez@chromium.org\u003e\nCommit-Queue: Olivier Li \u003colivierli@chromium.org\u003e\nReviewed-by: Olivier Li \u003colivierli@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1356595}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 5294874506edca885248dc8097ab0678a39461fe\n" }, { "commit": "a7bb45e874cd595c082fdbf9697afe3b49dcb9ae", "tree": "1d5551b6de9012073771991c61e575ebe2660442", "parents": [ "bd3ae34fa1ef7cf9510cae571bd27dd09600f321" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Mon Sep 02 23:57:20 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Sep 03 00:03:40 2024" }, "message": "[PA] Split SlotSpanMetadata into read-only one and writable one.\n\nThis is a part of ShadowMetadata CL (4/5). The entire CL is\ncrrev.com/c/5570783.\n\nSlotSpanMetadata\u003ckReadOnly\u003e is used for SlotSpanMetadata inside\nthe giga cage, and SlotSpanMetadata\u003ckWritable\u003e is used for\nSlotSpanMetadata inside the shadow pool.\n\nPS18 \u003d baseline.\n\nThe design doc of shadow metadata is\nhttps://docs.google.com/document/d/1LvMI_c1VuT4sWUqgKOdJlrZv0UWP5e3PLJ_oOlJ4l2k\n\nBug: 40238514\nChange-Id: I979a55bb35c0c186d882334931219dea80bb6538\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5684977\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nReviewed-by: Ali Hijazi \u003cahijazi@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1349950}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: bc702f6fd09bd8da20a48e7f4d8b13f8e728245b\n" }, { "commit": "bd3ae34fa1ef7cf9510cae571bd27dd09600f321", "tree": "e018beda6fd82fff746022761fbf447fc5935394", "parents": [ "d7a1bee91c1e4927b0f4d35f26aecb4cafa7df8c" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Wed Aug 21 03:10:53 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Aug 21 03:18:38 2024" }, "message": "[PA] Split PartitionPageMetadata into read-only one and writable one.\n\nThis is a part of ShadowMetadata CL (3/5). The entire CL is\ncrrev.com/c/5570783.\n\nReadOnlyPartitionPageMetadata is used for PartitionPageMetadata\ninside the giga cage, and WritablePartitionPageMetadata is used\nfor PartitionPageMetadata inside the shadow pool.\n\nThe design doc of shadow metadata is\nhttps://docs.google.com/document/d/1LvMI_c1VuT4sWUqgKOdJlrZv0UWP5e3PLJ_oOlJ4l2k\n\nBug: 40238514\nChange-Id: If11b168c3f371964c131528fcc92b365face2379\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5684976\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1344569}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 61f529a935b9053b0172c980a26e5389daf29aef\n" }, { "commit": "d7a1bee91c1e4927b0f4d35f26aecb4cafa7df8c", "tree": "b44cf5f81b5ebb694c761d1f09240b50599e0ede", "parents": [ "5a2fc6d03abe14d800811f42578872992c37c0dd" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Tue Aug 06 03:23:06 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Aug 06 03:32:19 2024" }, "message": "[PA] Split PartitionSuperPageExtentEntry into read-only one and writable one.\n\nThis is a part of ShadowMetadata CL (2/5). The entire CL is\ncrrev.com/c/5570783.\n\nReadOnlyPartitionSuperPageExtentEntry is used for\nPartitionSuperPageExtentEntry inside the giga cage, and\nWritablePartitionSuperPageExtentEntry is used for\nPartitionSuperPageExtentEntry inside the shadow pool.\n\nThe design doc of shadow metadata is\nhttps://docs.google.com/document/d/1LvMI_c1VuT4sWUqgKOdJlrZv0UWP5e3PLJ_oOlJ4l2k\n\nBug: 40238514\nNo-Try: true\nChange-Id: Ib06c7743701b98e168c49781d2d933789a144590\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5684975\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1337653}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 7cddc390c898096526a3911df8b97ac6bfd14bd8\n" }, { "commit": "5a2fc6d03abe14d800811f42578872992c37c0dd", "tree": "0588ee5386525acab05b81f09cd5e12bb35b4cdd", "parents": [ "87928d7073addb37a8183687a7d8446abf3c6ee4" ], "author": { "name": "mikt", "email": "mikt@google.com", "time": "Mon Jul 29 20:43:02 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Jul 29 20:51:15 2024" }, "message": "[PA] Update DEPS for new include path #cleanup\n\nBefore this CL,\n\n- `//DEPS` allows including `base/` headers\n- `//base/DEPS` allows including `partition_alloc` headers, while\n banning `base/allocator/partition_alloc`.\n\nThis forced us to use\n\n- `#include \"partition_alloc/*\"` in `//base`\n- `#include \"base/allocator/partition_alloc/src/partition_alloc/*\"`\n elsewhere\n\nand the rule was inconsistent.\n\nThis CL applies the DEP include rule in `//base` chrome-wide so that\nwe can only use `#include \"partition_alloc/*\"` style includes.\n\nAt the same time, all `#include` with the old style are rewritten.\nThis need to be done in the single CL to avoid DEP rule check errors.\n\nBug: 40276913\nChange-Id: I1410888184df71ec3bd6374d58d876740357272c\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5745369\nCommit-Queue: Mikihito Matsuura \u003cmikt@google.com\u003e\nReviewed-by: Rick Byers \u003crbyers@chromium.org\u003e\nReviewed-by: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nOwners-Override: Rick Byers \u003crbyers@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1334435}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: dc1d5a68d4e3cabecd33e1258d5902f7d9fc78dd\n" }, { "commit": "87928d7073addb37a8183687a7d8446abf3c6ee4", "tree": "24ce7c678454ea332beb29f173f2cdbce25df444", "parents": [ "a0eeba1c75aba820a482a8847946dae6f9078281" ], "author": { "name": "Helmut Januschka", "email": "helmut@januschka.com", "time": "Thu Jul 11 17:18:55 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jul 11 17:28:36 2024" }, "message": "Remove unneeded StringPiece header usage\n\nBug: 40506050\nChange-Id: I15636e78efea9243ad7c13c9e91e82d683f70a35\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5688848\nOwners-Override: Lei Zhang \u003cthestig@chromium.org\u003e\nAuto-Submit: Helmut Januschka \u003chelmut@januschka.com\u003e\nCommit-Queue: Helmut Januschka \u003chelmut@januschka.com\u003e\nReviewed-by: Lei Zhang \u003cthestig@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1326238}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 5452f216fc8465fcb0a20970e617edcbf897d1ed\n" }, { "commit": "a0eeba1c75aba820a482a8847946dae6f9078281", "tree": "a5376d58f8b98bbe9a3b7193e52961c7967132d5", "parents": [ "4ac80c5c63dc6072c0fee9439b43121c64e1301b" ], "author": { "name": "Arthur Sonzogni", "email": "arthursonzogni@chromium.org", "time": "Wed Jun 26 08:16:23 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jun 26 08:22:32 2024" }, "message": "partition_alloc: merge buildflags.h #cleanup\n\npartition_alloc has 5 different buildflags headers, but there isn\u0027t a\nneed to maintain multiple:\n- All of them were anyway grouped under the \"buildflags\" target.\n- All of them are included in every files via the `raw_ptr.h` file.\n\nTo keep things simpler to maintain, this merges them all.\n\nThis is #cleanup.\n\nWhy No-Try: It seems touching some PA files causes the CQ to run too\nmany tests and they are slower on the CQ due to option gathering code\ncoverage. As an example, adding empty lines causes the same failure:\nhttps://chromium-review.googlesource.com/c/chromium/src/+/5513777\nFeel free to revert this patch if some a bots fails two consecutive\nrun.\n\nNo-try: true\nBug: 41481467\nChange-Id: I56a30e5bb453c19a8d262f7a19b3d76ecae23a26\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5630428\nReviewed-by: Keishi Hattori \u003ckeishi@chromium.org\u003e\nOwners-Override: Keishi Hattori \u003ckeishi@chromium.org\u003e\nCommit-Queue: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1319627}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: fd39d619c7eadd934eb46f2d9d05821e71c9d491\n" }, { "commit": "4ac80c5c63dc6072c0fee9439b43121c64e1301b", "tree": "90ccabb0de3993746831c8fb00df7a66725258e1", "parents": [ "c5b623b3ba22e626ff51ab4f0b8d2bde749e970f" ], "author": { "name": "Helmut Januschka", "email": "helmut@januschka.com", "time": "Tue Jun 11 13:05:35 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jun 11 13:28:08 2024" }, "message": "Convert base::StringPiece to std::string_view left overs.\n\nThe changes of this CL are made using the following script.\nScript: https://issues.chromium.org/issues/40506050#comment343\n\nBug: 40506050\nChange-Id: I768b3590141a698385319a615f2aefb71215ec84\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5601472\nReviewed-by: Nico Weber \u003cthakis@chromium.org\u003e\nAuto-Submit: Helmut Januschka \u003chelmut@januschka.com\u003e\nReviewed-by: Daniel Cheng \u003cdcheng@chromium.org\u003e\nCommit-Queue: Nico Weber \u003cthakis@chromium.org\u003e\nOwners-Override: Nico Weber \u003cthakis@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1313356}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 1dce9dcaeaa1b6b2a278869566f7554c62b10dc7\n" }, { "commit": "c5b623b3ba22e626ff51ab4f0b8d2bde749e970f", "tree": "394472c13436d60d109050c938b19df3819c786c", "parents": [ "2448d81f9741a5f791b1515893fc18c0999970e7" ], "author": { "name": "David Benjamin", "email": "davidben@chromium.org", "time": "Tue May 28 15:36:47 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 28 15:47:36 2024" }, "message": "Tag easy suppressions with their sanitizer\n\nThese ignorelists are associated with a single sanitizer so, apart from\noversuppressions sneaking through, this one should be fairly\nstraightforward.\n\nBug: 342214574\nChange-Id: If9ac2d3abbfff07c744e9f46ef2edd4ab5b425fd\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5559453\nAuto-Submit: David Benjamin \u003cdavidben@chromium.org\u003e\nCommit-Queue: David Benjamin \u003cdavidben@chromium.org\u003e\nReviewed-by: Nico Weber \u003cthakis@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1306775}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 3626b52ae221cdc0f68e138859f548ba44a24f6d\n" }, { "commit": "2448d81f9741a5f791b1515893fc18c0999970e7", "tree": "6ff70ad6d275349fdec379409d65c4ce1e392145", "parents": [ "c6d669e13804ead0066fdd88e653dd4f9af7b0c1" ], "author": { "name": "Dan McArdle", "email": "dmcardle@chromium.org", "time": "Mon Apr 15 19:09:00 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Apr 15 19:16:11 2024" }, "message": "base/logging: Prevent test-only crash caused by std::string(nullptr)\n\nIf logging is not set up properly, ~ScopedLoggingSettings() can make a\ncall to InitLogging() that constructs a std::string from nullptr. This\nundefined behavior would occur when BaseInitLoggingImpl() sets the value\nof `*g_log_file_name`.\n\nThis can occur in at least one scenario: when the logging destination is\nLOG_TO_FILE, but no log file is set.\n\nI think this caused a crash in sql_recovery_lpm_fuzzer on Windows (see\nlinked bug). However, a little more UB may also have been involved\nbecause the fuzzer does not call InitLogging().\n\nTo prevent this undefined behavior from occuring, this CL makes the\nfollowing changes:\n\n* It changes the type of LoggingSettings::log_file_path from a pointer\n to a string type to prevent nullptr-related accidents in the future.\n\n* It changes ScopedLoggingSettings::log_file_name_ from a\n std::unique_ptr\u003c\u003e of a string type to just a plain string type.\n\n* It upgrades some DCHECKs to CHECKs in BaseInitLoggingImpl(). Note that\n the old `DCHECK(settings.log_file_path)` would have been sufficient to\n prevent the UB in question, but we observed the crash in a fuzzer\n where DCHECKs were not enabled.\n\nBug: 331909454\nChange-Id: I27abdb3eac6fec82853a7643643b2dacd7f7571c\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5416213\nReviewed-by: danakj \u003cdanakj@chromium.org\u003e\nOwners-Override: danakj \u003cdanakj@chromium.org\u003e\nCommit-Queue: Dan McArdle \u003cdmcardle@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1287534}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: b4b65e3b7219a0150106e6798d7d75b401aec1d0\n" }, { "commit": "c6d669e13804ead0066fdd88e653dd4f9af7b0c1", "tree": "64ad48a9d99763c8a8f1b30041161fe8ed9663de", "parents": [ "25e18b2750e5bb9c36177c7aa76a0b1aec058fa5" ], "author": { "name": "Kalvin Lee", "email": "kdlee@chromium.org", "time": "Mon Mar 11 02:30:57 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Mar 11 02:38:45 2024" }, "message": "PA: Prefer just \"PartitionAlloc\"\n\nCommon parlance has favored \"PartitionAlloc\" for quite a while now. This\nbest-effort CL was constructed from a naive `git grep` to replace other\nspellings (e.g. \"partition alloc,\" \"Partition Alloc,\" \"partition\nallocator,\" etc.) with the most common one.\n\nThis change was created with `git cl split` and is scoped to\n/tools/memory/partition_allocator.\n\nThis CL was uploaded by git cl split.\n\nR\u003dlizeb@chromium.org\n\nChange-Id: If4d2875442ca6353c1697a048e3dc51bdc001c38\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5353573\nAuto-Submit: Kalvin Lee \u003ckdlee@chromium.org\u003e\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1270774}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: ea48e4164402829c964c127265e4336ec596ac32\n" }, { "commit": "25e18b2750e5bb9c36177c7aa76a0b1aec058fa5", "tree": "5fa621515f13ba0826496ac3054a7a611ae416eb", "parents": [ "4753a8cbcd7ca9b8624bd909f915b1c3c7edebea" ], "author": { "name": "mikt", "email": "mikt@google.com", "time": "Thu Mar 07 07:10:47 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Mar 07 07:18:12 2024" }, "message": "[PA] Refactor: rename RefCount to InSlotMetadata\n\nTo support growing demands for in-slot metadata other than BRP\u0027s\nreference count, this CL renames `PartitionRefCount` to\n`InSlotMetadata` and update related symbol names and comments.\n\nChange-Id: I7dcaee714e33ebb67c4f4dd7082f39e8f1124484\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5331170\nCommit-Queue: Mikihito Matsuura \u003cmikt@google.com\u003e\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1269432}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: c986477cc62fc00827976dc655426d7649a00228\n" }, { "commit": "4753a8cbcd7ca9b8624bd909f915b1c3c7edebea", "tree": "b8738bd894a1f250e78ae3400f72e1febf1f5264", "parents": [ "af6dc8ec08b0f3ddc874bdeacbf87a18115c2748" ], "author": { "name": "Bartek Nowierski", "email": "bartekn@chromium.org", "time": "Wed Jan 17 03:03:11 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jan 17 03:10:28 2024" }, "message": "[BRP] Finchify \"same slot\" mode\n\nIn the current implementation, the only difference between \"same slot\"\nand \"previous slot\" modes is end of which slot we look for ref-count.\nThis means that when MTE is on, we still bloat up ref_count_size to\nthe granule size and skip tagging it, even though it is no longer\nnecessary in the \"same slot\" mode. In the final code, we will no\nlonger do that, but we have to make sure that calculating the\nref-count address doesn\u0027t lose the tag.\n\na finch flag, so the coverage should be the same)\n\nLow-Coverage-Reason: OTHER (this merely converts a buildflag into\nBug: 1511221, 1445816\nChange-Id: I92efd626c0f57f9ae34166e229433a476cbe2d0d\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5180030\nReviewed-by: Keishi Hattori \u003ckeishi@chromium.org\u003e\nCommit-Queue: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1247929}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 89ad25d1d82b660bc36caa14319ff65696b65580\n" }, { "commit": "af6dc8ec08b0f3ddc874bdeacbf87a18115c2748", "tree": "27221fea55b12cf0b68a270d14756ea9b72ce4bb", "parents": [ "137d838b1db2a7f4f83171a95654b4984e11283e" ], "author": { "name": "Arthur Sonzogni", "email": "arthursonzogni@chromium.org", "time": "Fri Jan 12 08:44:59 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Jan 12 08:59:56 2024" }, "message": "Reland \"Rename {absl \u003d\u003e std}::optional in //tools/\"\n\nThis is a reland of commit b263d258c9e4899afaa24036bde76d7e9e1d537c\n\nOriginal change\u0027s description:\n\u003e Rename {absl \u003d\u003e std}::optional in //tools/\n\u003e\n\u003e Automated patch, intended to be effectively a no-op.\n\u003e\n\u003e Context:\n\u003e https://groups.google.com/a/chromium.org/g/cxx/c/nBD_1LaanTc/m/ghh-ZZhWAwAJ?utm_medium\u003demail\u0026utm_source\u003dfooter\n\u003e\n\u003e As of https://crrev.com/1204351, absl::optional is now a type alias for\n\u003e std::optional. We should migrate toward it.\n\u003e\n\u003e Script:\n\u003e ```\n\u003e function replace {\n\u003e echo \"Replacing $1 by $2\"\n\u003e git grep -l \"$1\" \\\n\u003e | cut -f1 -d: \\\n\u003e | grep \\\n\u003e -e \"^tools\" \\\n\u003e | sort \\\n\u003e | uniq \\\n\u003e | grep \\\n\u003e -e \"\\.h\" \\\n\u003e -e \"\\.cc\" \\\n\u003e -e \"\\.mm\" \\\n\u003e -e \"\\.py\" \\\n\u003e | xargs sed -i \"s/$1/$2/g\"\n\u003e }\n\u003e replace \"absl::make_optional\" \"std::make_optional\"\n\u003e replace \"absl::optional\" \"std::optional\"\n\u003e replace \"absl::nullopt\" \"std::nullopt\"\n\u003e replace \"absl::in_place\" \"std::in_place\"\n\u003e replace \"absl::in_place_t\" \"std::in_place_t\"\n\u003e replace \"\\\"third_party\\/abseil-cpp\\/absl\\/types\\/optional.h\\\"\" \"\u003coptional\u003e\"\n\u003e git cl format\n\u003e ```\n\u003e\n\u003e Bug: chromium:1500249\n\u003e Change-Id: I7536d03fa09fb48f8cf7113eba8eb727d5efba7b\n\u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5185809\n\u003e Reviewed-by: Nico Weber \u003cthakis@chromium.org\u003e\n\u003e Commit-Queue: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\n\u003e Auto-Submit: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#1245840}\n\nBug: chromium:1500249\nChange-Id: I0abd981c73e34bec6ed83eafed20098cf82e4d12\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5190270\nCommit-Queue: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nReviewed-by: Nico Weber \u003cthakis@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1246334}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 69fa1c8ea7b8b207343b91ca8db5e3e7d372b1c7\n" }, { "commit": "137d838b1db2a7f4f83171a95654b4984e11283e", "tree": "eea10db92af2825a8b4f0e11eae7de05bec39563", "parents": [ "615bc9770fc35e8a5ae025be38c7632b86404766" ], "author": { "name": "Arthur Sonzogni", "email": "arthursonzogni@chromium.org", "time": "Thu Jan 11 17:15:28 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 11 17:20:36 2024" }, "message": "Revert \"Rename {absl \u003d\u003e std}::optional in //tools/\"\n\nThis reverts commit b263d258c9e4899afaa24036bde76d7e9e1d537c.\n\nReason for revert: Speculative revert. A `NOTRY\u003dTrue` was copy\npasted mistakenly.\n\nOriginal change\u0027s description:\n\u003e Rename {absl \u003d\u003e std}::optional in //tools/\n\u003e\n\u003e Automated patch, intended to be effectively a no-op.\n\u003e\n\u003e Context:\n\u003e https://groups.google.com/a/chromium.org/g/cxx/c/nBD_1LaanTc/m/ghh-ZZhWAwAJ?utm_medium\u003demail\u0026utm_source\u003dfooter\n\u003e\n\u003e As of https://crrev.com/1204351, absl::optional is now a type alias for\n\u003e std::optional. We should migrate toward it.\n\u003e\n\u003e Script:\n\u003e ```\n\u003e function replace {\n\u003e echo \"Replacing $1 by $2\"\n\u003e git grep -l \"$1\" \\\n\u003e | cut -f1 -d: \\\n\u003e | grep \\\n\u003e -e \"^tools\" \\\n\u003e | sort \\\n\u003e | uniq \\\n\u003e | grep \\\n\u003e -e \"\\.h\" \\\n\u003e -e \"\\.cc\" \\\n\u003e -e \"\\.mm\" \\\n\u003e -e \"\\.py\" \\\n\u003e | xargs sed -i \"s/$1/$2/g\"\n\u003e }\n\u003e replace \"absl::make_optional\" \"std::make_optional\"\n\u003e replace \"absl::optional\" \"std::optional\"\n\u003e replace \"absl::nullopt\" \"std::nullopt\"\n\u003e replace \"absl::in_place\" \"std::in_place\"\n\u003e replace \"absl::in_place_t\" \"std::in_place_t\"\n\u003e replace \"\\\"third_party\\/abseil-cpp\\/absl\\/types\\/optional.h\\\"\" \"\u003coptional\u003e\"\n\u003e git cl format\n\u003e ```\n\u003e\n\u003e NOTRY\u003dTrue\n\u003e\n\u003e Bug: chromium:1500249\n\u003e Change-Id: I7536d03fa09fb48f8cf7113eba8eb727d5efba7b\n\u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5185809\n\u003e Reviewed-by: Nico Weber \u003cthakis@chromium.org\u003e\n\u003e Commit-Queue: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\n\u003e Auto-Submit: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#1245840}\n\nBug: chromium:1500249\nChange-Id: I1b9749db1ddffe073b991065d0d5b21f8579075f\nNo-Presubmit: true\nNo-Tree-Checks: true\nNo-Try: true\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5189307\nCommit-Queue: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nBot-Commit: Rubber Stamper \u003crubber-stamper@appspot.gserviceaccount.com\u003e\nAuto-Submit: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nReviewed-by: Nico Weber \u003cthakis@chromium.org\u003e\nReviewed-by: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1245919}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 8c3a4c4fd8ddc5c8c3384f29e48932f81a38a462\n" }, { "commit": "615bc9770fc35e8a5ae025be38c7632b86404766", "tree": "27221fea55b12cf0b68a270d14756ea9b72ce4bb", "parents": [ "216d33bbf954ef7a6c277e8bb86d6a1b9a3dc3c6" ], "author": { "name": "Arthur Sonzogni", "email": "arthursonzogni@chromium.org", "time": "Thu Jan 11 15:08:39 2024" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 11 15:21:22 2024" }, "message": "Rename {absl \u003d\u003e std}::optional in //tools/\n\nAutomated patch, intended to be effectively a no-op.\n\nContext:\nhttps://groups.google.com/a/chromium.org/g/cxx/c/nBD_1LaanTc/m/ghh-ZZhWAwAJ?utm_medium\u003demail\u0026utm_source\u003dfooter\n\nAs of https://crrev.com/1204351, absl::optional is now a type alias for\nstd::optional. We should migrate toward it.\n\nScript:\n```\nfunction replace {\n echo \"Replacing $1 by $2\"\n git grep -l \"$1\" \\\n | cut -f1 -d: \\\n | grep \\\n -e \"^tools\" \\\n | sort \\\n | uniq \\\n | grep \\\n -e \"\\.h\" \\\n -e \"\\.cc\" \\\n -e \"\\.mm\" \\\n -e \"\\.py\" \\\n | xargs sed -i \"s/$1/$2/g\"\n}\nreplace \"absl::make_optional\" \"std::make_optional\"\nreplace \"absl::optional\" \"std::optional\"\nreplace \"absl::nullopt\" \"std::nullopt\"\nreplace \"absl::in_place\" \"std::in_place\"\nreplace \"absl::in_place_t\" \"std::in_place_t\"\nreplace \"\\\"third_party\\/abseil-cpp\\/absl\\/types\\/optional.h\\\"\" \"\u003coptional\u003e\"\ngit cl format\n```\n\nNOTRY\u003dTrue\n\nBug: chromium:1500249\nChange-Id: I7536d03fa09fb48f8cf7113eba8eb727d5efba7b\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5185809\nReviewed-by: Nico Weber \u003cthakis@chromium.org\u003e\nCommit-Queue: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nAuto-Submit: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1245840}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: b263d258c9e4899afaa24036bde76d7e9e1d537c\n" }, { "commit": "216d33bbf954ef7a6c277e8bb86d6a1b9a3dc3c6", "tree": "eea10db92af2825a8b4f0e11eae7de05bec39563", "parents": [ "802db79a6adb2c1abfb6da3c46c549932669fec7" ], "author": { "name": "Bartek Nowierski", "email": "bartekn@chromium.org", "time": "Thu Dec 28 06:02:14 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Dec 28 06:16:33 2023" }, "message": "[PA] Cleanup: Clarify naming of metadata types \u0026 variables\n\nThe clarifications are mostly related to partition page, but also\nfor slot span and direct map metadata.\n\nDecided to keep variables `slot_span` without the `_metadata` suffix,\nto avoid being too chatty. `slot_span_start` is already commonly used\nthroughout, to disambiguate slot span\u0027s address from its metadata.\n\nThis CL is a no-op: only type/variable renaming and comment tweaks.\n\nChange-Id: I7fb489561a6b6dcd591d6a6298bfbf03acaaab22\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5148624\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nAuto-Submit: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1241304}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 085a9fca51819df07ab206e15175757703c6eef8\n" }, { "commit": "802db79a6adb2c1abfb6da3c46c549932669fec7", "tree": "2357277fcbb790073534d9991d656035db97c6d3", "parents": [ "884774321e209bae8df789bb831695d5d1fa08d6" ], "author": { "name": "Marshall Greenblatt", "email": "marshall@chromium.org", "time": "Wed Dec 13 19:32:53 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Dec 13 19:38:51 2023" }, "message": "Fix implicit conversion error\n\nFixes the following compile error when building on Windows with\nGN arg use_custom_libcxx\u003dfalse.\n\n../../base/allocator/partition_allocator/src\\partition_alloc/thread_cache.h(187,42): error: implicit conversion loses integer precision: \u0027uint16_t\u0027 (aka \u0027unsigned short\u0027) to \u0027uint8_t\u0027 (aka \u0027unsigned char\u0027) [-Werror,-Wimplicit-int-conversion]\n 187 | uint8_t largest_active_bucket_index_ \u003d internal::BucketIndexLookup::GetIndex(\n | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n 188 | ThreadCacheLimits::kDefaultSizeThreshold);\n | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n.\n\nChange-Id: I62dfc334cc4003a59fd8626e207b059e346f1b38\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5120035\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Marshall Greenblatt \u003cmarshall@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1237115}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: d484f6ec10ea3a5b3088043c53df6fc8a2b8261a\n" }, { "commit": "884774321e209bae8df789bb831695d5d1fa08d6", "tree": "4e0ad5ce5292d9766b095990ec14e2d6506c8544", "parents": [ "bb03b820532d620e753e91a1ef9de538753d6899" ], "author": { "name": "Tom Sepez", "email": "tsepez@chromium.org", "time": "Thu Nov 09 18:12:01 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Nov 09 18:19:31 2023" }, "message": "Apply RAW_PTR_EXCLUSION to members passed to munmap().\n\nAdd a comment describing why raw_ptr\u003c\u003e to memory mapped region would\nbe undesirable.\n\nChange-Id: Iaaa4f9cb97e35c8861e9c5ab8b25905411dda324\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5001864\nReviewed-by: Keishi Hattori \u003ckeishi@chromium.org\u003e\nReviewed-by: Steve Cho \u003cstevecho@chromium.org\u003e\nReviewed-by: Tomas Gunnarsson \u003ctommi@chromium.org\u003e\nCommit-Queue: Tom Sepez \u003ctsepez@chromium.org\u003e\nReviewed-by: Brian Geffon \u003cbgeffon@chromium.org\u003e\nReviewed-by: Jungshik Shin \u003cjshin@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1222403}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: aa567abd1bb8a2c5dedfc2689806910605c8f95e\n" }, { "commit": "bb03b820532d620e753e91a1ef9de538753d6899", "tree": "1d2eba268776670189d0246108a1dbd890f035b6", "parents": [ "8b06a53701884108e156dfad4a5498d1f96e3ca2" ], "author": { "name": "Arthur Sonzogni", "email": "arthursonzogni@chromium.org", "time": "Mon Oct 16 19:08:55 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Oct 16 19:26:20 2023" }, "message": "[PA] Move README, OWNERS, DEPS, BUILD.gn to top-level dir.\n\nThis is a follow-up to:\nhttps://chromium-review.googlesource.com/c/chromium/src/+/4812338\n\nThe previous patch moved the whole PA directory into\n\"src/partition_alloc\" without putting much thinking about individual\nfiles.\n\nThis follow-up moves the README, OWNERS, DEPS, BUILD.gn file into the\ntop-level directory.\n\nBug: chromium:1467773\nChange-Id: Ie9f457fe957bbf27779892eece606eb220d50cb4\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4934632\nOwners-Override: danakj \u003cdanakj@chromium.org\u003e\nCommit-Queue: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nReviewed-by: danakj \u003cdanakj@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1210356}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: c8ca46524226127e51531c08bab01d049d4a3083\n" }, { "commit": "8b06a53701884108e156dfad4a5498d1f96e3ca2", "tree": "0f5e097a1ebc391b86a216d0d49b66f762b1294a", "parents": [ "68be214291389b563441d7d4e028333deb168463" ], "author": { "name": "Arthur Sonzogni", "email": "arthursonzogni@chromium.org", "time": "Tue Oct 03 08:48:32 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Oct 03 08:57:40 2023" }, "message": "Move partition_alloc into a \"partition_alloc\" dir.\n\nThis patch moves:\n\"base/allocator/partition_allocator/...\"\ninto:\n\"base/allocator/partition_allocator/src/partition_alloc/...\".\n\nThis tried to keep it as minimal as possible. Interesting follow-ups\nwill happen outside of the rename.\n\nContext:\n--------\nTo protect against UAF, we want to make the PartitionAlloc library\nembeddable by some third_parties: pdfium, angle, dawn, swiftshader, ...\n\nPartitionAlloc do not declare yet where is its \"include_dir\". It means\nthird parties must put the library files under\n\"base/allocator/partition_allocator\", even if they do not use \"base\" and\nwould have liked to put it inside their \"third_party\" directory.\n\nWe would like to make PartitionAllocator behave like a regular library.\nIt should be importable using:\n #include \u003cpartition_alloc/...\u003e\n\nThat\u0027s why we are using: \"src/partition_alloc\"\n\nThis implements step 1 of:\n1. Move from files under `src/partition_alloc/`\n2. Declare PartitionAlloc GN include_dirs.\n3. Update the dependants to use #include \u003cpartition_alloc/...\u003e\n\nBug: chromium:1467773\nCq-Include-Trybots: luci.chromium.try:linux-chromeos-siso-rel\nCq-Include-Trybots: luci.chromium.try:linux-lacros-siso-rel\nCq-Include-Trybots: luci.chromium.try:linux-siso-rel\nCq-Include-Trybots: luci.chromium.try:linux-wayland-siso-rel\nCq-Include-Trybots: luci.chromium.try:linux-wayland-siso-rel\nCq-Include-Trybots: luci.chromium.try:linux_chromium_asan_siso_rel_ng\nCq-Include-Trybots: luci.chromium.try:win-siso-rel\nChange-Id: I1b25cdc9f8077a7041221f51441c72b04b800dcc\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4812338\nCommit-Queue: Arthur Sonzogni \u003carthursonzogni@chromium.org\u003e\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nOwners-Override: danakj \u003cdanakj@chromium.org\u003e\nReviewed-by: danakj \u003cdanakj@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1204530}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 0bcc023b8cdbc073aa5c48db373810db3f765c87\n" }, { "commit": "68be214291389b563441d7d4e028333deb168463", "tree": "eda956634915534c0aef29d7840e8ffa235abf60", "parents": [ "6777190e1b80c0113dfdf1037a1e260905aef2fa" ], "author": { "name": "Avi Drissman", "email": "avi@chromium.org", "time": "Wed Aug 16 22:26:17 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Aug 16 22:32:00 2023" }, "message": "Move scoped_mach_port to base/apple, leave a forwarding header\n\nCrashpad is not yet updated for the new location, so leave a\nforwarding header to be removed later.\n\nBug: 1444927\nChange-Id: I624cae5d8b97e490bc6749fd35be3abf1b072546\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4784629\nOwners-Override: Avi Drissman \u003cavi@chromium.org\u003e\nReviewed-by: Mark Mentovai \u003cmark@chromium.org\u003e\nCommit-Queue: Avi Drissman \u003cavi@chromium.org\u003e\nAuto-Submit: Avi Drissman \u003cavi@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1184421}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: dd92b53f5b9b4b97a51985712c2271b6a48b2872\n" }, { "commit": "6777190e1b80c0113dfdf1037a1e260905aef2fa", "tree": "1f6b51cec9652ffbb6c4e6a9d71cf1aa0e4dd533", "parents": [ "2a4c4ba1f4a94231b01280a0c63d3fe4404cc9c2" ], "author": { "name": "Avi Drissman", "email": "avi@chromium.org", "time": "Wed Aug 16 20:35:49 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Aug 16 20:42:03 2023" }, "message": "Move mach_logging to base/apple, leave a forwarding header\n\nCrashpad is not yet updated for the new location, so leave a\nforwarding header to be removed later.\n\nBug: 1444927\nChange-Id: I8b1e2c9a3a59239f63d6ba7dc562b2221e67ec56\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4784749\nAuto-Submit: Avi Drissman \u003cavi@chromium.org\u003e\nCommit-Queue: Mark Mentovai \u003cmark@chromium.org\u003e\nReviewed-by: Mark Mentovai \u003cmark@chromium.org\u003e\nOwners-Override: Avi Drissman \u003cavi@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1184356}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 0b20071f092026e1bafb5226060bc2a95fbf0f0e\n" }, { "commit": "2a4c4ba1f4a94231b01280a0c63d3fe4404cc9c2", "tree": "f373fc9f0beb17d33c221c351a4e191adbaa8861", "parents": [ "9a047fd494174b132f98527807800bb34a4b6ff8" ], "author": { "name": "Kalvin Lee", "email": "kdlee@chromium.org", "time": "Tue Jul 25 00:31:20 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jul 25 00:48:04 2023" }, "message": "PA: Rename bucket distributions\n\nThis CL is trivial. This CL renames\n`PartitionRoot::BucketDistribution::kDefault` to `kNeutral` and removes\nsome allusions to \"alternate\" bucket distributions.\n\n`kNeutral` is in contrast to `kDenser` and to the deprecated `kCoarser`\n(see https://crrev.com/c/4373791). `kDefault` is a less descriptive\nname, as it only describes the starting state of PartitionAlloc.\nClients can optionally switch to the denser distribution.\n\nBecause of the prevalance of the `kDenser` distribution (non-browser\nprocesses and 32-bit Android excepted), the \"default\" designation is\nbest left to higher levels, e.g. `partition_alloc_features`.\n\nThis CL does not touch the (now defunct) feature flag infrastructure.\n\nMemo: server-side config was removed in revision 530243413.\n\nBug: 1238858\nChange-Id: I89b5b14b0d024ee9cdfce9e904b6f8faf911f235\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4710168\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nAuto-Submit: Kalvin Lee \u003ckdlee@chromium.org\u003e\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1174560}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: d0dfb8c5d8b912965e588d830d02840b3279f898\n" }, { "commit": "9a047fd494174b132f98527807800bb34a4b6ff8", "tree": "8262dc81b10e9d120e3bf9fb2e4a89182fc232be", "parents": [ "57b3b9e7c02f603108007e8a29d2015835d40d13" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Fri Jul 07 04:33:59 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Jul 07 04:43:22 2023" }, "message": "[PA] Follow-up CL after removing ThreadSafe from PartitionAllocator.\n\ncrrev.com/c/4436096 removed ThreadSafe template parameter from\nPartitionAllocator, but the CL has TODO-s, e.g. PartitionAllocator\nstill has \"using ThreadSafePartitionRoot\" and so on.\nThis CL will fix the TODOs.\n\nChange-Id: I5aa6c319aa102a483af32c39619e4e1e57688601\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4663317\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nReviewed-by: Kentaro Hara \u003charaken@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1167139}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 4781c0fed8665e916652c722e1bbf2dd29a71682\n" }, { "commit": "57b3b9e7c02f603108007e8a29d2015835d40d13", "tree": "8fa2e2e4846a376a72221c66b954b150b0f19f2a", "parents": [ "cc6bc216494584effaaf50d00512614ab0a530c0" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Thu Jun 08 01:02:29 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jun 08 01:08:44 2023" }, "message": "[PA] Remove ThreadSafe.\n\nAllowed header dependency:\n- partition_root.h \u003c partition_page.h \u003c partition_bucket.h \u003c partition_superpage_extent_entry.h \u003c\n partition_dcheck_helper.h \u003c partition_alloc_forward.h\n(*) A \u003c B: A can include B. B cannot include A.\n A \u003c B \u003c C: A can include B and C. B can include C.\n C cannot include either A or B.\n\nThis makes it impossible for partition_bucket.h and partition_page.h\nto see partition_root, i.e. root-\u003elock_. To keep thread\nanalysis, methods using thread analysis are moved into\npartition_root.h or partition_root.cc.\n\nChange-Id: I9093866326a958abc8dc3ad74e60463c9e72603d\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4436096\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1154701}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 354da2514b31df2aa14291199a567e10a7671621\n" }, { "commit": "cc6bc216494584effaaf50d00512614ab0a530c0", "tree": "23082ef5c7a865c75ad47ceee5aa6bb5855c337c", "parents": [ "1084fb884e9f3a2d75d984ceeb74904c0e46718a" ], "author": { "name": "Ian Struiksma", "email": "ianstruiksma@google.com", "time": "Wed Jun 07 01:31:19 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jun 07 01:39:08 2023" }, "message": "Have test failures from sanitizers warnings cause task failure.\n\nWhen at least 1 test fails due to sanitizer warning (with fail-san\u003d1\u0027), the task should also fail, and the failing tests should show in stdout.\n\nAlso, if a test suite had no test-launcher-summary-output, don\u0027t automatically fail the task (useful for json vs gtest suites).\n\nValidate-Test-Flakiness: skip\nBug: 1442587\nChange-Id: I38541589fea3e3b2928559daa389842f325903e1\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4581327\nReviewed-by: Alexander Potapenko \u003cglider@chromium.org\u003e\nReviewed-by: Ben Pastene \u003cbpastene@chromium.org\u003e\nCommit-Queue: Ian Struiksma \u003cianstruiksma@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1154204}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 8eaf5361a82e91e9ec9ce420212537b76f68e13f\n" }, { "commit": "1084fb884e9f3a2d75d984ceeb74904c0e46718a", "tree": "516dc6dce6253b1b303b2af6b37bf3b5d45d06af", "parents": [ "09f58c36cb63d3fd26bed8e3aca96d86ae03db91" ], "author": { "name": "Ian Struiksma", "email": "ianstruiksma@google.com", "time": "Wed May 24 00:28:46 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed May 24 00:37:12 2023" }, "message": "add parameter to invoke escalate sanitizer warnings.\n\nFor some sanitizer bots, we would like to post process the stdout\nlogs to scan for sanitizer warnings. these warnings do not cause tests\nto fail, but are valid issues that should be investigated and so for\nsome builders they should cause tests and thus steps to fail.\n\nBug: 1442587\nChange-Id: I6e3be1e66a2cb3da6fb1b98b0ee8724d0f780cfb\nValidate-Test-Flakiness: skip\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4485037\nReviewed-by: Erik Staab \u003cestaab@chromium.org\u003e\nReviewed-by: Ben Pastene \u003cbpastene@chromium.org\u003e\nCommit-Queue: Ian Struiksma \u003cianstruiksma@google.com\u003e\nReviewed-by: Alexander Potapenko \u003cglider@chromium.org\u003e\nReviewed-by: Jonathan Metzman \u003cmetzman@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1148281}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 2ebc322abad061390efbcc14633acb8703e0481a\n" }, { "commit": "09f58c36cb63d3fd26bed8e3aca96d86ae03db91", "tree": "dffaef3ae881a809bc18d16f89d6d6f0bbf0b473", "parents": [ "a295e5b2886b8f8b6e00a967447c7ed4e5f9e351" ], "author": { "name": "Thiabaud Engelbrecht", "email": "thiabaud@google.com", "time": "Thu Apr 20 17:57:02 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Apr 20 18:13:25 2023" }, "message": "[partition_alloc] Update tool to compare two distributions\n\nThis CL allows us to compare two different bucket distributions to each\nother, while Chrome is running. It displays the difference in internal\nfragmentation between the distributions, as well as the amount of\nmemory allocated in each bucket for each distribution.\n\nChange-Id: I8001dce5cc91057fd3ccde4c5110793e2c174bb7\nBug: 1238858\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3813966\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCommit-Queue: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCode-Coverage: Findit \u003cfindit-for-me@appspot.gserviceaccount.com\u003e\nCr-Commit-Position: refs/heads/main@{#1133230}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 7ef1266320e0e74cf5f5b7040186caec60586440\n" }, { "commit": "a295e5b2886b8f8b6e00a967447c7ed4e5f9e351", "tree": "f81c88f2704be2f61f53fb8c3edf322553dc966b", "parents": [ "13f0b81ce581364c5f0f2e9e16d6120073dc56a6" ], "author": { "name": "Thiabaud Engelbrecht", "email": "thiabaud@google.com", "time": "Wed Apr 12 21:57:03 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Apr 12 22:06:24 2023" }, "message": "[partition_alloc] Remove \u0027coarse\u0027 bucket distribution\n\nThis CL removes the \u0027coarse\u0027 bucket distribution, since\nfinch experiments have determined it regresses memory.\n\nBug: 1238858\nChange-Id: Id5d62531cc933f48137c6f46817fd7d04ce9ce5a\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4373791\nCommit-Queue: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1129523}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 16a9cfba3f9db1304f1cce12d04dbdd426c0f952\n" }, { "commit": "13f0b81ce581364c5f0f2e9e16d6120073dc56a6", "tree": "cbb0eaf93676cdf788aca16ab143c2514e909088", "parents": [ "b55593f4dc40ec73d72be2d82de64294e98dd342" ], "author": { "name": "François Doray", "email": "fdoray@chromium.org", "time": "Mon Mar 27 20:40:18 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Mar 27 20:45:42 2023" }, "message": "[tools] Add memory simulator.\n\nThis tool performs allocations, reads, writes at a configurable rate\nwhile collecting metrics. For now, system metrics are collected on\nmacOS but other platforms will be added in the future.\n\nBug: 1427619\nChange-Id: I1cbcbf6b98a39a397446e2316a2fcdfd4e031e19\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4237304\nReviewed-by: Lei Zhang \u003cthestig@chromium.org\u003e\nCommit-Queue: Robert Sesek \u003crsesek@chromium.org\u003e\nAuto-Submit: Francois Pierre Doray \u003cfdoray@chromium.org\u003e\nReviewed-by: Bruce Dawson \u003cbrucedawson@chromium.org\u003e\nReviewed-by: Robert Sesek \u003crsesek@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1122624}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 2ac3199aabb12e718dde48205ed6c316a611ad18\n" }, { "commit": "b55593f4dc40ec73d72be2d82de64294e98dd342", "tree": "4f88862aba1d580b358295dc5ca6fcb3cf680487", "parents": [ "930bae8b255f3cae63d8b1c05e8b6c8c8dde1ff8" ], "author": { "name": "Kalvin Lee", "email": "kdlee@chromium.org", "time": "Fri Feb 24 02:15:41 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 24 02:20:38 2023" }, "message": "[PA] Remove `migration_adapter.h`\n\nThe only alias left in `migration_adapter.h` is for `base::LapTimer`.\nAll non-test PartitionAlloc code has ceased from using `LapTimer`, so\nthere\u0027s no need to include it anywhere anymore.\n\nOTOH, two performance tests currently use `LapTimer` (though they do not\ninclude `migration_adapter.h`). This is a separate problem, as the\nentanglement of our tests with the rest of `//base` is a work item to be\nsolved at a later time.\n\nBug: 1288247\nChange-Id: I6dc6a488435b478af78eb0bce50bebc7d77f891a\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4278943\nAuto-Submit: Kalvin Lee \u003ckdlee@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1109394}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 9e050d7a9694de37d19cc9458f8b4e7c887156ba\n" }, { "commit": "930bae8b255f3cae63d8b1c05e8b6c8c8dde1ff8", "tree": "de35900efaff036e6a72af9b8ba43b6deeb5aa46", "parents": [ "568316d425f5744b5e34eb7d15332ecd001526f7" ], "author": { "name": "Avi Drissman", "email": "avi@chromium.org", "time": "Tue Jan 31 00:51:48 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 31 00:57:07 2023" }, "message": "Update PartitionAlloc tools to the new Value API\n\nBug: 646113\nCq-Include-Trybots: luci.chromium.try:mac-arm64-on-arm64-rel\nChange-Id: Iebbe5ba0a2bb778ee149007e0d778351fa6b5a4a\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4198088\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nAuto-Submit: Avi Drissman \u003cavi@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1099013}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: bc6f81349c604179d0fb358606db0e1106b38bc2\n" }, { "commit": "568316d425f5744b5e34eb7d15332ecd001526f7", "tree": "5f2d042a2ede26f2691c6616a68adc4cf4cfea74", "parents": [ "da557f2c897726fc629156c3fe28c2f33224d9cf" ], "author": { "name": "Avi Drissman", "email": "avi@chromium.org", "time": "Sat Jan 28 02:31:26 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sat Jan 28 02:39:33 2023" }, "message": "Remove base::Value::Type::DICTIONARY\n\nIt is deprecated in favor of base::Value::Type::DICT, so change uses\nof it to the new name and remove the enum value.\n\nFixed: 1291670\nChange-Id: I26136fcb723e7dff1f5ffe3990d7011b18c16a43\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4198081\nAuto-Submit: Avi Drissman \u003cavi@chromium.org\u003e\nReviewed-by: Daniel Cheng \u003cdcheng@chromium.org\u003e\nCommit-Queue: Daniel Cheng \u003cdcheng@chromium.org\u003e\nOwners-Override: Daniel Cheng \u003cdcheng@chromium.org\u003e\nOwners-Override: Avi Drissman \u003cavi@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1098238}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: d2a736f98543a51399e3047a52ba469556dc4633\n" }, { "commit": "da557f2c897726fc629156c3fe28c2f33224d9cf", "tree": "3342cf7d2837a539f65d6a0cfe703ffb6b61ddc3", "parents": [ "fcf89330327f9cc007915af925b75981efcb3961" ], "author": { "name": "Bartek Nowierski", "email": "bartekn@chromium.org", "time": "Thu Jan 19 09:45:12 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 19 09:50:13 2023" }, "message": "[PA] Add keishi@ to PartitionAlloc\u0027s OWNERS\n\nMake tools/memory/partition_allocator/OWNERS use PartitionAlloc\u0027s\nOWNERS, while ait it\n\nChange-Id: I7e3218f44eda1c453553fc10209e79fef7ca9842\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4177498\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nAuto-Submit: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1094361}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 30f1a5e5f46e1050a19ef2b61e090d84ace20f64\n" }, { "commit": "fcf89330327f9cc007915af925b75981efcb3961", "tree": "ad1ef6b4650f3d0008a06042ba99132881d2b3fc", "parents": [ "5d2efcc932883fa993dd6947456ed036bb41de9a" ], "author": { "name": "Bartek Nowierski", "email": "bartekn@chromium.org", "time": "Wed Jan 11 11:01:02 2023" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jan 11 11:06:48 2023" }, "message": "[PA] Make partition_alloc_config.h macros fool-proof\n\nThis defines PA_CONFIG(), which works in a similar way as BUILDFLAG(),\nso that typos and missing #include are caught by the compiler.\n\nImprove #endif/#else comments while at it.\n\nThis CL is expected to be a no-op.\n\nChange-Id: Ibbbf71c2cb4a8a27a2f00784fb4a615b3c54d336\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4123912\nReviewed-by: Kalvin Lee \u003ckdlee@chromium.org\u003e\nOwners-Override: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1091254}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: af8d5d2a6d2f816c1bf96481faae6349b92ae70d\n" }, { "commit": "5d2efcc932883fa993dd6947456ed036bb41de9a", "tree": "437728cb314e6946b8131f8cb7be209b5c0f5e7e", "parents": [ "797585e74b41e0b588562d04f076f5ffd4ff6d8b" ], "author": { "name": "Kalvin Lee", "email": "kdlee@chromium.org", "time": "Wed Nov 30 09:22:40 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Nov 30 09:30:15 2022" }, "message": "Reland \"[PA] Remove `BUILDFLAG(USE_BACKUP_REF_PTR)`\"\n\nThis is a reland of commit d02cbc7640649876b67b022fbb94050c4099e85d\n\nPatchset 1 provides the pure reland; patchset 2 tweaks the problematic\nBUILDFLAG to provide a complete fix.\n\nOriginal change\u0027s description:\n\u003e [PA] Remove `BUILDFLAG(USE_BACKUP_REF_PTR)`\n\u003e\n\u003e This buildflag is already congruent with\n\u003e `BUILDFLAG(ENABLE_BACKUP_REF_PTR_SUPPORT)`, provided by PartitionAlloc\u0027s\n\u003e buildflags. This CL trivially migrates all Chromium users to the\n\u003e PA-sourced buildflag, erasing `BUILDFLAG(USE_BACKUP_REF_PTR)` from the\n\u003e allocator build recipe.\n\u003e\n\u003e Bug: 1371729\n\u003e Change-Id: Ia9488aada1f942500c76e269e863c0e3cea8a546\n\u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4054728\n\u003e Owners-Override: Bartek Nowierski \u003cbartekn@chromium.org\u003e\n\u003e Commit-Queue: Kalvin Lee \u003ckdlee@chromium.org\u003e\n\u003e Reviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#1076086}\n\nBug: 1371729\nChange-Id: I87b84b97d2a0f36bc754a473f0e2972f36a3b5d5\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4060874\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nOwners-Override: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Kalvin Lee \u003ckdlee@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1077339}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 6d5078d22103a0c3855d7753aa828c3b44cb2f94\n" }, { "commit": "797585e74b41e0b588562d04f076f5ffd4ff6d8b", "tree": "7c46a47b092230a223b4679ebfb7d509e949fae6", "parents": [ "062c353a6a6d6011f5eff5a48bfa8fa136e0b3b0" ], "author": { "name": "Rakina Zata Amni", "email": "rakina@chromium.org", "time": "Mon Nov 28 08:24:23 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Nov 28 08:28:04 2022" }, "message": "Revert \"[PA] Remove `BUILDFLAG(USE_BACKUP_REF_PTR)`\"\n\nThis reverts commit d02cbc7640649876b67b022fbb94050c4099e85d.\n\nReason for revert: Causing build failures. Example: https://ci.chromium.org/ui/p/chromium/builders/ci/mac-arm64-on-arm64-rel/51001/overview\n\nOriginal change\u0027s description:\n\u003e [PA] Remove `BUILDFLAG(USE_BACKUP_REF_PTR)`\n\u003e\n\u003e This buildflag is already congruent with\n\u003e `BUILDFLAG(ENABLE_BACKUP_REF_PTR_SUPPORT)`, provided by PartitionAlloc\u0027s\n\u003e buildflags. This CL trivially migrates all Chromium users to the\n\u003e PA-sourced buildflag, erasing `BUILDFLAG(USE_BACKUP_REF_PTR)` from the\n\u003e allocator build recipe.\n\u003e\n\u003e Bug: 1371729\n\u003e Change-Id: Ia9488aada1f942500c76e269e863c0e3cea8a546\n\u003e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4054728\n\u003e Owners-Override: Bartek Nowierski \u003cbartekn@chromium.org\u003e\n\u003e Commit-Queue: Kalvin Lee \u003ckdlee@chromium.org\u003e\n\u003e Reviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#1076086}\n\nBug: 1371729\nChange-Id: I439be37d67d2e2c4584d1c7d149ce588ba57be26\nNo-Presubmit: true\nNo-Tree-Checks: true\nNo-Try: true\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4060410\nBot-Commit: Rubber Stamper \u003crubber-stamper@appspot.gserviceaccount.com\u003e\nAuto-Submit: Rakina Zata Amni \u003crakina@chromium.org\u003e\nOwners-Override: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1076090}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 9f2530c744d76fbfd8a663b0789b5c63b8a8979d\n" }, { "commit": "062c353a6a6d6011f5eff5a48bfa8fa136e0b3b0", "tree": "437728cb314e6946b8131f8cb7be209b5c0f5e7e", "parents": [ "4a3820026f089b960dbd17d1a84b1364f7dcf46c" ], "author": { "name": "Kalvin Lee", "email": "kdlee@chromium.org", "time": "Mon Nov 28 07:59:21 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Nov 28 08:04:01 2022" }, "message": "[PA] Remove `BUILDFLAG(USE_BACKUP_REF_PTR)`\n\nThis buildflag is already congruent with\n`BUILDFLAG(ENABLE_BACKUP_REF_PTR_SUPPORT)`, provided by PartitionAlloc\u0027s\nbuildflags. This CL trivially migrates all Chromium users to the\nPA-sourced buildflag, erasing `BUILDFLAG(USE_BACKUP_REF_PTR)` from the\nallocator build recipe.\n\nBug: 1371729\nChange-Id: Ia9488aada1f942500c76e269e863c0e3cea8a546\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4054728\nOwners-Override: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Kalvin Lee \u003ckdlee@chromium.org\u003e\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1076086}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: d02cbc7640649876b67b022fbb94050c4099e85d\n" }, { "commit": "4a3820026f089b960dbd17d1a84b1364f7dcf46c", "tree": "7c46a47b092230a223b4679ebfb7d509e949fae6", "parents": [ "06d80e1d518af0a283393cf3830d047927d43f1f" ], "author": { "name": "Lei Zhang", "email": "thestig@chromium.org", "time": "Thu Nov 10 00:12:55 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Nov 10 00:19:16 2022" }, "message": "Remove spurious stringprintf.h includes from non-headers.\n\nFiles that do not match the patterns StringPrint[fV] and\nStringAppend[FV] don\u0027t need this include.\n\nBug: 242216\nChange-Id: Id927c7500e6ac41f1aa1e411b0461cc75d8509fa\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4014655\nCommit-Queue: Daniel Cheng \u003cdcheng@chromium.org\u003e\nOwners-Override: Daniel Cheng \u003cdcheng@chromium.org\u003e\nReviewed-by: Daniel Cheng \u003cdcheng@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1069457}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 2d62ab6b0363a9e409f289b6ecfc57515a05f379\n" }, { "commit": "06d80e1d518af0a283393cf3830d047927d43f1f", "tree": "dcb2868cd90aab0fd896305b0ab6e84f9b43e11b", "parents": [ "8581381d92045707a699f1fc14525cac75a802e0" ], "author": { "name": "Kalvin Lee", "email": "kdlee@chromium.org", "time": "Mon Nov 07 13:55:18 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Nov 07 14:00:44 2022" }, "message": "[PA] Move `use_partition_alloc`\n\nThis CL trivially moves the declaration of the build arg\n`use_partition_alloc` from `//base/allocator/allocator.gni` to PA\u0027s\n`partition_alloc.gni`. Some other `BUILD.gn`s are changed to import\n`partition_alloc.gni`, in order to continue using `use_partition_alloc`.\n\nThis CL introduces no behavioral change, but is intended to allow PA to\nexpress the `use_partition_alloc` concept in embedder GN recipes. Much\nas Chromium already does, the idea is to allow embedders to gate\ninclusion of the PA sources on `use_partition_alloc`.\n\nBug: 1371729\nChange-Id: I342a0860602a3c74c8254e37eda6d5c5700447b2\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3985428\nReviewed-by: Leszek Swirski \u003cleszeks@chromium.org\u003e\nReviewed-by: Hans Wennborg \u003chans@chromium.org\u003e\nAuto-Submit: Kalvin Lee \u003ckdlee@chromium.org\u003e\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Matthew Denton \u003cmpdenton@chromium.org\u003e\nReviewed-by: Matthew Denton \u003cmpdenton@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1068074}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 41a852742c157b191a98a2739548b2308117a1c6\n" }, { "commit": "8581381d92045707a699f1fc14525cac75a802e0", "tree": "f78d956b52004ee071166f4a019918cee0f9f1e1", "parents": [ "0f4bd26063bbce7d9d81c991921530a529cad5a6" ], "author": { "name": "Bartek Nowierski", "email": "bartekn@chromium.org", "time": "Fri Oct 21 20:00:08 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Oct 21 20:05:36 2022" }, "message": "[PA] Introduce use_partition_alloc_as_malloc (2 of N)\n\nThis is one in the series of CLs that aims at replacing use_allocator\nand use_partition_alloc_as_malloc with use_partition_alloc_as_malloc.\n use_allocator \u003d\u003d \"partition\" \u003c\u003d\u003e use_partition_alloc_as_malloc \u003d\u003d true\n use_allocator \u003d\u003d \"none\" \u003c\u003d\u003e use_partition_alloc_as_malloc \u003d\u003d false\n\nThis CL switches from enable_partition_alloc_as_malloc_support and\nuse_allocator to use_partition_alloc_as_malloc everywhere. Once landed,\nuse_allocator will be useless and can be removed from GN args. enable_partition_alloc_as_malloc_support too, but that doesn\u0027t seem to\nbe set anywhere anyway.\n\nBug: 1151236\nChange-Id: Ie1c99b92e0f533513a20d56d69cecb1c903dd320\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3965990\nCommit-Queue: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nAuto-Submit: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nReviewed-by: Mark Mentovai \u003cmark@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1062311}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 07f51c672390638de7eb7310b627d2d22e483a65\n" }, { "commit": "0f4bd26063bbce7d9d81c991921530a529cad5a6", "tree": "41a90f6859b6f4d62c790c90240d3873a3cd53b2", "parents": [ "98140a69489b05f65f7a4649b9949e074fc2da8a" ], "author": { "name": "Bartek Nowierski", "email": "bartekn@chromium.org", "time": "Thu Oct 20 06:43:12 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 20 06:49:06 2022" }, "message": "[PA] Introduce use_partition_alloc_as_malloc (1 of N)\n\nThis is one in the series of CLs that aims at replacing use_allocator\nand use_partition_alloc_as_malloc with use_partition_alloc_as_malloc.\n use_allocator \u003d\u003d \"partition\" \u003c\u003d\u003e use_partition_alloc_as_malloc \u003d\u003d true\n use_allocator \u003d\u003d \"none\" \u003c\u003d\u003e use_partition_alloc_as_malloc \u003d\u003d false\n\nThis CL introduces use_partition_alloc_as_malloc as a dummy GN arg, and\nhas users of use_allocator set it appropriately in parallel.\n\nBug: 1151236\nChange-Id: I437c4942ca0f3c92ecd8ec002352c435077b1cc0\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3963869\nReviewed-by: Jochen Eisinger \u003cjochen@chromium.org\u003e\nReviewed-by: Garrett Beaty \u003cgbeaty@google.com\u003e\nCommit-Queue: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1061442}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 0281a1ce046355e00563847cc254e47f2499ce22\n" }, { "commit": "98140a69489b05f65f7a4649b9949e074fc2da8a", "tree": "8ff3315ea87799b06a6a190eb66d666dac90d16f", "parents": [ "e5f1a8a76621314780eb9018e7ad049562cdd895" ], "author": { "name": "Avi Drissman", "email": "avi@chromium.org", "time": "Thu Sep 15 20:11:09 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Sep 15 20:19:35 2022" }, "message": "Update copyright headers in testing/, tools/\n\nThe methodology used to generate this CL is documented in\nhttps://crbug.com/1098010#c95.\n\nNo-Try: true\nNo-Presubmit: true\nBug: 1098010\nChange-Id: I3a8a7b150e7bd64690534727150646081df50439\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3900697\nReviewed-by: Mark Mentovai \u003cmark@chromium.org\u003e\nAuto-Submit: Avi Drissman \u003cavi@chromium.org\u003e\nOwners-Override: Avi Drissman \u003cavi@chromium.org\u003e\nCommit-Queue: Avi Drissman \u003cavi@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1047644}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: dfd88085261b662a5c0a1abea1a3b120b08e8e48\n" }, { "commit": "e5f1a8a76621314780eb9018e7ad049562cdd895", "tree": "cd6273d14b8f5b084becc671f1659f0ed0406fa6", "parents": [ "7d9556cfdabc65332f4bc4d20f675b98467d471d" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Thu Aug 11 16:24:41 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Aug 11 16:36:17 2022" }, "message": "[tools/memory] Prettier printing and sorting of IOSurfaces\n\nAdds a more readable way to show IOSurfaces, and add sorting by\ndirty/swapped footprint.\n\nSee below for an example:\n\t175024000-175e34000\tVirtual Size: 14.1MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 14.1MiB\tDimensions: 2560x1440\n\t2906c8000-2914d8000\tVirtual Size: 14.1MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 14.1MiB\tDimensions: 2560x1440\n\t293e18000-294c28000\tVirtual Size: 14.1MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 14.1MiB\tDimensions: 2560x1440\n\t294c28000-295a38000\tVirtual Size: 14.1MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 14.1MiB\tDimensions: 2560x1440\n\t2880c8000-288ed8000\tVirtual Size: 14.1MiB\tResident: 13.8MiB\tDirty: 13.8MiB\tSwapped: 320.0kiB\tDimensions: 2560x1440\n\t15dbd0000-15f1bc000\tVirtual Size: 21.9MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 21.9MiB\tDimensions: 3024x1890\n\t175e34000-177420000\tVirtual Size: 21.9MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 21.9MiB\tDimensions: 3024x1890\n\t280000000-2815ec000\tVirtual Size: 21.9MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 21.9MiB\tDimensions: 3024x1890\n\t1709d4000-17201c000\tVirtual Size: 22.3MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 22.3MiB\tDimensions: 2448x2370\n\t28687c000-2880c8000\tVirtual Size: 24.3MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 24.3MiB\tDimensions: 2448x2583\n\t17bce0000-17d5d8000\tVirtual Size: 25.0MiB\tResident: 0.0kiB\tDirty: 0.0kiB\tSwapped: 25.0MiB\tDimensions: 2448x2655\n\nBug: 1329375\nChange-Id: I34d9f45356de8b69288a20adfdd2aa07360c0edd\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3819846\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1034021}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: e6b1f3a1dae15b997f4cbed69a95ef602a27d8a8\n" }, { "commit": "7d9556cfdabc65332f4bc4d20f675b98467d471d", "tree": "272edd3a00d214416b792a456e9a519532f4d42c", "parents": [ "41d69d665a21f21858f9992bc751c2a94da37f9e" ], "author": { "name": "Thiabaud Engelbrecht", "email": "thiabaud@google.com", "time": "Thu Aug 04 15:38:21 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Aug 04 15:49:10 2022" }, "message": "[partition_alloc] Update script for bucketing scheme to display size\n\nCurrently, the graphs from the script show only internal fragmentation.\nThis CL updates them to also show the number of MiB wasted.\n\nChange-Id: Iea5aa7b5cac46bcbb818b8c8b8d92742c35b664a\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3809572\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCommit-Queue: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1031516}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 1073ca69e8d3a8527869eb8668fef1ce99023833\n" }, { "commit": "41d69d665a21f21858f9992bc751c2a94da37f9e", "tree": "3716aa9b6ffed77b43dff05066caa87d11cd61c5", "parents": [ "345862f410b5869f93cbd4d4fa60f88255696e11" ], "author": { "name": "Thiabaud Engelbrecht", "email": "thiabaud@google.com", "time": "Fri Jul 29 05:45:15 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Jul 29 05:56:29 2022" }, "message": "[partition_alloc] Fix tooling for locally profiling allocations.\n\nThis CL addresses a few issues with the scripts for locally profiling PA\n\n(1) We increase the number of allocations tracked in the circular\nbuffer, so we do not lose track of any allocations that are done near\nthe start of a process.\n(2) We use the default bucket allocation in the script instead of the\nalternate distribution.\n\nallocations:\nChange-Id: I0bc51c85d2bad3c5977c700679ef22a3df9f7c44\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3770987\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCommit-Queue: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1029642}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: aeb34597486c1eacf6fdc707826b88ef258f9b80\n" }, { "commit": "345862f410b5869f93cbd4d4fa60f88255696e11", "tree": "8e8a0cfc96c6982944b312bd22a9461cbe571a88", "parents": [ "f8a4cfe4f95a2d5b9ec0b67b5434adab88057a52" ], "author": { "name": "Kalvin Lee", "email": "kdlee@chromium.org", "time": "Wed Jun 15 22:41:55 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jun 15 22:52:16 2022" }, "message": "[PA] Namespace migration (38 of N)\n\nThis change erases multiple `base::` aliases from PartitionAlloc\nheaders.\n\nBug: 1288247\nChange-Id: I1dfdbd9477d8b0ac6e1e8a5ee4fe26453a9df558\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3703038\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nReviewed-by: Alexander Timin \u003caltimin@chromium.org\u003e\nReviewed-by: Erik Chen \u003cerikchen@chromium.org\u003e\nAuto-Submit: Kalvin Lee \u003ckdlee@chromium.org\u003e\nReviewed-by: Brian Geffon \u003cbgeffon@chromium.org\u003e\nReviewed-by: Kentaro Hara \u003charaken@chromium.org\u003e\nReviewed-by: Jonathan Metzman \u003cmetzman@chromium.org\u003e\nReviewed-by: Michael Lippautz \u003cmlippautz@chromium.org\u003e\nCommit-Queue: Kalvin Lee \u003ckdlee@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1014685}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 01b3850d8f45c8d0c1b9d7ef518700eb55db31f6\n" }, { "commit": "f8a4cfe4f95a2d5b9ec0b67b5434adab88057a52", "tree": "0113b803c1c9c4ee0f543f77fec6051150e24dfe", "parents": [ "aa17e00c3a6d2309295ee015e1abbe8fb09c5454" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Mon May 30 16:21:26 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon May 30 16:33:28 2022" }, "message": "[tools/memory] Add a tool to plot IOSurface memory usage over time\n\nThis CL adds a tool to plot live data from a Chrome GPU process about\nits IOSurface memory usage, and the breakdown between virtual, dirty\n(and resident) and swapped.\n\nTo use it, tools/memory/mac/plot_iosurface.y --pid GPU_PROCESS_PID\n(requires matplotlib and numpy to be installed locally)\n\nBug: 1329375\nChange-Id: Ib46ed61be40a6bffd8d28a9bc2559eb845d45d56\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3674467\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1008866}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 632073326435bf90c73ba525ccbe4b9e0603d6d7\n" }, { "commit": "aa17e00c3a6d2309295ee015e1abbe8fb09c5454", "tree": "e1506ce85448f1d5db52780321fcd9cfa9918b29", "parents": [ "f55fade59b3ce25bc0c1e111b02f1aef4f9d3217" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Wed May 25 21:23:39 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed May 25 21:33:42 2022" }, "message": "[tools/memory] Add a tool to track IOSurface memory usage\n\nIOSurfaces are used for GPU rendering. Using vmmap PID on macOS, one can\nget all the surfaces, along with their characteristics.\nThe added tool does that.\n\nSample output:\n\nIOSurfaces sorted by width:\n\tIOSurface(start\u003d4514004992, end\u003d4514054144, virtual\u003d49152, resident\u003d49152, dirty\u003d49152, swapped\u003d0, width\u003d1, height\u003d280, size\u003d35840)\n\tIOSurface(start\u003d4500996096, end\u003d4501061632, virtual\u003d65536, resident\u003d0, dirty\u003d0, swapped\u003d65536, width\u003d15, height\u003d450, size\u003d57344)\n\tIOSurface(start\u003d4505305088, end\u003d4505370624, virtual\u003d65536, resident\u003d0, dirty\u003d0, swapped\u003d65536, width\u003d15, height\u003d456, size\u003d58368)\n\tIOSurface(start\u003d4531732480, end\u003d4531798016, virtual\u003d65536, resident\u003d0, dirty\u003d0, swapped\u003d65536, width\u003d15, height\u003d456, size\u003d58368)\n[...]\n\nMemory lost due to page rounding \u003d 5.1MiB\nDirty Memory \u003d 173.4MiB\nSwapped Memory \u003d 732.6MiB\n\nShowing that the process is using close to 1GiB of IOSurfaces, with more\nthan 700MiB swapped.\n\nChange-Id: I2d3410228fce732ee1b3ed3905bd25cf6e75d727\nBug: 1298417\nTBR: thestig # Per instructions in //tools/OWNERS\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3610117\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nReviewed-by: Bruce Dawson \u003cbrucedawson@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1007568}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: e61837e86514559aefde30e6b45a6e4569ca413f\n" }, { "commit": "f55fade59b3ce25bc0c1e111b02f1aef4f9d3217", "tree": "42c14811adcbb2168e8a35ea0396b047e1277b53", "parents": [ "be0b201de9dd8625af7718ee325fe95516fcd5f2" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Fri May 13 14:00:04 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri May 13 14:11:43 2022" }, "message": "partition_alloc: copy PlatformThread to PA library.\n\nSplit PlatformThread into (1) PlatformThread and\n(2) PlatformThreadForTesting.\n(1) is the code PartitionAlloc really depends on. (2) is the code\nPartitionAlloc tests depend on. For example, PlatformThread::Join() is\nused by thread_cache_unittests, but PartitionAlloc library doesn\u0027t use\nit.\n\nPS1 \u003d just copying platform_thread* to PA library\n\nBug: 1151236\nChange-Id: Ibce58993c15155b9f521a181b75e93e6daa14c66\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3612910\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Takashi Sakamoto \u003ctasak@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#1003104}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 2fa10639fd51d4f1aac5d616db91d43493449cb5\n" }, { "commit": "be0b201de9dd8625af7718ee325fe95516fcd5f2", "tree": "785d30f472c8a9d21928b8bf2a43ebabe8257dde", "parents": [ "c8563c1c1da2834c62c7652e78478274febd0960" ], "author": { "name": "Bartek Nowierski", "email": "bartekn@chromium.org", "time": "Tue May 10 04:23:01 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 10 04:34:03 2022" }, "message": "[PA] Copy base/posix/eintr_wrapper.h to PA library\n\nPS1 \u003d Simple copy of //base/posix/eintr_wrapper.h\n\nBug: 1151236\nChange-Id: Ieaeff6013d388e871523902bd35673bbbcb44661\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3634838\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nReviewed-by: Takashi Sakamoto \u003ctasak@google.com\u003e\nCommit-Queue: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#1001345}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: aff355921175ce3c8060e8835169fe7b127d5ffc\n" }, { "commit": "c8563c1c1da2834c62c7652e78478274febd0960", "tree": "ffa7323142f9708cc6e077d2088747c24df5793e", "parents": [ "ca7c1ec42aae70d6b04972c99020467314a90187" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Mon May 02 08:48:00 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon May 02 08:57:47 2022" }, "message": "[tools/memory] Make pa_heap_dump record heap compressibility on macOS\n\nBug: 1217582\nChange-Id: I9c905ce518050ffe6e87ed9fd0ac72d81724c7c8\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3613651\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#998283}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 420973215f8f98f0a2bec0996758da39f3aa0a20\n" }, { "commit": "ca7c1ec42aae70d6b04972c99020467314a90187", "tree": "964c123078497321cf249f745b2a8d7337cab817", "parents": [ "4bec803ccb78849af41f68c4b75c147d6a975e4b" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Wed Apr 27 08:55:33 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Apr 27 09:05:08 2022" }, "message": "[tools/memory] Separate linux and mac code in PA\u0027s tools\n\nBug: 1217582\nChange-Id: I61139ce8aecf35317e8d2667520b19e990fdc642\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3607954\nReviewed-by: Sylvain Defresne \u003csdefresne@chromium.org\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#996592}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: c6ba000ec09e78a8e60c6e299180ebf3a9dcfced\n" }, { "commit": "4bec803ccb78849af41f68c4b75c147d6a975e4b", "tree": "b8f6e80bd8b8a30a2c52b6461dde3f8fcf1fc604", "parents": [ "74cc27492a436f7a07024a6c78f3076ea4e69af3" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Tue Apr 26 09:45:19 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Apr 26 09:55:35 2022" }, "message": "[tools/memory] Make pa_dump_heap work on macOS\n\nRemote memory reading on macOS requires the remote mapping to be\nreadable. Since superpages are never entirely readable, need to read\npage by page.\n\nAlso removing some newer typing annotations in Python, as they are not\nworking with Python 3.8.\n\nBug: 1217582\nChange-Id: Icdd4f4a14e1ad2f50ed00775a9eb2f83b6bc410e\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3602880\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#996078}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 7e1fc4f4a8ef241123fb53bf7f1ee91cf67f6968\n" }, { "commit": "74cc27492a436f7a07024a6c78f3076ea4e69af3", "tree": "518da5369af6a602c77e3727a79e478d340e6e73", "parents": [ "087c7630a33cb558fec1b863cc5076d97a67aef8" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Mon Apr 25 10:26:00 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Apr 25 10:36:18 2022" }, "message": "[tools/memory] Add macOS support to PartitionAlloc tools\n\nIn order to inspect PartitionAlloc\u0027s heap for running processes, this\ncommit adds macOS support. These tools need:\n- To enumerate the mappings from a process\n- To read its remote memory\n\nFor the first one, mach provides native APIs that we use. For the second\none, we also use the mach API. Since on Linux we rely on reading files\n(/proc/PID/maps and /proc/PID/mem), this commit adds cross-platform\nwrappers.\n\nOne additional constraint is that on macOS, binaries need extra\npermissions to read the mappings and memory of a remote process, which\nis added to the BUILD.gn file. Finally, this has only been tested for\nlocal builds.\n\nBug: 1217582\nChange-Id: Iba945b586ed46bde47f5926f7d51804169d09590\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3593015\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nReviewed-by: Sylvain Defresne \u003csdefresne@chromium.org\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#995640}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: d85493fe9fc38ef07388be9bb8bf0c369255bb96\n" }, { "commit": "087c7630a33cb558fec1b863cc5076d97a67aef8", "tree": "d56cb4c430adcfb038c820ea83f3c302280c0f32", "parents": [ "6efd4ff84419a48a22f5bd63ff28a49aa86e9dcc" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Wed Apr 13 08:38:46 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Apr 13 08:47:18 2022" }, "message": "[tools/memory] Show the heap compressibility in pa_dump_heap\n\nBug: 1217582\nChange-Id: If2899069d08854584d6ad2de3c63a5d1c4793c4f\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3578681\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nReviewed-by: Austin Sullivan \u003casully@chromium.org\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#991944}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 56cec324ea2b17ee221d6c3dce5d51d2a1c64cad\n" }, { "commit": "6efd4ff84419a48a22f5bd63ff28a49aa86e9dcc", "tree": "ded0c5e37fa1b0fc3e481797964d09c747a0b63a", "parents": [ "d8c227955829741f0a171268ff1696fe4ef341f0" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Fri Apr 08 14:32:54 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Apr 08 14:43:55 2022" }, "message": "[tools/memory] Small fixes and tweaks in live fragmentation plots\n\nChange-Id: I8f55dd158d1fc8ebe9ff161aa396f49091da118e\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3578538\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#990382}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: a66bcfd86fb58348aabb781b2ea51d55714ea1f4\n" }, { "commit": "d8c227955829741f0a171268ff1696fe4ef341f0", "tree": "3325360b35cd64be8beef9cddfe6a43c39e734cb", "parents": [ "7b0a74688f0f847f88044575cd43d9e6ca3f348f" ], "author": { "name": "Takashi Sakamoto", "email": "tasak@google.com", "time": "Thu Apr 07 12:50:06 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Apr 07 13:01:34 2022" }, "message": "partition_alloc: Copy base/bits.h to the PA library\n\n- Copied base/bits.h and base/bits_unittest.cc to\n//base/allocator/partition_allocator/base, and introduced\npartition_alloc::internal::base namespace.\n- Added migration_adapter.h to make base:: inside partition_alloc\nnamespace to use ::base instead of ::partition_alloc::internal::base.\n- Replaced some base:: with ::base because of typedef or not to update\nDEPS.\n\nBug: 1151236\n\nChange-Id: Ibfffcace7a55a7cc740c1229b8d6f4621d23e2f6\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3564028\nReviewed-by: Kalvin Lee \u003ckdlee@chromium.org\u003e\nReviewed-by: Kentaro Hara \u003charaken@chromium.org\u003e\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nCommit-Queue: Kentaro Hara \u003charaken@chromium.org\u003e\nOwners-Override: Kentaro Hara \u003charaken@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#989885}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: b0ba4bf35107ddf6a7bebd6cef3e5d4560cf1870\n" }, { "commit": "7b0a74688f0f847f88044575cd43d9e6ca3f348f", "tree": "d441382cc49d9b2997131652a961ed0e475f65b4", "parents": [ "821dbcc2a318c89e19786cdf68f30e645b32e396" ], "author": { "name": "Kalvin Lee", "email": "kdlee@chromium.org", "time": "Wed Apr 06 19:28:30 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Apr 06 19:38:30 2022" }, "message": "[PA] Adjust use of `base::*`\n\nWe\u0027re working on a CL to copy parts of `//base` into a PA-internal\ndirectory (and namespace). As a prerequisite, we would like to fix\nas many namespace lookup failures as we are able.\n\nThis CL adjusts some call sites to drop the `base::` qualification\nto prevent lookup into `partition_alloc::internal::base`.\n\nBug: 1151236\nChange-Id: I592a5b6930cfcec3a215295f693b6fbd6202eaac\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3560839\nReviewed-by: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCommit-Queue: Kalvin Lee \u003ckdlee@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#989572}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 2fe4f0da74efdbcf3f53b99817d27bc81cc9d78a\n" }, { "commit": "821dbcc2a318c89e19786cdf68f30e645b32e396", "tree": "926e968319ce4665c57cbd0663330e074153cdba", "parents": [ "ff56ec6dea1a17293134a1f3932fbc4bea8943df" ], "author": { "name": "Yuki Shiino", "email": "yukishiino@chromium.org", "time": "Mon Apr 04 08:22:02 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Apr 04 08:31:00 2022" }, "message": "[PA] Namespace migration (24 of N)\n\nThis change migrates //tools/memory/partition_alloc/\nfrom partition_alloc::internal::tools to partition_alloc::tools\nbecause it\u0027s not likely that //tools/memory/partition_alloc/\nwill become part of PA stand-alone library very soon, and\nit\u0027s also not likely that it uses the (copied, minimum) base\npart of the PA library.\n\nBug: 1288247\nChange-Id: I4406191deea6fdf36c18016a021fd6b2b2031461\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3552152\nReviewed-by: Kalvin Lee \u003ckdlee@chromium.org\u003e\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCommit-Queue: Yuki Shiino \u003cyukishiino@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#988408}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 0bb9b1e9b669b4c241a6d4a2c71afe505c9431ff\n" }, { "commit": "ff56ec6dea1a17293134a1f3932fbc4bea8943df", "tree": "c0cfbe46209602082461562999e3e17fd2cc7084", "parents": [ "b9fd5e8f4b5afd54828352bae86939aee6c5657d" ], "author": { "name": "Thiabaud Engelbrecht", "email": "thiabaud@google.com", "time": "Tue Mar 29 20:15:40 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Mar 29 20:26:55 2022" }, "message": "[tools/memory] Add tool for live-graphing of PA fragmentation.\n\nThis CL adds a script to allow us to see graphs of PA fragmentation for\na Chrome process, updated as Chrome runs.\n\nBug: 1217582\nChange-Id: I2dcae38a0585aaedbad954ac5a9ee6d0b0c507cf\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3537395\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCommit-Queue: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#986665}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 332723d61473ddf1b1e4cee9e8b03e11ae94a0c7\n" }, { "commit": "b9fd5e8f4b5afd54828352bae86939aee6c5657d", "tree": "2200355c8c25dd10e22960eaf831c61154009269", "parents": [ "22690aecc3a6a173cfe83622e1f3aa8fb226a227" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Tue Mar 29 12:49:05 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Mar 29 13:01:16 2022" }, "message": "[tools/memory] Do not build PartitionAlloc\u0027s tools when it\u0027s not used\n\nChange-Id: I0a1689775f972b77a29b214020906f3eac4e0b04\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3557449\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nReviewed-by: Egor Pasko \u003cpasko@chromium.org\u003e\nCommit-Queue: Egor Pasko \u003cpasko@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#986447}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: faaede670d9e565325a82b374f94285d1e045aa2\n" }, { "commit": "22690aecc3a6a173cfe83622e1f3aa8fb226a227", "tree": "b9881d11f118570827d8876773a737e00fea8e07", "parents": [ "92599d6fd1a306a062df628a0a9a7c7d57cc1358" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Mon Mar 28 10:41:54 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Mar 28 10:53:04 2022" }, "message": "[tools/memory] Log requested size in PartitionAlloc, analyze it\n\nWhen an allocation is made, the requested size is rounded up to the\nnearest bucket. This means that the initial allocation size is lost.\n\nThis CL uses the extra space in BRP\u0027s reference count to store the\ninitially requested size. It also adds a tool to collect and process a\nheap dump, and represent the memory waste incurred by BRP.\n\nBug: 1217582\nChange-Id: I0103ee8fbe8e791b984e8a72f9649adc4f7af7aa\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3540324\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#985908}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 544172191b760edc875f7659180e6909a7991f15\n" }, { "commit": "92599d6fd1a306a062df628a0a9a7c7d57cc1358", "tree": "dee73c2e8c7b7d19985a6771d2b23f75f76c537e", "parents": [ "abae8f52d391dcdd1faab0692f0b532876a17dda" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Mon Mar 21 09:59:58 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Mar 21 10:08:22 2022" }, "message": "[tools/memory] Better superpage plots\n\nThis extends the pa_dump_heap tool to get more detailed data about the\nstate of PartitionAlloc\u0027s heap. To that end, we copy the whole heap in\nthe remote process in pa_dump_heap\u0027s address space at the *same*\naddress.\n\nBug: 1217582\nChange-Id: I7e834beae5fa97ac571e94c01f6e19d87b617305\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3536657\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#983241}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 77664f82141054762ef77ae08f527293842642c4\n" }, { "commit": "abae8f52d391dcdd1faab0692f0b532876a17dda", "tree": "b0c67004b28cd3fc69bef747a3f7cf0bfb751641", "parents": [ "8a340b64dce4b578494881a1112cfa11b72330e2" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Fri Mar 18 10:25:53 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Mar 18 10:38:15 2022" }, "message": "[tools/memory] Add a tool to dump PartitionAlloc\u0027s heap\n\nThis adds a tool to extract PA\u0027s heap from a running process, and dump\nsome metadata into a JSON file.\n\nThis is similar in principle to e.g. pa_tcache_inspect, and uses the\nsame steps, namely reading a remote process memory, copying data\nlocally, and writing to a JSON output file. Then a python scrpts handles\ndrawing.\n\nIn this case, this shows the occupancy of super pages for the thread\ncache partition.\n\nBug: 1217582\nChange-Id: I953491d12c13aba1abb5ad876639420f0c20a26c\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3530330\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#982650}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 731edddb2c8d3bcd5a03af2f6d65e5dec71a8a9b\n" }, { "commit": "8a340b64dce4b578494881a1112cfa11b72330e2", "tree": "d35dbd46c00223a4b8ed4c73798f67e02c6ae95c", "parents": [ "feb42b9a96d13435e234653a4f94d62cb6df0ad8" ], "author": { "name": "Jann Horn", "email": "jannh@google.com", "time": "Tue Mar 15 10:58:11 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Mar 15 11:08:16 2022" }, "message": "[tools/memory] Show proper thread names even when pidns is used\n\nWhen a Chrome renderer looks at its own thread IDs, it sees the\nPID-namespaced IDs, but when looking at a renderer from outside,\nwe normally see the non-namespaced IDs.\n\nUse NSpid from /proc/$pid/task/$tid/status to look up the\nnamespaced IDs that the Chrome renderer threads think they have.\n\nThis makes pa_tcache_inspect actually show thread names for me.\n\nChange-Id: I40e4621e9a1ec905c58c22dd23c6bdc60a0cf383\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3522389\nAuto-Submit: Jann Horn \u003cjannh@google.com\u003e\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#981046}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 68a970d5d86123d08b309ef7ea2b5431a5c181bf\n" }, { "commit": "feb42b9a96d13435e234653a4f94d62cb6df0ad8", "tree": "a034c70dd2bd672c145f5f88471551b7d8816131", "parents": [ "da435db0891bc725c2dc96b2b24b89724daef027" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Thu Mar 10 09:45:01 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Mar 10 09:55:57 2022" }, "message": "[tools/memory] Better accounting and more precise graphs\n\nBug: 1217582\nChange-Id: I23435418d58df9f4df4e06191b535714d1c677d2\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3506511\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#979692}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 1ffd852290b36d773cc9bb3dcb138d238e211927\n" }, { "commit": "da435db0891bc725c2dc96b2b24b89724daef027", "tree": "5bd3e08bfc8b44dc25565a7a38b604b478cb7752", "parents": [ "3c34a54fcdfc90398f4a561c9b2b10aa03ebe4aa" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Tue Mar 08 10:05:18 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Mar 08 10:14:23 2022" }, "message": "[tools/memory] Plot Slot span utilization\n\nThis adds a python script to visualize the state of the active slot span\nlist for various bucket sizes. See the comments in plot_bucket_stats.py\non how to use it.\n\nBug: 1217582\nChange-Id: I2e060bd8ea11865557d3e2a4f6f7be90125a4f2c\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3503632\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#978662}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 6665e24ce57f1fc30efd0cf959681ffa696a8bfd\n" }, { "commit": "3c34a54fcdfc90398f4a561c9b2b10aa03ebe4aa", "tree": "28a8e9274dce69eccbfacd4358a72ad53767fb86", "parents": [ "4d131c20da34a03fcde256ed31a23115b171c172" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Fri Feb 18 09:31:35 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 18 09:40:31 2022" }, "message": "[tools/memory] Add more data to pa_tcache_inspect\n\nChange-Id: I6e86fb84915ea63dcd0d6f812de47a628650fb00\nBug: 1217582\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3461691\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#972872}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: bf9117206395767750b7daf9c16fa2c8a54538a8\n" }, { "commit": "4d131c20da34a03fcde256ed31a23115b171c172", "tree": "712514c76f5543c08ada66cefd93f4d571e97712", "parents": [ "7d12c5dcdd4f0b2543aac39edf5a5ef2306c60fa" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Mon Feb 14 10:01:43 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Feb 14 10:14:09 2022" }, "message": "[tools/memory] Dump bucket data to a JSON file in pa_tcache_inspect\n\nTo ease offline analysis, add a --json parameter to pa_tcache_inspect.\n\nBug: 1217582\nChange-Id: I24b592e0f2df513272eb583641424e1dc7af2cfd\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3456504\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#970551}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 5300a748b9ca34b51bec88c8d912795bbbbcd17b\n" }, { "commit": "7d12c5dcdd4f0b2543aac39edf5a5ef2306c60fa", "tree": "7002c64a2afd7990127938ca9905977f054a77c1", "parents": [ "5b5485d9df586b17d0a25d668989b2fb0174f334" ], "author": { "name": "Thiabaud Engelbrecht", "email": "thiabaud@google.com", "time": "Fri Feb 11 05:11:14 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 11 05:22:08 2022" }, "message": "[partition_alloc] Add tool for recording memory allocations.\n\nBug: 1295565\nChange-Id: If4b4da56924c83ef0cad2f78f367b6f9870d552f\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3415974\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCommit-Queue: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#969826}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 77639da217e2a141da52e1897e6092adc6dc3758\n" }, { "commit": "5b5485d9df586b17d0a25d668989b2fb0174f334", "tree": "81a0f4ad75691cbb693208e4bd59aaba904b3fa7", "parents": [ "8e8704fdfac5e10c5777f3093627a482f9dd2d47" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Fri Jan 21 17:04:22 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Jan 21 17:14:34 2022" }, "message": "[tools/memory] Display freelist size in pa_tcache_inspect\n\nFreelists are per-SlotSpan. This CL adds a rolling display of a SlotSpan\nfreelist size.\n\nExample output:\n\nFreelist size for active buckets of size \u003d 16\n18 53 259 221 387 338 319 650 750 526 269 1020 1022 1024\n\nBug: 1217582\nChange-Id: I51de25ab673ee3c84c45fef987b798029d952837\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3404652\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit Lize \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#961937}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 02d35417baf569b9c92f996eb8668c35603ee5bf\n" }, { "commit": "8e8704fdfac5e10c5777f3093627a482f9dd2d47", "tree": "bc76fcedccdc50bef7a5267bc9b03eac592e55f4", "parents": [ "0639a8c19c66272f5064fcaf4e60ca1d03a84976" ], "author": { "name": "Bartek Nowierski", "email": "bartekn@chromium.org", "time": "Thu Jan 13 10:39:07 2022" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 13 10:50:32 2022" }, "message": "[PA] Remove negative \"special\" values from SlotSpanMetadata\n\n- num_allocated_slots now uses a separate bit, instead of a negative\n value, to indicate that a slot span is marked as full (i.e. out of the\n active list).\n- empty_cache_index now uses a separate bit, instead of a negative\n value, to indicate being in the empty cache (cyclic buffer).\n\nExtra cleanups:\n- Lower the number of bits needed for slot-counting fields.\n- Make the maximum for empty cache index more precise. kMaxFreeableSpans\n is an array size, and not used as a value in itself.\n- Refactor FreeSlowPath() to avoid the confusing recursion.\n\nWe now have 11 spare bits in SlotSpanMetadata!\n\nChange-Id: I49ba0c8f0ee15793ad97fdb019d3d9d8f22e15e1\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3352265\nReviewed-by: Benoit Lize \u003clizeb@chromium.org\u003e\nCommit-Queue: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#958508}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: e5b03e85ea180d1d1ab0dec471c7fd5d1706a9e4\n" }, { "commit": "0639a8c19c66272f5064fcaf4e60ca1d03a84976", "tree": "c7ddf693660e45170de4b6012d13c05fb6f5371f", "parents": [ "bfbdc93fb6b777a69e4fa0d6e4339cbddb311270" ], "author": { "name": "Henrique Ferreiro", "email": "hferreiro@igalia.com", "time": "Tue Nov 30 13:31:56 2021" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Nov 30 13:42:16 2021" }, "message": "Use #!/usr/bin/env python/2/3 in remaining .py files\n\nReplace every shebang like /usr/bin/python[23] with /usr/bin/env\npython[23] as appropriate in Chromium and Blink, except for WPT. Also,\nadd a presubmit check to disallow the former going forward.\n\nbuild/print_python_deps.py is changed from using python2.7 to python2.\n\nremoting/ python scripts are left out as they are meant to be run in\nusers\u0027 machines.\n\nBug: 1191100\nChange-Id: If7ebf7bd8ad9b2695a471e0a403a592815a0d959\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2537832\nCommit-Queue: Henrique Ferreiro \u003chferreiro@igalia.com\u003e\nReviewed-by: Dirk Pranke \u003cdpranke@google.com\u003e\nReviewed-by: Colin Blundell \u003cblundell@chromium.org\u003e\nReviewed-by: Jamie Walch \u003cjamiewalch@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#946427}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: f9819f2e35220e1a25c548f19aa3afb6b37f642c\n" }, { "commit": "bfbdc93fb6b777a69e4fa0d6e4339cbddb311270", "tree": "8050c847140db9f2d22efcbbf50611fa01b1a824", "parents": [ "20319341da9c7b8a36aaea2c28d527cc12210dfe" ], "author": { "name": "Peter Kasting", "email": "pkasting@chromium.org", "time": "Thu Nov 04 01:34:43 2021" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Nov 04 01:42:46 2021" }, "message": "Fix -Wshadow instances: tools/\n\nBug: 794619\nChange-Id: Ieed8b4e91a8e2e447590a1aea4940a339377a63d\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3257904\nAuto-Submit: Peter Kasting \u003cpkasting@chromium.org\u003e\nReviewed-by: Bruce Dawson \u003cbrucedawson@chromium.org\u003e\nCommit-Queue: Peter Kasting \u003cpkasting@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#938095}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 885818381294ea71c21e771b17e94b0800938d66\n" }, { "commit": "20319341da9c7b8a36aaea2c28d527cc12210dfe", "tree": "52512ff9bf52852d570b46c22dc17fe10a04d2a4", "parents": [ "4dc64cab424aec02f60d892e857cc9a0155ea4fc" ], "author": { "name": "Benoit Lize", "email": "lizeb@chromium.org", "time": "Wed Oct 27 15:51:24 2021" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Oct 27 16:01:20 2021" }, "message": "[tools/memory] Add allocated/committed/virtual size to pa_tcache_inspect\n\nReport these three sizes. Example output:\n\nMax committed size \u003d 67216kiB\nAllocated/Committed/Virtual \u003d 35187 / 53200 / 96256 kiB\n\nChange-Id: I2633d9b1f186a08e19441645f7f80a7769734d80\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3247356\nReviewed-by: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCommit-Queue: Benoit L \u003clizeb@chromium.org\u003e\nCr-Commit-Position: refs/heads/main@{#935440}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: e5aa3844cba89f533345393ba9acf92e19564a89\n" }, { "commit": "4dc64cab424aec02f60d892e857cc9a0155ea4fc", "tree": "ffa6054287f034ca3cf4d00a2d007ac8583307fe", "parents": [ "e0429447bd43b7bc3d9f2b7008cb9ce3b76c7885" ], "author": { "name": "Thiabaud Engelbrecht", "email": "thiabaud@google.com", "time": "Mon Oct 04 15:32:56 2021" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Oct 04 15:43:59 2021" }, "message": "[partition_alloc] Add tool to compute the internal fragmentation of\nPartitionAlloc.\n\nThis script takes a trace from about:tracing, and a list of bucket\nsizes, and computes how much extra space would be used if the\nallocations in the trace were done with the given bucket sizes. This\ngives us a rough estimate of how much extra memory a given bucket\nmapping would use, computed statically based on surviving allocations\nonly.\n\nBug: 1238858\nChange-Id: Iaa9b24b1d7669f3b12c71396911b47990ae402e1\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3188991\nReviewed-by: Egor Pasko \u003cpasko@chromium.org\u003e\nReviewed-by: Bartek Nowierski \u003cbartekn@chromium.org\u003e\nCommit-Queue: Thiabaud Engelbrecht \u003cthiabaud@google.com\u003e\nCr-Commit-Position: refs/heads/main@{#927688}\nNOKEYCHECK\u003dTrue\nGitOrigin-RevId: 6d2fa574b67fb6462be475fce5adbc482352e437\n" } ], "next": "e0429447bd43b7bc3d9f2b7008cb9ce3b76c7885" }