On Friday 27 November 2015 16:42:54 Aya Mahfouz wrote:
On Wed, Nov 18, 2015 at 04:35:58PM +0100, Arnd Bergmann wrote:
On Wednesday 18 November 2015 17:33:38 Aya Mahfouz wrote:
Changelog: v1: The changes were originally made by Arnd Bergmann in relation to time_t. I've broken down a patch sent to me into two independent patches.
I suppose you have tried and failed to split it up into smaller chunks?
I'm playing it safe since there are uses of key->expiry that are dependent on key_preparsed_payload->expiry. If you would like for me to break it down further, I can try.
I think that would be helpful. The current version is a little too big to review properly in one chunk.
I've tried several time to break it up into 2 or 3 patches without ignoring dependencies but I fail every time to come with a set of independent patches. Can I keep it as is? I'm open to any division suggestions.
My first idea would be to split the key_preparsed_payload changes from the struct key changes. Have you tried that? What problems did you run into?
Arnd