[Webkit-unassigned] [Bug 230283] New: [bmalloc] Simplify LargeRange constructors and remove meaningless one

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 14 16:22:05 PDT 2021


https://bugs.webkit.org/show_bug.cgi?id=230283

            Bug ID: 230283
           Summary: [bmalloc] Simplify LargeRange constructors and remove
                    meaningless one
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: bmalloc
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: Basuke.Suzuki at sony.com
                CC: ggaren at apple.com

LargeRange is mainly created in Heap and most usages are simply memory region with full of physical pages. This patch add simple constructor with address and size, which generates LargeRange with full of pages.

Also the constructor which accept Range is used in only one place, shrinkLarge(), and that's also can be simplify with this new constructor because the range is only created to be passed to shrinkLarge(). Size information is fetched in shrinkLarge anyway and at worst, it is confusing not to use size information from the passed Range argument. Replacing the argument with simple void pointer and there's no change for its behavior  at all.

Actually I still cannot understand the behavior or meaning of physical page information stored in LargeRange. This is the first step of the investigation to simplify the case with full physical pages.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20210914/75a6143e/attachment.htm>


More information about the webkit-unassigned mailing list