Messages in this thread | | | From | Alexander Potapenko <> | Date | Mon, 13 Nov 2023 11:45:00 +0100 | Subject | Re: [PATCH v8 3/3] arm64: mte: add compression support to mteswap.c |
| |
> > > > > > void mte_free_tag_storage(char *storage) > > > > If you use a term 'free' here, the counter name should probably > > reflect that. > > I still want to keep the terms "allocation/deallocation" in the > user-facing code (in the /sys/ file) though. > So renaming the counter here will cause a mismatch between its name > and the stats output.
To move forward, I am going to send out v9 that addresses your comments except for this one and for moving the conditionals inside mte_compress() and mte_decompress(). Happy to discuss those three points further.
| |