linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Vitaly Bordug <vitb@kernel.crashing.org>
To: Wolfgang Denk <wd@denx.de>
Cc: dzu@denx.de, linuxppc-dev@ozlabs.org,
	Mackerras <paulus@samba.org>,
	Paul@ozlabs.org, Scott Wood <scottwood@freescale.com>
Subject: Re: [PATCH 1/1] [PPC] 8xx swap bug-fix
Date: Fri, 22 Feb 2008 01:57:28 +0300	[thread overview]
Message-ID: <20080222015728.31516db3@kernel.crashing.org> (raw)
In-Reply-To: <20080221122158.11BB324D16@gemini.denx.de>

On Thu, 21 Feb 2008 13:21:58 +0100
Wolfgang Denk wrote:

> Dear Vitaly,
> 
> in message <20080205013857.47ebe544@kernel.crashing.org> you wrote:
> > On Mon, 4 Feb 2008 12:24:21 -0600
> > Scott Wood wrote:
> > 
> > > On Sat, Feb 02, 2008 at 12:22:17PM +0100, Jochen Friedrich wrote:
> > > > Hi Yuri,
> > > > 
> > > > >  Here is the patch which makes Linux-2.6 swap routines operate
> > > > > correctly on the ppc-8xx-based machines.
> > > > 
> > > > is there any 8xx board left which isn't ported to ARCH=powerpc?
> > > 
> > > More importantly, is this something that is also broken in
> > > arch/powerpc?  It looks like it has the same code...
> > > 
> > yes, it is. Though there is currently no best-at-all solution, this
> > looks worth trying. I'll have a look how the 8xx feels under load
> > and report back.
> 
> Have you been able to check if arch/powerpc needs similar fixing?
> 
> 
> Also, what is the current state of this patch? I would like to avoid
> that it get's lost completeley.
> 
I have revalidated it (version modded to fit powerpc) with pretty tight stressing and can conclude
this is good enough for mainline (it gets things much better than they are now, 
in  case of swap enabled).

I'll follow-up with relevant patch shortly.



> 
> BenH commented on Tue, 05 Feb 2008 18:37:07 +1100:
> 
> > Best is to just remove writeback completely and let the generic
> > code handle it.
> 
> Has this been considered / tested by anybody?
> 
> Best regards,
> 
> Wolfgang Denk
> 


-- 
Sincerely, Vitaly

  reply	other threads:[~2008-02-21 22:58 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-02  0:10 [PATCH 3/3] powerpc: mpc83xx_defconfig: enable math emulation and ucc_geth Kim Phillips
2008-02-02  7:47 ` [PATCH 1/1] [PPC] 8xx swap bug-fix Yuri Tikhonov
2008-02-02 11:22   ` Jochen Friedrich
2008-02-02 11:30     ` Yuri Tikhonov
2008-02-04 18:24     ` Scott Wood
2008-02-04 22:38       ` Vitaly Bordug
2008-02-21 12:21         ` Wolfgang Denk
2008-02-21 22:57           ` Vitaly Bordug [this message]
2008-02-05  6:46       ` Yuri Tikhonov
2008-02-05 10:01         ` Jochen Friedrich
2008-02-05  7:37   ` Benjamin Herrenschmidt
2008-03-10 14:39   ` Kumar Gala

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=20080222015728.31516db3@kernel.crashing.org \
    --to=vitb@kernel.crashing.org \
    --cc=Paul@ozlabs.org \
    --cc=dzu@denx.de \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=paulus@samba.org \
    --cc=scottwood@freescale.com \
    --cc=wd@denx.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;
as well as URLs for NNTP newsgroup(s).