Subject: [PATCH v5 01/30] regset: Add support for dynamically sized regsets

From: [email-masked] (Alex Bennée)
Date: Wed, 08 Nov 2017 11:50:00 +0000

Dave Martin <[email-masked]> writes:

> On Wed, Nov 01, 2017 at 11:42:29AM +0000, Catalin Marinas wrote:
>> On Tue, Oct 31, 2017 at 03:50:53PM +0000, Dave P Martin wrote:
>> > Currently the regset API doesn't allow for the possibility that
>> > regsets (or at least, the amount of meaningful data in a regset)
>> > may change in size.
>> >
>> > In particular, this results in useless padding being added to
>> > coredumps if a regset's current size is smaller than its
>> > theoretical maximum size.
>> >
>> > This patch adds a get_size() function to struct user_regset.
>> > Individual regset implementations can implement this function to
>> > return the current size of the regset data.  A regset_size()
>> > function is added to provide callers with an abstract interface for
>> > determining the size of a regset without needing to know whether
>> > the regset is dynamically sized or not.
>> >
>> > The only affected user of this interface is the ELF coredump code:
>> > This patch ports ELF coredump to dump regsets with their actual
>> > size in the coredump.  This has no effect except for new regsets
>> > that are dynamically sized and provide a get_size() implementation.
>> >
>> > Signed-off-by: Dave Martin <[email-masked]>
>> > Reviewed-by: Alex Benn?e <[email-masked]>
>> > Reviewed-by: Catalin Marinas <[email-masked]>
>> > Cc: Oleg Nesterov <[email-masked]>
>> > Cc: Alexander Viro <[email-masked]>
>> > Cc: Thomas Gleixner <[email-masked]>
>> > Cc: Ingo Molnar <[email-masked]>
>> > Cc: "H. Peter Anvin" <[email-masked]>
>> > Cc: Dmitry Safonov <[email-masked]>
>> > Cc: H. J. Lu <[email-masked]>
>> >
>> > ---
>> >
>> > **Dropped** Reviewed-by: Alex Benn?e <[email-masked]>
>> > **Dropped** Reviewed-by: Catalin Marinas <[email-masked]>
>> > (due to bug fix applied against v4 of this patch.)
>>
>> It seems that the Reviewed-by tags are still there ;). Anyway, on the
>
> Oops, looks like that only happened on this patch.  I've removed
> Alex's R-b from my own version in case of repost -- can you also
> do the same?
>
>> new patch:
>>
>> Reviewed-by: Catalin Marinas <[email-masked]>

You can have my:

Reviewed-by: Alex Benn?e <[email-masked]>

> Cheers
> ---Dave

--
Alex Benn?e