[rpms/chromium] epel9-next: removed patch for 64kpage support on aarch64, merged by upstream

[Than Ngo <than@redhat.com>]

A new commit has been pushed.

Repo   : rpms/chromium
Branch : epel9-next
Commit : ada748216c555e29dcae054214d045c3ae0f6a90
Author : Than Ngo <than@redhat.com>
Date   : 2024-09-06T20:23:23+02:00
Stats  : +0/-130 in 2 file(s)
URL    : https://src.fedoraproject.org/rpms/chromium/c/ada748216c555e29dcae054214d045c3ae0f6a90?branch=epel9-next

Log:
removed patch for 64kpage support on aarch64, merged by upstream

---
diff --git a/chromium-124-el8-support-64kpage.patch b/chromium-124-el8-support-64kpage.patch
deleted file mode 100644
index ed4a4e6..0000000
--- a/chromium-124-el8-support-64kpage.patch
+++ /dev/null
@@ -1,127 +0,0 @@
-diff -up chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_bucket.h.orig chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_bucket.h
---- chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_bucket.h.orig	2024-04-12 21:56:54.000000000 +0200
-+++ chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_bucket.h	2024-04-14 20:53:16.043867871 +0200
-@@ -147,7 +147,13 @@ struct PartitionBucket {
-   // Returns a slot number starting from the beginning of the slot span.
-   PA_ALWAYS_INLINE size_t GetSlotNumber(size_t offset_in_slot_span) const {
-     // See the static assertion for `kReciprocalShift` above.
--    PA_DCHECK(offset_in_slot_span <= kMaxBucketed);
-+    // TODO(casey.smalley@arm.com): triggers on Aarch64/Linux
-+    // systems with 64k system pages. Constants need to be
-+    // adjusted to prevent different parts of the allocator
-+    // from overlapping. For now this will allow 64k pages
-+    // to function on Aarch64/Linux systems, albeit not
-+    // very efficiently.
-+    PA_DCHECK(internal::SystemPageSize() == (size_t{1} << 16) || offset_in_slot_span <= kMaxBucketed);
-     PA_DCHECK(slot_size <= kMaxBucketed);
- 
-     const size_t offset_in_slot =
-diff -up chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_page_constants.h.orig chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_page_constants.h
---- chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_page_constants.h.orig	2024-04-14 20:53:16.043867871 +0200
-+++ chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_page_constants.h	2024-04-14 21:39:18.147052845 +0200
-@@ -17,10 +17,8 @@ namespace partition_alloc::internal {
- // PartitionPageSize() is 4 times the OS page size.
- static constexpr size_t kMaxSlotsPerSlotSpan = 4 * (1 << 14) / kSmallestBucket;
- #elif defined(PARTITION_ALLOCATOR_CONSTANTS_POSIX_NONCONST_PAGE_SIZE)
--// System page size can be 4, 16, or 64 kiB on Linux on arm64. 64 kiB is
--// currently (kMaxSlotsPerSlotSpanBits == 13) not supported by the code,
--// so we use the 16 kiB maximum (64 kiB will crash).
--static constexpr size_t kMaxSlotsPerSlotSpan = 4 * (1 << 14) / kSmallestBucket;
-+// System page size can be 4, 16, or 64 kiB on Linux on AArch64.
-+static constexpr size_t kMaxSlotsPerSlotSpan = 4 * (1 << 16) / kSmallestBucket;
- #else
- // A slot span can "span" multiple PartitionPages, but then its slot size is
- // larger, so it doesn't have as many slots.
-diff -up chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_root.cc.orig chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_root.cc
---- chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_root.cc.orig	2024-04-14 20:53:16.044867889 +0200
-+++ chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_root.cc	2024-04-14 21:50:57.926716718 +0200
-@@ -952,12 +952,11 @@ void PartitionRoot::Init(PartitionOption
-              (internal::SystemPageSize() == (size_t{1} << 14)));
- #elif BUILDFLAG(IS_LINUX) && defined(ARCH_CPU_ARM64)
-     // Check runtime pagesize. Though the code is currently the same, it is
--    // not merged with the IS_APPLE case above as a 1 << 16 case needs to be
--    // added here in the future, to allow 64 kiB pagesize. That is only
--    // supported on Linux on arm64, not on IS_APPLE, but not yet present here
--    // as the rest of the PartitionAlloc does not currently support it.
-+    // not merged with the IS_APPLE case above as a 1 << 16 case is only
-+    // supported on Linux on AArch64.
-     PA_CHECK((internal::SystemPageSize() == (size_t{1} << 12)) ||
--             (internal::SystemPageSize() == (size_t{1} << 14)));
-+             (internal::SystemPageSize() == (size_t{1} << 14)) ||
-+             (internal::SystemPageSize() == (size_t{1} << 16)));
- #endif
- 
-     ::partition_alloc::internal::ScopedGuard guard{lock_};
-diff -up chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_page.h.me chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_page.h
---- chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_page.h.me	2024-04-14 22:08:40.655011536 +0200
-+++ chromium-124.0.6367.49/base/allocator/partition_allocator/src/partition_alloc/partition_page.h	2024-04-14 22:12:46.665731577 +0200
-@@ -86,27 +86,25 @@ struct SlotSpanMetadata {
- 
-   // CHECK()ed in AllocNewSlotSpan().
-   // The maximum number of bits needed to cover all currently supported OSes.
--  static constexpr size_t kMaxSlotsPerSlotSpanBits = 13;
-+  static constexpr size_t kMaxSlotsPerSlotSpanBits = 15;
-   static_assert(kMaxSlotsPerSlotSpan < (1 << kMaxSlotsPerSlotSpanBits), "");
- 
--  // |marked_full| isn't equivalent to being full. Slot span is marked as full
--  // iff it isn't on the active slot span list (or any other list).
--  uint32_t marked_full : 1;
-   // |num_allocated_slots| is 0 for empty or decommitted slot spans, which can
-   // be further differentiated by checking existence of the freelist.
--  uint32_t num_allocated_slots : kMaxSlotsPerSlotSpanBits;
--  uint32_t num_unprovisioned_slots : kMaxSlotsPerSlotSpanBits;
-+  uint16_t num_allocated_slots : kMaxSlotsPerSlotSpanBits;
-+  uint16_t num_unprovisioned_slots : kMaxSlotsPerSlotSpanBits;
-+
-+  // |marked_full| isn't equivalent to being full. Slot span is marked as full
-+  // iff it isn't on the active slot span list (or any other list).
-+  bool marked_full : 1;
- 
-  private:
--  const uint32_t can_store_raw_size_ : 1;
--  uint32_t freelist_is_sorted_ : 1;
--  uint32_t unused1_ : (32 - 1 - 2 * kMaxSlotsPerSlotSpanBits - 1 - 1);
-+  const uint8_t can_store_raw_size_ : 1;
-+  uint8_t freelist_is_sorted_ : 1;
-   // If |in_empty_cache_|==1, |empty_cache_index| is undefined and mustn't be
-   // used.
--  uint16_t in_empty_cache_ : 1;
--  uint16_t empty_cache_index_
--      : kMaxEmptyCacheIndexBits;  // < kMaxFreeableSpans.
--  uint16_t unused2_ : (16 - 1 - kMaxEmptyCacheIndexBits);
-+  bool in_empty_cache_ : 1;
-+  uint8_t empty_cache_index_ : kMaxEmptyCacheIndexBits;  // < kMaxFreeableSpans.
-   // Can use only 48 bits (6B) in this bitfield, as this structure is embedded
-   // in PartitionPage which has 2B worth of fields and must fit in 32B.
- 
-@@ -246,18 +244,13 @@ static_assert(sizeof(SlotSpanMetadata) <
-               "SlotSpanMetadata must fit into a Page Metadata slot.");
- 
- inline constexpr SlotSpanMetadata::SlotSpanMetadata() noexcept
--    : marked_full(0),
--      num_allocated_slots(0),
-+    : num_allocated_slots(0),
-       num_unprovisioned_slots(0),
-+      marked_full(0),
-       can_store_raw_size_(false),
-       freelist_is_sorted_(true),
--      unused1_(0),
-       in_empty_cache_(0),
--      empty_cache_index_(0),
--      unused2_(0) {
--  (void)unused1_;
--  (void)unused2_;
--}
-+      empty_cache_index_(0) {}
- 
- inline SlotSpanMetadata::SlotSpanMetadata(const SlotSpanMetadata&) = default;
- 
-@@ -752,7 +745,7 @@ PA_ALWAYS_INLINE void SlotSpanMetadata::
- 
-   size_t num_slots_per_span = bucket->get_slots_per_span();
-   PA_DCHECK(num_slots_per_span <= kMaxSlotsPerSlotSpan);
--  num_unprovisioned_slots = static_cast<uint32_t>(num_slots_per_span);
-+  num_unprovisioned_slots = static_cast<uint16_t>(num_slots_per_span);
-   PA_DCHECK(num_unprovisioned_slots);
- 
-   ToSuperPageExtent()->IncrementNumberOfNonemptySlotSpans();

diff --git a/chromium.spec b/chromium.spec
index 4504799..bbda085 100644
--- a/chromium.spec
+++ b/chromium.spec
@@ -354,8 +354,6 @@ Patch150: chromium-124-qt6.patch
 # it is not supported in old glibc < 2.30, error: fatal error: 'sys/ifunc.h' file not found
 Patch305: chromium-124-el8-arm64-memory_tagging.patch
 Patch306: chromium-127-el8-ifunc-header.patch
-# 64kpage support on aarch64 (el8)
-Patch308: chromium-124-el8-support-64kpage.patch
 
 # enable fstack-protector-strong
 Patch312: chromium-123-fstack-protector-strong.patch
@@ -1081,7 +1079,6 @@ Qt6 UI for chromium.
 %ifarch aarch64
 %patch -P305 -p1 -b .el8-memory_tagging
 %patch -P306 -p1 -b .el8-ifunc-header
-%patch -P308 -p1 -b .el8-support-64kpage.patch
 %endif
 %endif