DistroKit Mailinglist
 help / color / mirror / Atom feed
From: Lucas Stach <l.stach@pengutronix.de>
To: Michael Olbrich <m.olbrich@pengutronix.de>,
	Robert Schwebel <r.schwebel@pengutronix.de>
Cc: "Marc Kleine-Budde" <mkl@pengutronix.de>,
	distrokit@pengutronix.de, "Uwe Kleine-König" <ukl@pengutronix.de>
Subject: Re: [DistroKit] [PATCH v4] platform-v8a: kernelconfig: cleanup
Date: Mon, 29 Jan 2024 12:53:18 +0100	[thread overview]
Message-ID: <2240a8b0ba51dd4750bafb4317615f9ec4220ee6.camel@pengutronix.de> (raw)
In-Reply-To: <Zbdj7QU-qmUklPyv@pengutronix.de>

Am Montag, dem 29.01.2024 um 09:38 +0100 schrieb Michael Olbrich:
> Hi,
> 
> On Sat, Jan 27, 2024 at 11:58:48AM +0100, Robert Schwebel wrote:
> > On Fri, Jan 26, 2024 at 07:40:34PM +0100, Robert Schwebel wrote:
> > > Thanks, applied to next.
> > 
> > The testsuite found two issues:
> > 
> > - CONFIG_COMPACTION: You switched this off, but 'reason' claims that it
> >   should be on, and the help test for the option does also suggest that
> >   we better enable this. All of our other kernel configs for other
> >   platforms have it enabled.
> 
> Lucas, Marc, I don't remember why we disabled this. Unless you know and
> have a good reason, I'd say enable it.
> 
There is no good reason, we simply didn't look close enough. The
reasoning was that only MIGRATION is required for CMA, but we clearly
failed to consider the Kconfig help text when disabling COMPACTION.

Regards,
Lucas



  parent reply	other threads:[~2024-01-29 11:53 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-26 17:04 Michael Olbrich
2024-01-26 18:40 ` Robert Schwebel
2024-01-27 10:58   ` Robert Schwebel
2024-01-29  8:38     ` Michael Olbrich
2024-01-29 11:38       ` Robert Schwebel
2024-01-29 11:53       ` Lucas Stach [this message]
2024-02-01  8:00         ` Michael Olbrich
2024-02-01  8:02           ` Michael Olbrich

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=2240a8b0ba51dd4750bafb4317615f9ec4220ee6.camel@pengutronix.de \
    --to=l.stach@pengutronix.de \
    --cc=distrokit@pengutronix.de \
    --cc=m.olbrich@pengutronix.de \
    --cc=mkl@pengutronix.de \
    --cc=r.schwebel@pengutronix.de \
    --cc=ukl@pengutronix.de \
    /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