Re: zswap: Delete an error message for a failed memory allocation in zswap_dstmem_prepare()

From: Wolfram Sang
Date: Sun May 21 2017 - 05:57:05 EST


Markus,

> > Markus, can you please stop CCing me on every of those patches?
>
> Yes, of course.

Thanks!

> >> Link: http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf
>
> Did I interpret any information from your presentation slides in an
> inappropriate way?

Have you read my LWN article "Best practices for a big patch series"?

https://lwn.net/Articles/585782/

> > And why do you create a patch for every occasion in the same file?
>
> This can occasionally happen when I am more unsure about the change acceptance
> for a specific place.

Why were you unsure here?

> This can also happen as a side effect if such a source code search pattern
> will point hundreds of places out for further software development considerations.
> How would you prefer to clarify the remaining update candidates there?

Maybe the article mentioned can provice further guidance?

Have a nice sunday,

Wolfram

Attachment: signature.asc
Description: PGP signature