On Mon, Jul 15, 2019 at 3:04 PM Stephen Boyd sboyd@kernel.org wrote:
Quoting Brendan Higgins (2019-07-15 14:11:50)
On Mon, Jul 15, 2019 at 1:43 PM Stephen Boyd sboyd@kernel.org wrote:
I also wonder if it would be better to just have a big slop buffer of a 4K page or something so that we almost never have to allocate anything with a string_stream and we can just rely on a reader consuming data while writers are writing. That might work out better, but I don't quite understand the use case for the string stream.
That makes sense, but might that also waste memory since we will almost never need that much memory?
Why do we care? These are unit tests.
Agreed.
Having allocations in here makes things more complicated, whereas it would be simpler to have a pointer and a spinlock operating on a chunk of memory that gets flushed out periodically.
I am not so sure. I have to have the logic to allocate memory in some case no matter what (what if I need more memory that my preallocated chuck?). I think it is simpler to always request an allocation than to only sometimes request an allocation.