[swift-server-dev] [swift-dev] Performance Refinement of Data
Alex Blewitt
alblue at apple.com
Tue Nov 29 09:36:54 CST 2016
> On 29 Nov 2016, at 14:12, Helge Heß via swift-server-dev <swift-server-dev at swift.org> wrote:
>
> On 28 Nov 2016, at 20:04, Alex Blewitt via swift-server-dev <swift-server-dev at swift.org <mailto:swift-server-dev at swift.org>> wrote:
>> This might be of interest to the swift-server-dev community who aren't on either of the other lists.
>
> Thanks.
>
> Hm, I don’t know. Something I don’t quite get is why this is even required. What is really different between that `Data` and a `[ UInt8 ]`? Just the bridging to NSData (which we don’t have on Linux), and couldn’t all that bridging stuff described be done against a special [UInt8] array-backend class too?
For one thing, a [UInt8] backed type has a fixed structure in memory (for example, it must be contiguous). A Data type has an opaque storage type which could permit a linked list of blocks (for example). It's also possible to create a Data object which is based on a subset of an original Data without copying. There are also custom deallocator routines which do things differently based on what type the data came from (for example, unmap or free).
Of course the other reason is that Data exists across a number of existing APIs and that's unlikely to change. So making this more efficient will help those use cases.
> The performance patterns attached are focused around a single data object. Also: "Data does not follow this pattern. Often it is sourced from files on disk (which could be exceedingly large)”.
> For the server I think the focus needs to be quite different.
>
> So what are the requirements for a ‘byte bucket’ data structure on the server. Some stuff I can think of:
>
> - There are a lot (“millions") of read and write calls with
> tons of small byte buffers (essentially socket buffer
> sizes, but when framing has been processed even smaller ones ...)
Yes, that's true. Dispatch has a mechanism to provide these Data types as well.
> - Copying of those buffers should be avoided as much as possible
> - Ideally it should be possible to reuse such buffers (instead
> of malloc/free’ing them at a high rate)
I concur. In fact the custom Data deallocator is precisely for this kind of purpose; you can use it to create a magazine of such Data elements, and then return them to the Data pool when they're finished with.
https://developer.apple.com/reference/foundation/data.deallocator
> - It is rather rare that the buffers in servers are itself
> modified, usually you create a new, transformed buffer (or
> other object, like a String or Int) out of existing ones
> - except for one operation: you often slice them (e.g. when
> encountering a HTTP chunk marker, or some other framing in
> a protocol or data format).
> - that might imply that the CoW overhead - even if small -
> may be unnecessarily high
> - instead there needs to be a focus an dealing with ranges
Data has a method "split" which can be used to return multiple slices of the underlying Data object, which is inherited from the Collection type. This gives an array of ranges, which can be used to pass on to other elements.
> - It should be highly efficient to create String’s from such
> byte buckets (arrays). while it should be delayed to create
> such to the latest point possible, it quite often will be
> the API the top-level framework user is expecting (JSON,
> query-strings, XML, whatever).
String can be used to be created from a Data with init:
https://developer.apple.com/reference/swift/string/1418413-init
I would expect that a high-performance JSON or XML parser that takes a Data would probably find it more efficient to iterate through the contents directly rather than going through a String representation first, though.
> - There needs to be something to represent arrays of those
> buckets, Apache calls them brigades. One could use Arrays
> for such, but maybe they should be lists instead.
> (this is required to avoid copying split buffers into
> larger ones, and is supported by writev/readv)
>
> - Except for some special cases like caches - unlike mentioned
> in the proposal - server apps should rarely ‘source byte
> buckets from disks’.
> Instead of loading the buckets into memory, maybe there
> should be a way to have byte-buckets which are files.
> NSData could do mmap’ed files, what you more often really
> want in the server context is `sendfile()` though.
> - some kind of a ‘FileDescriptorBackedBuffer’ protocol
>
> - Scanning of buffers should be really fast - and by that I
> mean C `index()` like fast, not `for byte in bytes {}` array
> fast. But that is desirable for [UInt8] too :-)
>
> There is probably more, but I think the stuff above are things that should be addressed :-)
I think performance is definitely high up on the list of things that are required, but we should also look at the aspects of using the standard buffer type provided by the platform. If we have performance tests/metrics that show a performance delta to using a plan [UInt8] or equivalent then it would be good to test those or provide them as test cases for the core Swift team.
Alex
> It may also be worth considering whether that ‘buffer’ data structure should only be able to carry bytes, or other payloads as well (along the lines of http://noze.io/noze4nonnode/#streams <http://noze.io/noze4nonnode/#streams>). But maybe that is too ambitious for v1 :-)
>
> hh
>
> P.S.: This may be good reading material: http://www.apachetutor.org/dev/brigades <http://www.apachetutor.org/dev/brigades>
>
>> Alex
>>
>>> Begin forwarded message:
>>>
>>> From: Philippe Hausler via swift-dev <swift-dev at swift.org>
>>> Subject: [swift-dev] Performance Refinement of Data
>>> Date: 19 November 2016 at 19:10:32 GMT
>>> To: swift-dev at swift.org, swift-corelibs-dev at swift.org
>>> Reply-To: Philippe Hausler <phausler at apple.com>
>>>
>>> Performance Refinement of Data
>>> Introduction
>>> In Swift 3 the Foundation team introduced a new structural type to represent NSData exposed into Swift.
>>>
>>> Data allows developers to interact with binary data with value semantics, which is often more appropriate than using pointers like UnsafeMutablePointer. Having an encapsulating type to abstract the common operations is often quite advantageous to tasks like parsing, where types like String may not be appropriate for or have hidden performance "gotchas".
>>>
>>> Data can easily be a critical point of performance for many tasks. Data is an appropriate common currency of transacting a safe managed buffer of bytes that interoperates well with existing Cocoa APIs. This means that it should be tuned for performance as much as possible.
>>>
>>> Motivation
>>> There are several outstanding performance improvements which can be made to Data; issues regarding Data's performance have been raised with the Foundation team both publicly and privately, and we would like to address those.
>>>
>>> Data should be as fast as possible. Currently, most of the backing of Data is implemented in Foundation, while being quite fast for reallocations and other operations, this means that calls are made between Swift and Objective-C even for simple things like count for every access.
>>>
>>> This Swift–Objective-C boundary means that no inlining can be performed across it; even when we have full control over the backing implementation and the caller, what would normally be just a single offset load instructions ends up becoming many just for the benefit of an objc_msgSend (not to mention ARC operations). Even though the calls are heavily optimized, they will never be as fast as a single instruction.
>>>
>>> Proposed solution
>>> In order to make Data as fast as possible the implementation needs to be inlined; since that is one of the best performance optimizations that Swift can offer. To do this it requires a re-think of how Data and NSData interact. This means that the structure Data will need to adopt certain attributes that will allow it to be inlined into the call-sites. The function dispatch overhead will be reduced but also optimization passes like cold paths and branch elimination can be possibilities for the compiler to do the best possible thing for the call site.
>>>
>>> Data will adopt the annotation @inline(__always) in key locations and use a non-Objective-C backing class to store the pointer and length (and other internal ivars). That backing object will allow for a reduction of capture points as well as avoid extra retain/releases that were added for mutation detection.
>>>
>>> Instead of using _MutablePairBoxing (which uses closures to map invocations to references or apply mutations with copy on write semantics) the new backing implementation can easily be applied with copy on write semantics without any Unmanaged "dancing". That "dancing" complicates code and it can be made considerably simpler. Furthermore, avoiding this "dance" can reduce the calls to retain and release down to zero in the application of mutations in unique referenced cases as well as mapping non mutating invocations to backing storage.
>>>
>>> Subclassing the reference type NSData is still something that Foundation should support for the wrapping of the reference type in a structure. This effectively means there are five types of backing for Data: Swift-implemented, immutable NSData, mutable NSMutableData, custom subclasses of NSData, and custom subclasses of NSMutableData. These specific cases are delineated to offer the most efficient inline cases possible.
>>>
>>> Since Foundation can enforce a no dynamic dispatch needed contract with itself in the cases of the standard class cluster members of NSData and NSMutableData Foundation can assure these cases are acceptable to not need dynamic dispatch for every time bytes or length are accessed and the values can be cached until the data is mutated or disposed of. In the cases where a subclass is used of course all bets are off and every point requires dynamically calling out.
>>>
>>> In short this will mean that fetching the count of a Data can be optimized to a single branch and load from an offset and this same optimization can be applied to many other methods on Data.
>>>
>>> Bridging to and from Objective-C
>>> Many of the sources that Data is derived from are sourced from the SDKs written in Objective-C. For other types like Array,Set, Dictionary, or String the objects returned are many times not very large. Arrays may have a handful of objects, strings may only be a few hundred characters and so on. In these small cases it makes sense to "eagerly" bridge those reference types into a more inline-able version (there are exclusions to this but in general it is most often the case).
>>>
>>> Data does not follow this pattern. Often it is sourced from files on disk (which could be exceedingly large) or results from network transactions of downloads. These cases would definitely suffer from having an "eager" O(n) bridge; due to not only memory allocation duplications to hold both backing stores but also to the byte copy from the reference type to the value type. Data should be fast no matter where it came from unless it is truly unknown on it's dynamic dispatch requirements.
>>>
>>> To build a Data that is fast for inline optimizations the bytes pointer and length need to be cached for the duration of the object. When as is used to bridge a custom reference to Data dynamic dispatch must occur on every call to count and every time bytes are touched but if the Data is known to be obtained from a source that we can control the dynamic dispatch expectations that dynamic dispatch can be elided and behavior can be preserved by mimicking the Objective-C implementation in Swift.
>>>
>>> Bridging in the other direction also has some distinct performance optimizations that can be taken advantage of as well.
>>>
>>> When the lifespan of the callout to Objective-C is well known the cases of Swift constructed Data can easily pass a NSData with a no-copy of the backing buffer. It is the responsibility of the Objective-C APIs to appropriately either not directly retain past the scope of the call or copy in the cases of long lasting references. Any Objective-C method or function that takes a NSData and just retains or unsafely stores it past the function callout is likely incorrect and has bugs no matter the language it was invoked in. This case where the Data is created in Swift to bridge it only needs to allocate the wrapper NSData but no O(n) copy needs to occur (unless it is holding a reference as previously stated).
>>>
>>> The final case of bridging is when a Data is obtained from Objective-C and then passed directly back to Objective-C. The compiler has potentials of optimizations in direct callout cases such as returnsAData() as NSData with "peephole" optimizations but these are only really enforceable in limited scope (sometimes just the same line of code). Since the backing store can hold a reference to the reference type the bridge method (when not mutated) in those cases can pass that back over to Objective-C. For mutated versions a copy of that mutated version can be passed along as well (taking advantage of any optimizations the dynamic dispatch affords for calls to copy).
>>>
>>> Detailed performance breakdown
>>> Each graph below is a comparison between the Swift 3 Data and the new version of Data for each of the inline cases. The horizontal axis in each graph represent N and the vertical axis in each graph represents the sampled duration in nanoseconds. Each data set in the plots are an average over 100 (unless otherwise specified) per value of N. The attached graphs were generated from optimized builds on a Mac Pro (Late 2013) 3.5 GHz 6-Core Intel Xeon E5 with 16 GB 1866 MHz DDR3.
>>>
>>> func createSampleData(ofLength N: Int) -> Data {
>>> var buffer = [UInt8](repeating: 0, count: N)
>>> return buffer.withUnsafeMutableBytes { (buffer: UnsafeMutableRawBufferPointer) -> Data in
>>> arc4random_buf(buffer.baseAddress!, N)
>>> return Data(bytes: buffer.baseAddress!, count: N)
>>> }
>>> }
>>>
>>> func createSampleDataReference(ofLength N: Int) -> NSData {
>>> var buffer = [UInt8](repeating: 0, count: N)
>>> return buffer.withUnsafeMutableBytes { (buffer: UnsafeMutableRawBufferPointer) -> NSData in
>>> arc4random_buf(buffer.baseAddress!, N)
>>> return NSData(bytes: buffer.baseAddress, length: N)
>>> }
>>> }
>>>
>>> func createSampleArray(ofLength N: Int) -> [UInt8] {
>>> var buffer = [UInt8](repeating: 0, count: N)
>>> buffer.withUnsafeMutableBytes { (buffer: UnsafeMutableRawBufferPointer) -> Void in
>>> arc4random_buf(buffer.baseAddress!, N)
>>> }
>>> return buffer
>>> }
>>>
>>> Accessing count
>>>
>>> This should be a O(1) operation. The y axis is measured in nanoseconds sampled over 100000 iterations.
>>>
>>> // setup
>>> let data = createSampleData(ofLength: N)
>>> // start measuring
>>> _ = data.count
>>> // end measuring
>>>
>>> <access_count.png>
>>>
>>> Subscripting
>>>
>>> This should be a O(1) operation. The y axis is measured in nanoseconds sampled over 100000 iterations.
>>>
>>> // setup
>>> let data = createSampleData(ofLength: N)
>>> // start measuring
>>> _ = data[index]
>>> // end measuring
>>>
>>> <getting_subscript.png>
>>>
>>> // setup
>>> var data = createSampleData(ofLength: N)
>>> // start measuring
>>> data[index] = 0x00
>>> // end measuring
>>>
>>> <setting_subscript.png>
>>>
>>> Appending
>>>
>>> This should be a O(N) operation
>>>
>>> // setup
>>> let dataToAppend = createSampleData(ofLength: N)
>>> var data = Data()
>>> // start measuring
>>> data.append(dataToAppend)
>>> // end measuring
>>>
>>> <append_n_bytes_with_data.png>
>>>
>>> // setup
>>> let arrayToAppend = createSampleArray(ofLength: N)
>>> var data = Data()
>>> // start measuring
>>> data.append(contentsOf: arrayToAppend)
>>> // end measuring
>>>
>>> <append_array_of_bytes.png>
>>>
>>> The new version is still O(N) just a much smaller constant multiplier.
>>>
>>> var data = Data()
>>> // start measuring
>>> for _ in 0..<N {
>>> data.append(contentsOf: [0xFF, 0xFE, 0xFD, 0xFC, 0xFB, 0xFA])
>>> }
>>> //end measuring
>>>
>>> <append_n_arrays.png>
>>>
>>> Replacing sub ranges
>>>
>>> // setup
>>> var data = createSampleData(ofLength: N)
>>> // start measuring
>>> data.replaceSubrange(0..<N, with: replacement)
>>> // end measuring
>>>
>>> <replace_entire_subrange.png>
>>>
>>> // setup
>>> var data = createSampleData(ofLength: N)
>>> // start measuring
>>> data.replaceSubrange(0..<min(N, 5), with: replacement)
>>> // end measuring
>>>
>>> <replace_fixed_subrange.png>
>>>
>>> Growth of count
>>>
>>> // setup
>>> var data = Data()
>>> // start measuring
>>> data.count = N
>>> // end measuring
>>>
>>> <grow_small.png>
>>>
>>> // setup
>>> var data = Data()
>>> // start measuring
>>> data.count = N
>>> // end measuring
>>>
>>> <grow_large.png>
>>>
>>> // setup
>>> var data = Data()
>>> data.count = starting
>>> // start measuring
>>> data.count = N
>>> // end measuring
>>>
>>> <grow_from_mid_to_large.png>
>>>
>>> Bridging to reference types
>>>
>>> This should be a O(1) operation. In bridging to a reference case the previous implementation was a bit faster. The only real extra overhead here is an allocation of the NSData object since the Swift backed Data has no existing reference type to pass along. There are a few extra optimizations that can be done in this path to reduce it by the approximately 150 nanosecond difference. In practice the cases where Data is being bridged back out to Objective-C are usually cases like writing to a file or socket which dwarf that 150 nanosecond differential.
>>>
>>> // setup
>>> let data = createSampleData(ofLength: N)
>>> // start measuring
>>> _ = data as NSData
>>> // end measuring
>>>
>>> <bridge_to_objectivec.png>
>>>
>>> Bridging from reference types
>>>
>>> This should be a O(1) operation
>>>
>>> // setup
>>> let data = createSampleDataReference(ofLength: N)
>>> // start measuring
>>> _ = data as Data
>>> // end measuring
>>>
>>> <bridge_from_objectivec.png>
>>> _______________________________________________
>>> swift-dev mailing list
>>> swift-dev at swift.org
>>> https://lists.swift.org/mailman/listinfo/swift-dev
>>
>> _______________________________________________
>> swift-server-dev mailing list
>> swift-server-dev at swift.org <mailto:swift-server-dev at swift.org>
>> https://lists.swift.org/mailman/listinfo/swift-server-dev <https://lists.swift.org/mailman/listinfo/swift-server-dev>
>
> _______________________________________________
> swift-server-dev mailing list
> swift-server-dev at swift.org <mailto:swift-server-dev at swift.org>
> https://lists.swift.org/mailman/listinfo/swift-server-dev <https://lists.swift.org/mailman/listinfo/swift-server-dev>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-server-dev/attachments/20161129/c19d5de3/attachment.html>
More information about the swift-server-dev
mailing list