From aadaefa800d0725c89dbde3f4259a8f6d2a29153 Mon Sep 17 00:00:00 2001 From: Anton Firszov Date: Sat, 13 Nov 2021 21:19:28 +0100 Subject: [PATCH] memory clearing should not be UniformUnmanagedMemoryPool concern --- .../Internals/UniformUnmanagedMemoryPool.cs | 7 +------ .../UniformUnmanagedMemoryPoolMemoryAllocator.cs | 15 ++++++++++++--- 2 files changed, 13 insertions(+), 9 deletions(-) diff --git a/src/ImageSharp/Memory/Allocators/Internals/UniformUnmanagedMemoryPool.cs b/src/ImageSharp/Memory/Allocators/Internals/UniformUnmanagedMemoryPool.cs index cfb0b3eb5d..3d5c0da2c3 100644 --- a/src/ImageSharp/Memory/Allocators/Internals/UniformUnmanagedMemoryPool.cs +++ b/src/ImageSharp/Memory/Allocators/Internals/UniformUnmanagedMemoryPool.cs @@ -52,7 +52,7 @@ namespace SixLabors.ImageSharp.Memory.Internals public int Capacity { get; } - public UnmanagedMemoryHandle Rent(AllocationOptions allocationOptions = AllocationOptions.None) + public UnmanagedMemoryHandle Rent() { UnmanagedMemoryHandle[] buffersLocal = this.buffers; @@ -81,11 +81,6 @@ namespace SixLabors.ImageSharp.Memory.Internals buffer = UnmanagedMemoryHandle.Allocate(this.BufferLength); } - if (allocationOptions.Has(AllocationOptions.Clean)) - { - this.GetSpan(buffer).Clear(); - } - return buffer; } diff --git a/src/ImageSharp/Memory/Allocators/UniformUnmanagedMemoryPoolMemoryAllocator.cs b/src/ImageSharp/Memory/Allocators/UniformUnmanagedMemoryPoolMemoryAllocator.cs index 3aa2bbf83a..5fc563d095 100644 --- a/src/ImageSharp/Memory/Allocators/UniformUnmanagedMemoryPoolMemoryAllocator.cs +++ b/src/ImageSharp/Memory/Allocators/UniformUnmanagedMemoryPoolMemoryAllocator.cs @@ -98,10 +98,14 @@ namespace SixLabors.ImageSharp.Memory if (lengthInBytes <= this.poolBufferSizeInBytes) { - UnmanagedMemoryHandle array = this.pool.Rent(options); + UnmanagedMemoryHandle array = this.pool.Rent(); if (array != null) { - return new UniformUnmanagedMemoryPool.FinalizableBuffer(this.pool, array, length); + var buffer = new UniformUnmanagedMemoryPool.FinalizableBuffer(this.pool, array, length); + if (options.Has(AllocationOptions.Clean)) + { + buffer.Clear(); + } } } @@ -124,10 +128,15 @@ namespace SixLabors.ImageSharp.Memory if (totalLengthInBytes <= this.poolBufferSizeInBytes) { // Optimized path renting single array from the pool - UnmanagedMemoryHandle array = this.pool.Rent(options); + UnmanagedMemoryHandle array = this.pool.Rent(); if (array != null) { var buffer = new UniformUnmanagedMemoryPool.FinalizableBuffer(this.pool, array, (int)totalLength); + if (options.Has(AllocationOptions.Clean)) + { + buffer.Clear(); + } + return MemoryGroup.CreateContiguous(buffer, options.Has(AllocationOptions.Clean)); } }