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

From	Wolfram Sang <wsa@the-dreams.de>
Newsgroups	linux.kernel
Subject	Re: zswap: Delete an error message for a failed memory allocation in zswap_dstmem_prepare()
Date	2017-05-21 12:00 +0200
Message-ID	<tJvOy-660-15@gated-at.bofh.it> (permalink)
References	<tJupr-5nU-3@gated-at.bofh.it> <tJupr-5nU-5@gated-at.bofh.it> <tJuIN-5uw-7@gated-at.bofh.it> <tJvES-63O-11@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

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

Thread

[PATCH 3/3] zswap: Delete an error message for a failed memory  allocation in zswap_dstmem_prepare() SF Markus Elfring <elfring@users.sourceforge.net> - 2017-05-21 10:30 +0200
  Re: [PATCH 3/3] zswap: Delete an error message for a failed memory  allocation in zswap_dstmem_prepare() Wolfram Sang <wsa@the-dreams.de> - 2017-05-21 10:50 +0200
    Re: zswap: Delete an error message for a failed memory allocation in  zswap_dstmem_prepare() SF Markus Elfring <elfring@users.sourceforge.net> - 2017-05-21 11:50 +0200
      Re: zswap: Delete an error message for a failed memory allocation in  zswap_dstmem_prepare() Wolfram Sang <wsa@the-dreams.de> - 2017-05-21 12:00 +0200
        Re: Using best practices for big software change possibilities SF Markus Elfring <elfring@users.sourceforge.net> - 2017-05-21 12:20 +0200
          Re: Using best practices for big software change possibilities Wolfram Sang <wsa@the-dreams.de> - 2017-05-21 12:30 +0200
            Re: Using best practices for big software change possibilities SF Markus Elfring <elfring@users.sourceforge.net> - 2017-05-21 12:50 +0200

csiph-web