All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christophe Leroy <christophe.leroy@c-s.fr>
To: Michael Ellerman <patch-notifications@ellerman.id.au>,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Paul Mackerras <paulus@samba.org>,
	npiggin@gmail.com, hch@infradead.org
Cc: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 1/4] powerpc/fixmap: don't clear fixmap area in paging_init()
Date: Thu, 28 Nov 2019 09:06:52 +0100	[thread overview]
Message-ID: <51c97c23-251d-90d9-0c52-41ef55c9f4c9@c-s.fr> (raw)
In-Reply-To: <bedbd52f-9eea-7253-e28b-68e01a848405@c-s.fr>



Le 27/11/2019 à 15:43, Christophe Leroy a écrit :
> 
> 
> Le 26/11/2019 à 02:13, Michael Ellerman a écrit :
>> On Thu, 2019-09-12 at 13:49:41 UTC, Christophe Leroy wrote:
>>> fixmap is intended to map things permanently like the IMMR region on
>>> FSL SOC (8xx, 83xx, ...), so don't clear it when initialising paging()
>>>
>>> Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
>>
>> Applied to powerpc next, thanks.
>>
>> https://git.kernel.org/powerpc/c/f2bb86937d86ebcb0e52f95b6d19aba1d850e601
>>
> 
> Hi,
> 
> What happened ?
> 
> It looks like it is gone in today's powerpc next.
> 

I see it back today in powerpc/merge, thanks.

The issue is it is crashing with HIGHMEM, indeed exactly the same as you 
saw with "[PATCH] powerpc/8xx: Fix permanently mapped IMMR region.", 
since that was exactly the same code.

I sent a fix for it, and I also rebased "[PATCH] powerpc/8xx: Fix 
permanently mapped IMMR region." into v2 to exclude from it the already 
merged part.

It would be wise if you could merge both.

Thanks
Christophe

  reply	other threads:[~2019-11-28  8:06 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-12 13:49 [PATCH v1 0/4] powerpc: Add support for GENERIC_EARLY_IOREMAP Christophe Leroy
2019-09-12 13:49 ` Christophe Leroy
2019-09-12 13:49 ` [PATCH v1 1/4] powerpc/fixmap: don't clear fixmap area in paging_init() Christophe Leroy
2019-09-12 13:49   ` Christophe Leroy
2019-11-26  1:13   ` Michael Ellerman
2019-11-27 14:43     ` Christophe Leroy
2019-11-28  8:06       ` Christophe Leroy [this message]
2019-09-12 13:49 ` [PATCH v1 2/4] powerpc/fixmap: Use __fix_to_virt() instead of fix_to_virt() Christophe Leroy
2019-09-12 13:49   ` Christophe Leroy
2019-11-25 10:47   ` Michael Ellerman
2019-09-12 13:49 ` [PATCH v1 3/4] powerpc: Add support for GENERIC_EARLY_IOREMAP Christophe Leroy
2019-09-12 13:49   ` Christophe Leroy
2019-09-12 15:37   ` Aneesh Kumar K.V
2019-09-12 15:43     ` Christophe Leroy
     [not found]       ` <874l1hsed6.fsf@linux.ibm.com>
2019-09-12 16:01         ` Christophe Leroy
2019-11-25 10:47   ` Michael Ellerman
2019-09-12 13:49 ` [PATCH v1 4/4] powerpc/ioremap: warn on early use of ioremap() Christophe Leroy
2019-09-12 13:49   ` Christophe Leroy
2019-11-25 10:47   ` Michael Ellerman
2019-09-18 13:40 ` [PATCH v1 0/4] powerpc: Add support for GENERIC_EARLY_IOREMAP Christoph Hellwig
2019-09-18 13:40   ` Christoph Hellwig

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=51c97c23-251d-90d9-0c52-41ef55c9f4c9@c-s.fr \
    --to=christophe.leroy@c-s.fr \
    --cc=benh@kernel.crashing.org \
    --cc=hch@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=npiggin@gmail.com \
    --cc=patch-notifications@ellerman.id.au \
    --cc=paulus@samba.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.