From: Christian Zigotzky <chzigotzky@xenosoft.de>
To: Michael Ellerman <mpe@ellerman.id.au>,
"Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>,
linuxppc-dev@lists.ozlabs.org, darren@stevens-zone.net,
Michael Ellerman <michaele@au1.ibm.com>,
Julian Margetson <runaway@candw.ms>,
Adrian Cox <adrian@humboldt.co.uk>,
"R.T.Dickinson" <rtd@a-eon.com>,
"R.T.Dickinson" <rtd2@xtra.co.nz>, Pat Wall <pjwall@mac.com>,
Pat Wall <pjwall@me.com>, "contact@a-eon.com" <contact@a-eon.com>,
Matthew Leaman <matthew@a-eon.biz>,
luigi burdo <intermediadc@hotmail.com>
Subject: Kernel 4.7: PAGE_GUARDED and _PAGE_NO_CACHE
Date: Tue, 7 Jun 2016 22:17:08 +0200 [thread overview]
Message-ID: <50194690-18ac-edfd-42cb-8f1cc96055c9@xenosoft.de> (raw)
In-Reply-To: <1465174304.12265.0.camel@ellerman.id.au>
Hi Michael,
On 06 June 2016 at 02:51 AM, Michael Ellerman wrote:
> On Sat, 2016-06-04 at 17:07 +0200, Christian Zigotzky wrote:
>
>> Aneesh,
>>
>> Shall I bisect the kernel from the powerpc git?
> No just use linus' tree.
>
>> Shall I start with the following commit?
>>
>> https://git.kernel.org/cgit/linux/kernel/git/powerpc/linux.git/commit/?id=8ffb4103f5e28d7e7890ed4774d8e009f253f56e
> Yeah that would be a good one to start with.
>
> Then mark rc1 as bad and bisect should do the rest.
>
> cheers
>
>
"range.size, pgprot_val(pgprot_noncached(__pgprot(0))));" isn't the
problem. :-) It works.
764041e0f43cc7846f6d8eb246d65b53cc06c764 is the first bad commit
commit 764041e0f43cc7846f6d8eb246d65b53cc06c764
Author: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Date: Fri Apr 29 23:26:09 2016 +1000
powerpc/mm/radix: Add checks in slice code to catch radix usage
Radix doesn't need slice support. Catch incorrect usage of slice code
when radix is enabled.
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Cheers,
Christian
Git Log:
git clone
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git linux-git
git bisect start
git bisect good 8ffb4103f5e28d7e7890ed4774d8e009f253f56e
git bisect bad 1a695a905c18548062509178b98bc91e67510864 (Linux 4.7-rc1)
Output:
Bisecting: 6333 revisions left to test after this (roughly 13 steps)
[4741526b83c5d3a3d661d1896f9e7414c5730bcb] mm, page_alloc: restore the
original nodemask if the fast path allocation failed
----
git bisect good (Linux AmigaoneX1000
4.6.0_A-EON_AmigaONE_X1000_Nemo-05145-g4741526-dirty #1 SMP Mon Jun 6
14:35:01 CEST 2016 ppc64 GNU/Linux)
Output:
Bisecting: 3014 revisions left to test after this (roughly 12 steps)
[2f37dd131c5d3a2eac21cd5baf80658b1b02a8ac] Merge tag 'staging-4.7-rc1'
of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging
----
I had to replace
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
with
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size,
pgprot_val(pgprot_noncached(__pgprot(0))));
in the file "pci-common.c". After that it compiled but it doesn't boot
so "git bisect bad"
Output:
Bisecting: 1693 revisions left to test after this (roughly 11 steps)
[54cf809b9512be95f53ed4a5e3b631d1ac42f0fa] locking,qspinlock: Fix
spin_is_locked() and spin_unlock_wait()
----
I had to replace
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
with
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size,
pgprot_val(pgprot_noncached(__pgprot(0))));
in the file "pci-common.c". After that it compiled but it doesn't boot
so "git bisect bad"
Output:
Bisecting: 721 revisions left to test after this (roughly 10 steps)
[f4c80d5a16eb4b08a0d9ade154af1ebdc63f5752] Merge tag 'sound-4.7-rc1' of
git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound
----
git bisect good (Linux AmigaoneX1000
4.6.0_A-EON_AmigaONE_X1000_Nemo-05931-gf4c80d5-dirty #1 SMP Mon Jun 6
19:13:42 CEST 2016 ppc64 GNU/Linux)
Output:
Bisecting: 324 revisions left to test after this (roughly 9 steps)
[c04a5880299eab3da8c10547db96ea9cdffd44a6] Merge tag 'powerpc-4.7-1' of
git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux
----
I had to replace
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
with
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size,
pgprot_val(pgprot_noncached(__pgprot(0))));
in the file "pci-common.c". After that it compiled but it doesn't boot
so "git bisect bad"
Output:
Bisecting: 186 revisions left to test after this (roughly 8 steps)
[e9ad9b9bd3a3b95c89a29b2a197476e662db4233] Merge tag 'docs-for-linus' of
git://git.lwn.net/linux
----
git bisect good (Linux AmigaoneX1000
4.6.0_A-EON_AmigaONE_X1000_Nemo-06141-ge9ad9b9-dirty #1 SMP Tue Jun 7
08:32:33 CEST 2016 ppc64 GNU/Linux)
Bisecting: 93 revisions left to test after this (roughly 7 steps)
[0e5b5ba17ac33a05d9f4a48b5eb8b5e30f2274d7] cxl: Remove duplicate #defines
----
I had to replace
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
with
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size,
pgprot_val(pgprot_noncached(__pgprot(0))));
in the file "pci-common.c". After that it compiled but it doesn't boot
so "git bisect bad"
Output:
Bisecting: 46 revisions left to test after this (roughly 6 steps)
[764041e0f43cc7846f6d8eb246d65b53cc06c764] powerpc/mm/radix: Add checks
in slice code to catch radix usage
----
It doesn't boot. Booting Linux via __start()... didn't appear in the CFE.
git bisect bad
Output:
Bisecting: 22 revisions left to test after this (roughly 5 steps)
[2e8735198af0392d9032a356ee6c82686afca779] powerpc/mm: Move common pte
bits and accessors to book3s/64/pgtable.h
----
I had to replace
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
with
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size,
pgprot_val(pgprot_noncached(__pgprot(0))));
in the file "pci-common.c". After that it compiled and booted with
"range.size, pgprot_val(pgprot_noncached(__pgprot(0))));". That means,
"range.size, pgprot_val(pgprot_noncached(__pgprot(0))));" isn't the
problem. :-)
git bisect good (Linux AmigaoneX1000
4.6.0-rc3_A-EON_AmigaONE_X1000_Nemo-00067-g2e87351-dirty #1 SMP Tue Jun
7 15:29:19 CEST 2016 ppc64 GNU/Linux)
Output:
Bisecting: 11 revisions left to test after this (roughly 4 steps)
[2bfd65e45e877fb5704730244da67c748d28a1b8] powerpc/mm/radix: Add radix
callbacks for early init routines
----
I had to replace
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
with
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size,
pgprot_val(pgprot_noncached(__pgprot(0))));
in the file "pci-common.c". After that it compiled and it also booted
with "range.size, pgprot_val(pgprot_noncached(__pgprot(0))));"!
git bisect good (Linux AmigaoneX1000
4.6.0-rc3_A-EON_AmigaONE_X1000_Nemo-00078-g2bfd65e-dirty #1 SMP Tue Jun
7 16:50:47 CEST 2016 ppc64 GNU/Linux)
Output:
Bisecting: 5 revisions left to test after this (roughly 3 steps)
[676012a66f651a98808459bc8ab75661828ed96f] powerpc/mm: Hash abstraction
for tlbflush routines
----
I had to replace
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
with
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size,
pgprot_val(pgprot_noncached(__pgprot(0))));
in the file "pci-common.c". After that it compiled and it also booted
with "range.size, pgprot_val(pgprot_noncached(__pgprot(0))));"!
git bisect good (Linux AmigaoneX1000
4.6.0-rc3_A-EON_AmigaONE_X1000_Nemo-00084-g676012a-dirty #1 SMP Tue Jun
7 19:03:59 CEST 2016 ppc64 GNU/Linux)
Output:
Bisecting: 2 revisions left to test after this (roughly 2 steps)
[a8ed87c92adf1fd45142323e04f15b522117d575] powerpc/mm/radix: Add
MMU_FTR_RADIX
----
I had to replace
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
with
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size,
pgprot_val(pgprot_noncached(__pgprot(0))));
in the file "pci-common.c". After that it compiled and it also booted
with "range.size, pgprot_val(pgprot_noncached(__pgprot(0))));"!
git bisect good (Linux AmigaoneX1000
4.6.0-rc3_A-EON_AmigaONE_X1000_Nemo-00087-ga8ed87c-dirty #1 SMP Tue Jun
7 20:25:11 CEST 2016 ppc64 GNU/Linux)
Output:
Bisecting: 0 revisions left to test after this (roughly 1 step)
[d8c476eeb697c9aac46b390e9cbd7af7032814f1] powerpc/mm/radix: Isolate
hash table function from pseries guest code
----
I had to replace
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
with
__ioremap_at(range.cpu_addr, (void *)ISA_IO_BASE,
range.size,
pgprot_val(pgprot_noncached(__pgprot(0))));
in the file "pci-common.c". After that it compiled and it also booted
with "range.size, pgprot_val(pgprot_noncached(__pgprot(0))));"!
git bisect good (Linux AmigaoneX1000
4.6.0-rc3_A-EON_AmigaONE_X1000_Nemo-00089-gd8c476e-dirty #1 SMP Tue Jun
7 21:53:24 CEST 2016 ppc64 GNU/Linux)
Output:
764041e0f43cc7846f6d8eb246d65b53cc06c764 is the first bad commit
commit 764041e0f43cc7846f6d8eb246d65b53cc06c764
Author: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Date: Fri Apr 29 23:26:09 2016 +1000
powerpc/mm/radix: Add checks in slice code to catch radix usage
Radix doesn't need slice support. Catch incorrect usage of slice code
when radix is enabled.
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
:040000 040000 e66f81f989a42a4de238a49a2191f787ae65d4a2
6b7b4380d14db00d9aae28b5990e5dd8d6cf23a0 M arch
next prev parent reply other threads:[~2016-06-07 20:23 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-23 4:38 PAGE_GUARDED Christian Zigotzky
2016-05-23 8:01 ` PAGE_GUARDED Benjamin Herrenschmidt
2016-05-24 10:48 ` PAGE_GUARDED Christian Zigotzky
2016-05-24 11:16 ` PAGE_GUARDED Benjamin Herrenschmidt
2016-05-24 13:37 ` PAGE_GUARDED Christian Zigotzky
2016-05-24 15:37 ` PAGE_GUARDED Aneesh Kumar K.V
2016-05-25 6:58 ` PAGE_GUARDED Christian Zigotzky
2016-05-25 11:44 ` PAGE_GUARDED Christian Zigotzky
2016-06-04 7:19 ` Kernel 4.7: PAGE_GUARDED and _PAGE_NO_CACHE Christian Zigotzky
2016-06-04 7:40 ` Christian Zigotzky
2016-06-04 11:08 ` Christian Zigotzky
2016-06-04 14:46 ` Aneesh Kumar K.V
2016-06-04 15:13 ` Christian Zigotzky
2016-06-04 15:25 ` Christian Zigotzky
2016-06-05 16:09 ` Christian Zigotzky
2016-06-06 0:57 ` Michael Ellerman
2016-06-06 1:23 ` Julian Margetson
2016-06-06 2:00 ` Michael Ellerman
2016-06-06 8:59 ` Julian Margetson
2016-06-06 0:54 ` Michael Ellerman
2016-06-04 14:42 ` Aneesh Kumar K.V
2016-06-04 15:07 ` Christian Zigotzky
2016-06-06 0:51 ` Michael Ellerman
2016-06-06 5:21 ` Christian Zigotzky
2016-06-07 20:17 ` Christian Zigotzky [this message]
2016-06-07 22:14 ` Christian Zigotzky
2016-06-08 1:51 ` Michael Ellerman
2016-06-08 2:52 ` Michael Ellerman
2016-06-08 10:58 ` Christian Zigotzky
2016-06-08 11:30 ` Michael Ellerman
2016-06-08 12:50 ` Christian Zigotzky
2016-06-08 11:33 ` Darren Stevens
2016-06-08 13:24 ` Christian Zigotzky
2016-06-08 13:51 ` Christian Zigotzky
2016-06-08 13:52 ` Michael Ellerman
2016-06-08 15:11 ` Aneesh Kumar K.V
2016-06-08 15:47 ` Christian Zigotzky
2016-06-09 8:48 ` Christian Zigotzky
2016-06-09 11:15 ` Michael Ellerman
2016-06-09 13:13 ` Christian Zigotzky
2016-06-13 8:14 ` Christian Zigotzky
2016-06-13 8:19 ` Benjamin Herrenschmidt
2016-06-13 18:09 ` Christian Zigotzky
2016-06-22 19:07 ` Christian Zigotzky
2016-06-22 19:58 ` luigi burdo
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=50194690-18ac-edfd-42cb-8f1cc96055c9@xenosoft.de \
--to=chzigotzky@xenosoft.de \
--cc=adrian@humboldt.co.uk \
--cc=aneesh.kumar@linux.vnet.ibm.com \
--cc=benh@kernel.crashing.org \
--cc=contact@a-eon.com \
--cc=darren@stevens-zone.net \
--cc=intermediadc@hotmail.com \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=matthew@a-eon.biz \
--cc=michaele@au1.ibm.com \
--cc=mpe@ellerman.id.au \
--cc=pjwall@mac.com \
--cc=pjwall@me.com \
--cc=rtd2@xtra.co.nz \
--cc=rtd@a-eon.com \
--cc=runaway@candw.ms \
/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.