perf: Perf

[driftluo]

in place decrypt on p2p bench

before：

After counting 10 cycles, estimated total time spent on task "10mb_benchmark_with_secio" is 2.6936522s                                 
task name: 10mb_benchmark_with_secio                                                                                                   
cycles: 100                                                                                                                            
total cost: 3.628672348s                                                                                                               
average: 36.286723ms                                                                                                                   
median: 45.029141ms                                                                                                                    
max: 52.898023ms                                                                                                                       
min: 20.852433ms

after：

After counting 10 cycles, estimated total time spent on task "10mb_benchmark_with_secio" is 2.1110216s
task name: 10mb_benchmark_with_secio
cycles: 100
total cost: 2.433076186s
average: 24.330761ms
median: 24.386613ms
max: 55.059342ms
min: 19.604614ms

[quake]

Why add a new fn to trait since the invoker has the mut reference already, can we change the original fn

fn decrypt(&mut self, input: &[u8]) -> Result<Vec<u8>, SecioError>

to

fn decrypt(&mut self, input: &mut [u8]) -> Result<(), SecioError>

[driftluo]

If no new fn is added, OpenSSL will copy one more time, because OpenSSL bind does not have an in-place interface

[driftluo]

At the same time, because &mut [u8] does not have shortened semantics(In other words, it doesn't have ownership, so it cannot be deleted/shortened), the delete tag operation on in-place cannot be performed on this type. Forcibly shortening its length with a pointer will cause a dangling pointer, so the type must be marked with shortening semantics type

[quake]

how about change to BytesMut?

fn decrypt(&mut self, input: &mut BytesMut) -> Result<(), SecioError>