linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Yosry Ahmed <yosryahmed@google.com>
Cc: kernel test robot <lkp@intel.com>,
	Domenico Cerasuolo <cerasuolodomenico@gmail.com>,
	oe-kbuild-all@lists.linux.dev,
	Linux Memory Management List <linux-mm@kvack.org>,
	Nhat Pham <nphamcs@gmail.com>
Subject: Re: [akpm-mm:mm-unstable 179/192] mm/zswap.c:825:17: error: dereferencing pointer to incomplete type 'struct mem_cgroup'
Date: Wed, 29 Nov 2023 14:53:00 -0800	[thread overview]
Message-ID: <20231129145300.9a178d9bb8518f8bec92b350@linux-foundation.org> (raw)
In-Reply-To: <CAJD7tkZVRPfR6jN3ymuD3Ae2h4tZ3ga6un2ieFKoKrb7YE2JRg@mail.gmail.com>

On Wed, 29 Nov 2023 14:31:12 -0800 Yosry Ahmed <yosryahmed@google.com> wrote:

> > mm-memcg-change-flush_next_time-to-flush_last_time.patch
> > mm-memcg-move-vmstats-structs-definition-above-flushing-code.patch
> > mm-memcg-make-stats-flushing-threshold-per-memcg.patch
> > mm-workingset-move-the-stats-flush-into-workingset_test_recent.patch
> > mm-memcg-restore-subtree-stats-flushing.patch
> >
> > in the hope that a new version of the first series ("workload-specific
> > and memory pressure-driven zswap writeback") can be dropped in place of
> > the old version.
> >
> 
> I see. So Nhat needs to *not* rebase his patches on top of the current
> mm-unstable, right?

Sure.  Normally basing on current -linus works out.

For best accuracy the base could be the preceding patch in mm-unstable
which in this case is "zram: use kmap_local_page()", but that level of
fiddling is rarely needed.

Alternatively, if the changes are small then sending me a shower of
small fixups against latest mm-unstable works.  I'll figure out how to
arrange things so everything lands nicely.



  reply	other threads:[~2023-11-29 22:53 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-29 15:42 kernel test robot
2023-11-29 21:43 ` Andrew Morton
2023-11-29 21:53   ` Andrew Morton
2023-11-29 22:18     ` Yosry Ahmed
2023-11-29 22:29       ` Andrew Morton
2023-11-29 22:31         ` Yosry Ahmed
2023-11-29 22:53           ` Andrew Morton [this message]
2023-11-30  0:02           ` Nhat Pham
2023-11-30  0:13             ` Yosry Ahmed
2023-11-30  0:18               ` Nhat Pham
2023-11-30 19:46                 ` Nhat Pham

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20231129145300.9a178d9bb8518f8bec92b350@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=cerasuolodomenico@gmail.com \
    --cc=linux-mm@kvack.org \
    --cc=lkp@intel.com \
    --cc=nphamcs@gmail.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=yosryahmed@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox