Re: PASEMI: PA6T board doesn't boot with the RC1 of kernel 4.2 anymore

2015-07-13 Thread Denis Kirjanov
On 7/13/15, Benjamin Herrenschmidt b...@kernel.crashing.org wrote:
 On Thu, 2015-07-09 at 12:52 +0300, Denis Kirjanov wrote:
 On 7/9/15, Christian Zigotzky chzigot...@xenosoft.de wrote:
  All
 
  I think you haven't received the SB600 patch yet. I have pasted it in
  this email directly. Thank you for your help. I am sorry because of
  this
  long patch but I hope you could help me a bit.

 But the first thing then is to upstream the sb600 patch...

 Well, the patch is gross ... That stuff needs to be mostly re-written,
 randomly ifdef'ing things around isn't going to fly. There are also
 some mentions of FW issues, can this be fixed in FW ?

A good question. Unfortunately, I don't have the PA6T (

 Ben.

 
  Thanks
 
  Christian
 
  - nemo_4.1-3.patch -
 
  diff -rupN linux-4.1/arch/powerpc/kernel/head_64.S
  linux-4.1-nemo/arch/powerpc/kernel/head_64.S
  --- linux-4.1/arch/powerpc/kernel/head_64.S2015-06-22
  03:47:01.850778538 +0200
  +++ linux-4.1-nemo/arch/powerpc/kernel/head_64.S2015-06-22
  03:27:29.590650112 +0200
  @@ -70,6 +70,13 @@ _GLOBAL(__start)
/* NOP this out unconditionally */
BEGIN_FTR_SECTION
FIXUP_ENDIAN
  +/* Hack for PWRficient platforms: Due to CFE(?) bug, the 64-bit
  + * word at 0x8 needs to be set to 0. Patch it up here once we're
  + * done executing it (we can be lazy and avoid invalidating
  + * icache)
  + */
  +li r0,0
  +std0,8(0)
b__start_initialization_multiplatform
END_FTR_SECTION(0, 1)
 
  diff -rupN linux-4.1/arch/powerpc/kernel/pci-common.c
  linux-4.1-nemo/arch/powerpc/kernel/pci-common.c
  --- linux-4.1/arch/powerpc/kernel/pci-common.c2015-06-22
  03:47:01.866778538 +0200
  +++ linux-4.1-nemo/arch/powerpc/kernel/pci-common.c2015-06-22
  03:27:29.603650164 +0200
  @@ -721,6 +721,19 @@ void pci_process_bridge_OF_ranges(struct
isa_io_base =
(unsigned long)hose-io_base_virt;
#endif /* CONFIG_PPC32 */
  +
  +
  +#ifdef CONFIG_PPC_PASEMI_SB600
  +   /* Workaround for lack of device tree. New for
  kernel 3.17: range.cpu_addr instead of cpu_addr and range.size instead
  of size Ch. Zigotzky */
  +   if (primary) {
  +   __ioremap_at(range.cpu_addr, (void
  *)ISA_IO_BASE,
  +   range.size,
  _PAGE_NO_CACHE|_PAGE_GUARDED);
  +   hose-io_base_virt = (void *)_IO_BASE;
  +   /* _IO_BASE needs unsigned long long for the kernel
  3.17
  Ch. Zigotzky */
  +   printk(Initialised io_base_virt 0x%lx _IO_BASE
  0x%llx\n, (unsigned long)hose-io_base_virt, (unsigned long
  long)_IO_BASE);
  +}
  +#endif
  +
/* pci_io_size and io_base_phys always represent IO
 * space starting at 0 so we factor in pci_addr
 */
  @@ -1194,6 +1207,7 @@ static void pcibios_allocate_bus_resourc
ioport_resource : iomem_resource;
else {
pr = pci_find_parent_resource(bus-self, res);
  +#ifndef CONFIG_PPC_PASEMI_SB600
if (pr == res) {
/* this happens when the generic PCI
 * code (wrongly) decides that this
  @@ -1201,6 +1215,7 @@ static void pcibios_allocate_bus_resourc
 */
continue;
}
  +#endif
}
 
pr_debug(PCI: %s (bus %d) bridge rsrc %d: %016llx-%016llx 
  @@ -1632,8 +1647,9 @@ void pcibios_scan_phb(struct pci_control
pr_debug(PCI: Scanning PHB %s\n, of_node_full_name(node));
 
/* Get some IO space for the new PHB */
  +#ifndef CONFIG_PPC_PASEMI_SB600
pcibios_setup_phb_io_space(hose);
  -
  +#endif
/* Wire up PHB bus resources */
pcibios_setup_phb_resources(hose, resources);
 
  diff -rupN linux-4.1/arch/powerpc/kernel/setup-common.c
  linux-4.1-nemo/arch/powerpc/kernel/setup-common.c
  --- linux-4.1/arch/powerpc/kernel/setup-common.c2015-06-22
  03:47:01.879778538 +0200
  +++ linux-4.1-nemo/arch/powerpc/kernel/setup-common.c2015-06-22
  03:27:29.605650172 +0200
  @@ -76,7 +76,15 @@ EXPORT_SYMBOL(ppc_md);
struct machdep_calls *machine_id;
EXPORT_SYMBOL(machine_id);
 
  +#ifdef CONFIG_PPC_PASEMI_SB600
  +/* FIXME!!
  + * Current PASemi code does not correctly update the value of
  boot_cpuid
  + * As a temporary fix we use the default 0, which is known to work
  + */
  +int boot_cpuid = 0;
  +#else
int boot_cpuid = -1;
  +#endif
EXPORT_SYMBOL_GPL(boot_cpuid);
 
unsigned long klimit = (unsigned long) _end;
  diff -rupN linux-4.1/arch/powerpc/platforms/pasemi/iommu.c
  linux-4.1-nemo/arch/powerpc/platforms/pasemi/iommu.c
  --- linux-4.1/arch/powerpc/platforms/pasemi/iommu.c2015-06-22
  03:47:02.018778538 +0200
  +++ linux-4.1-nemo/arch/powerpc/platforms/pasemi/iommu.c 2015-06-22
  03:27:29.611650196 +0200
  @@ -1,3 +1,9 

Re: PASEMI: PA6T board doesn't boot with the RC1 of kernel 4.2 anymore

2015-07-13 Thread Benjamin Herrenschmidt
On Thu, 2015-07-09 at 12:52 +0300, Denis Kirjanov wrote:
 On 7/9/15, Christian Zigotzky chzigot...@xenosoft.de wrote:
  All
 
  I think you haven't received the SB600 patch yet. I have pasted it in
  this email directly. Thank you for your help. I am sorry because of this
  long patch but I hope you could help me a bit.
 
 But the first thing then is to upstream the sb600 patch...

Well, the patch is gross ... That stuff needs to be mostly re-written,
randomly ifdef'ing things around isn't going to fly. There are also
some mentions of FW issues, can this be fixed in FW ?

Ben.
 
 
  Thanks
 
  Christian
 
  - nemo_4.1-3.patch -
 
  diff -rupN linux-4.1/arch/powerpc/kernel/head_64.S
  linux-4.1-nemo/arch/powerpc/kernel/head_64.S
  --- linux-4.1/arch/powerpc/kernel/head_64.S2015-06-22
  03:47:01.850778538 +0200
  +++ linux-4.1-nemo/arch/powerpc/kernel/head_64.S2015-06-22
  03:27:29.590650112 +0200
  @@ -70,6 +70,13 @@ _GLOBAL(__start)
/* NOP this out unconditionally */
BEGIN_FTR_SECTION
FIXUP_ENDIAN
  +/* Hack for PWRficient platforms: Due to CFE(?) bug, the 64-bit
  + * word at 0x8 needs to be set to 0. Patch it up here once we're
  + * done executing it (we can be lazy and avoid invalidating
  + * icache)
  + */
  +li r0,0
  +std0,8(0)
b__start_initialization_multiplatform
END_FTR_SECTION(0, 1)
 
  diff -rupN linux-4.1/arch/powerpc/kernel/pci-common.c
  linux-4.1-nemo/arch/powerpc/kernel/pci-common.c
  --- linux-4.1/arch/powerpc/kernel/pci-common.c2015-06-22
  03:47:01.866778538 +0200
  +++ linux-4.1-nemo/arch/powerpc/kernel/pci-common.c2015-06-22
  03:27:29.603650164 +0200
  @@ -721,6 +721,19 @@ void pci_process_bridge_OF_ranges(struct
isa_io_base =
(unsigned long)hose-io_base_virt;
#endif /* CONFIG_PPC32 */
  +
  +
  +#ifdef CONFIG_PPC_PASEMI_SB600
  +   /* Workaround for lack of device tree. New for
  kernel 3.17: range.cpu_addr instead of cpu_addr and range.size instead
  of size Ch. Zigotzky */
  +   if (primary) {
  +   __ioremap_at(range.cpu_addr, (void
  *)ISA_IO_BASE,
  +   range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
  +   hose-io_base_virt = (void *)_IO_BASE;
  +   /* _IO_BASE needs unsigned long long for the kernel 3.17
  Ch. Zigotzky */
  +   printk(Initialised io_base_virt 0x%lx _IO_BASE
  0x%llx\n, (unsigned long)hose-io_base_virt, (unsigned long long)_IO_BASE);
  +}
  +#endif
  +
/* pci_io_size and io_base_phys always represent IO
 * space starting at 0 so we factor in pci_addr
 */
  @@ -1194,6 +1207,7 @@ static void pcibios_allocate_bus_resourc
ioport_resource : iomem_resource;
else {
pr = pci_find_parent_resource(bus-self, res);
  +#ifndef CONFIG_PPC_PASEMI_SB600
if (pr == res) {
/* this happens when the generic PCI
 * code (wrongly) decides that this
  @@ -1201,6 +1215,7 @@ static void pcibios_allocate_bus_resourc
 */
continue;
}
  +#endif
}
 
pr_debug(PCI: %s (bus %d) bridge rsrc %d: %016llx-%016llx 
  @@ -1632,8 +1647,9 @@ void pcibios_scan_phb(struct pci_control
pr_debug(PCI: Scanning PHB %s\n, of_node_full_name(node));
 
/* Get some IO space for the new PHB */
  +#ifndef CONFIG_PPC_PASEMI_SB600
pcibios_setup_phb_io_space(hose);
  -
  +#endif
/* Wire up PHB bus resources */
pcibios_setup_phb_resources(hose, resources);
 
  diff -rupN linux-4.1/arch/powerpc/kernel/setup-common.c
  linux-4.1-nemo/arch/powerpc/kernel/setup-common.c
  --- linux-4.1/arch/powerpc/kernel/setup-common.c2015-06-22
  03:47:01.879778538 +0200
  +++ linux-4.1-nemo/arch/powerpc/kernel/setup-common.c2015-06-22
  03:27:29.605650172 +0200
  @@ -76,7 +76,15 @@ EXPORT_SYMBOL(ppc_md);
struct machdep_calls *machine_id;
EXPORT_SYMBOL(machine_id);
 
  +#ifdef CONFIG_PPC_PASEMI_SB600
  +/* FIXME!!
  + * Current PASemi code does not correctly update the value of boot_cpuid
  + * As a temporary fix we use the default 0, which is known to work
  + */
  +int boot_cpuid = 0;
  +#else
int boot_cpuid = -1;
  +#endif
EXPORT_SYMBOL_GPL(boot_cpuid);
 
unsigned long klimit = (unsigned long) _end;
  diff -rupN linux-4.1/arch/powerpc/platforms/pasemi/iommu.c
  linux-4.1-nemo/arch/powerpc/platforms/pasemi/iommu.c
  --- linux-4.1/arch/powerpc/platforms/pasemi/iommu.c2015-06-22
  03:47:02.018778538 +0200
  +++ linux-4.1-nemo/arch/powerpc/platforms/pasemi/iommu.c 2015-06-22
  03:27:29.611650196 +0200
  @@ -1,3 +1,9 @@
  +/* This is a modified copy of
  +* usr/src/linux-2.6.39.4/arch/powerpc/platforms/pasemi/iommu.c
  +* The mod is on line 150
  +* 

Re: PASEMI: PA6T board doesn't boot with the RC1 of kernel 4.2 anymore

2015-07-13 Thread Benjamin Herrenschmidt
On Fri, 2015-07-10 at 00:27 +0200, Christian Zigotzky wrote:
 Hi All,
 
 Many thanks for your help. You're right. It was something wrong with my 
 last bisect. I did another bisect. I evaluated the one sometimes boots 
 with good.

 .../...

Now that makes more sense since the problem is around IRQ handling...
more specifically the SB600 code, so I suspect it's your SB600 patch
that has an issue. Look at how that series from Jiang modified the
irq handlers and see if the sb600 patch needs a similar change maybe ?

Ben.

 
 Output:
 0d0b4c866bcce647f40d73efe5e90aeeb079050a is the first bad commit
 commit 0d0b4c866bcce647f40d73efe5e90aeeb079050a
 Author: Jiang Liu jiang@linux.intel.com
 Date:   Mon Jun 1 16:05:12 2015 +0800
 
  genirq: Introduce struct irq_common_data to host shared irq data
 
  With the introduction of hierarchy irqdomain, struct irq_data becomes
  per-chip instead of per-irq and there may be multiple irq_datas
  associated with the same irq. Some per-irq data stored in struct
  irq_data now may get duplicated into multiple irq_datas, and causes
  inconsistent view.
 
  So introduce struct irq_common_data to host per-irq common data and to
  achieve consistent view among irq_chips.
 
  Signed-off-by: Jiang Liu jiang@linux.intel.com
  Cc: Konrad Rzeszutek Wilk konrad.w...@oracle.com
  Cc: Tony Luck tony.l...@intel.com
  Cc: Bjorn Helgaas bhelg...@google.com
  Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
  Cc: Randy Dunlap rdun...@infradead.org
  Cc: Yinghai Lu ying...@kernel.org
  Cc: Borislav Petkov b...@alien8.de
  Cc: Jason Cooper ja...@lakedaemon.net
  Cc: Kevin Cernekee cerne...@gmail.com
  Cc: Arnd Bergmann a...@arndb.de
  Cc: Marc Zyngier marc.zyng...@arm.com
  Link: 
 http://lkml.kernel.org/r/1433145945-789-4-git-send-email-jiang@linux.intel.com
  Signed-off-by: Thomas Gleixner t...@linutronix.de
 
 :04 04 4cb60e0b1d4beb046834e75c66b5d0a0613935b9 
 df0b465b70ec087560482267cca2d9ff54d92c49 Minclude
 :04 04 3808922cd797449f8f9b33300064a2ff1727f3a6 
 25a75bf65be0ba9e397151de0ed1beb1695fddb2 Mkernel
 
 
 
 Rgds,
 
 Christian
 
 ___
 Linuxppc-dev mailing list
 Linuxppc-dev@lists.ozlabs.org
 https://lists.ozlabs.org/listinfo/linuxppc-dev


___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: PASEMI: PA6T board doesn't boot with the RC1 of kernel 4.2 anymore

2015-07-09 Thread Michael Ellerman
On Thu, 2015-07-09 at 07:19 +0200, Christian Zigotzky wrote:
 Hi All
 
 Many thanks for your answers. You're right. It is something wrong with 
 my bisect. I will do another bisect. I will evaluate the one sometimes 
 boots with good.

Thanks Christian.

cheers


___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: PASEMI: PA6T board doesn't boot with the RC1 of kernel 4.2 anymore

2015-07-09 Thread Benjamin Herrenschmidt
On Thu, 2015-07-09 at 11:42 +1000, Michael Ellerman wrote:
  
  There is no way that commit affects anything on that platform, it
 only
  changes a file in arch/x86 that isn't compiled on a powerpc build.
 You
  must have made a mistake in your bisection, possibly the one
 sometimes
  boots should be considered good, but I can't say for sure.
  
  Michael has a PA6T board at work, so I assume he will see if he can
  reproduce.
 
 Yeah I can't reproduce here.
 
 I think you need to bisect again and look for the particular crash you
 reported. If you see that crash then mark it bad, else mark it good.

Michael, the crash looks like a neat NULL dereference, any chance you
can spot something in the code that might explain it ?

The strange thing is that the crash is in sb600_8259_cascade(), however
that function doesn't seem to exist in a current kernel tree, or if it
does it's somewhat hidden behind grep-defeating macros.

All I can find that relates to something called sb600 is some quirks
in AHCI and x86 fixups that seem ATI related.

Christian, do you have some out of tree driver in that kernel ?

Ben.
 

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: PASEMI: PA6T board doesn't boot with the RC1 of kernel 4.2 anymore

2015-07-09 Thread Denis Kirjanov
On 7/9/15, Christian Zigotzky chzigot...@xenosoft.de wrote:
 All

 I think you haven't received the SB600 patch yet. I have pasted it in
 this email directly. Thank you for your help. I am sorry because of this
 long patch but I hope you could help me a bit.

But the first thing then is to upstream the sb600 patch...

 Thanks

 Christian

 - nemo_4.1-3.patch -

 diff -rupN linux-4.1/arch/powerpc/kernel/head_64.S
 linux-4.1-nemo/arch/powerpc/kernel/head_64.S
 --- linux-4.1/arch/powerpc/kernel/head_64.S2015-06-22
 03:47:01.850778538 +0200
 +++ linux-4.1-nemo/arch/powerpc/kernel/head_64.S2015-06-22
 03:27:29.590650112 +0200
 @@ -70,6 +70,13 @@ _GLOBAL(__start)
   /* NOP this out unconditionally */
   BEGIN_FTR_SECTION
   FIXUP_ENDIAN
 +/* Hack for PWRficient platforms: Due to CFE(?) bug, the 64-bit
 + * word at 0x8 needs to be set to 0. Patch it up here once we're
 + * done executing it (we can be lazy and avoid invalidating
 + * icache)
 + */
 +li r0,0
 +std0,8(0)
   b__start_initialization_multiplatform
   END_FTR_SECTION(0, 1)

 diff -rupN linux-4.1/arch/powerpc/kernel/pci-common.c
 linux-4.1-nemo/arch/powerpc/kernel/pci-common.c
 --- linux-4.1/arch/powerpc/kernel/pci-common.c2015-06-22
 03:47:01.866778538 +0200
 +++ linux-4.1-nemo/arch/powerpc/kernel/pci-common.c2015-06-22
 03:27:29.603650164 +0200
 @@ -721,6 +721,19 @@ void pci_process_bridge_OF_ranges(struct
   isa_io_base =
   (unsigned long)hose-io_base_virt;
   #endif /* CONFIG_PPC32 */
 +
 +
 +#ifdef CONFIG_PPC_PASEMI_SB600
 +   /* Workaround for lack of device tree. New for
 kernel 3.17: range.cpu_addr instead of cpu_addr and range.size instead
 of size Ch. Zigotzky */
 +   if (primary) {
 +   __ioremap_at(range.cpu_addr, (void
 *)ISA_IO_BASE,
 +   range.size, _PAGE_NO_CACHE|_PAGE_GUARDED);
 +   hose-io_base_virt = (void *)_IO_BASE;
 +   /* _IO_BASE needs unsigned long long for the kernel 3.17
 Ch. Zigotzky */
 +   printk(Initialised io_base_virt 0x%lx _IO_BASE
 0x%llx\n, (unsigned long)hose-io_base_virt, (unsigned long long)_IO_BASE);
 +}
 +#endif
 +
   /* pci_io_size and io_base_phys always represent IO
* space starting at 0 so we factor in pci_addr
*/
 @@ -1194,6 +1207,7 @@ static void pcibios_allocate_bus_resourc
   ioport_resource : iomem_resource;
   else {
   pr = pci_find_parent_resource(bus-self, res);
 +#ifndef CONFIG_PPC_PASEMI_SB600
   if (pr == res) {
   /* this happens when the generic PCI
* code (wrongly) decides that this
 @@ -1201,6 +1215,7 @@ static void pcibios_allocate_bus_resourc
*/
   continue;
   }
 +#endif
   }

   pr_debug(PCI: %s (bus %d) bridge rsrc %d: %016llx-%016llx 
 @@ -1632,8 +1647,9 @@ void pcibios_scan_phb(struct pci_control
   pr_debug(PCI: Scanning PHB %s\n, of_node_full_name(node));

   /* Get some IO space for the new PHB */
 +#ifndef CONFIG_PPC_PASEMI_SB600
   pcibios_setup_phb_io_space(hose);
 -
 +#endif
   /* Wire up PHB bus resources */
   pcibios_setup_phb_resources(hose, resources);

 diff -rupN linux-4.1/arch/powerpc/kernel/setup-common.c
 linux-4.1-nemo/arch/powerpc/kernel/setup-common.c
 --- linux-4.1/arch/powerpc/kernel/setup-common.c2015-06-22
 03:47:01.879778538 +0200
 +++ linux-4.1-nemo/arch/powerpc/kernel/setup-common.c2015-06-22
 03:27:29.605650172 +0200
 @@ -76,7 +76,15 @@ EXPORT_SYMBOL(ppc_md);
   struct machdep_calls *machine_id;
   EXPORT_SYMBOL(machine_id);

 +#ifdef CONFIG_PPC_PASEMI_SB600
 +/* FIXME!!
 + * Current PASemi code does not correctly update the value of boot_cpuid
 + * As a temporary fix we use the default 0, which is known to work
 + */
 +int boot_cpuid = 0;
 +#else
   int boot_cpuid = -1;
 +#endif
   EXPORT_SYMBOL_GPL(boot_cpuid);

   unsigned long klimit = (unsigned long) _end;
 diff -rupN linux-4.1/arch/powerpc/platforms/pasemi/iommu.c
 linux-4.1-nemo/arch/powerpc/platforms/pasemi/iommu.c
 --- linux-4.1/arch/powerpc/platforms/pasemi/iommu.c2015-06-22
 03:47:02.018778538 +0200
 +++ linux-4.1-nemo/arch/powerpc/platforms/pasemi/iommu.c 2015-06-22
 03:27:29.611650196 +0200
 @@ -1,3 +1,9 @@
 +/* This is a modified copy of
 +* usr/src/linux-2.6.39.4/arch/powerpc/platforms/pasemi/iommu.c
 +* The mod is on line 150
 +* By Len Karpowicz twota...@embarqmail.com
 +*/
 +
   /*
* Copyright (C) 2005-2008, PA Semi, Inc
*
 @@ -143,8 +149,11 @@ static void iommu_table_iobmap_setup(voi
   iommu_table_iobmap.it_page_shift = IOBMAP_PAGE_SHIFT;

   /* it_size is in number of entries */
 -iommu_table_iobmap.it_size =
 -0x8000  iommu_table_iobmap.it_page_shift;
 +
 +/* Note: 

Re: PASEMI: PA6T board doesn't boot with the RC1 of kernel 4.2 anymore

2015-07-08 Thread Benjamin Herrenschmidt
On Wed, 2015-07-08 at 20:00 +0200, Christian Zigotzky wrote:
 Dear Linuxppc-dev mailing list,
 
 I used git bisect and found out that the following commit is the problem.
 
 commit 3ceaccdf92073d193f0bfbe24280dd736e3fed86
 Author: Dave Hansen dave.han...@linux.intel.com

There is no way that commit affects anything on that platform, it only
changes a file in arch/x86 that isn't compiled on a powerpc build. You
must have made a mistake in your bisection, possibly the one sometimes
boots should be considered good, but I can't say for sure.

Michael has a PA6T board at work, so I assume he will see if he can
reproduce.

Ben.

 
 Log:
 
 git clone 
 git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git linux-git
 
 git bisect start b953c0d234bc72e8489d3bf51a276c5c4ec85345 (Linux 4.1)
 
 git bisect bad d770e558e21961ad6cfdf0ff7df0eb5d7d4f0754 (Linux 4.2-rc1)
 
 Output:
 
 Bisecting: 6261 revisions left to test after this (roughly 13 steps)
 [4570a37169d4b44d316f40b2ccc681dc93fedc7b] Merge tag 'sound-4.2-rc1' of 
 git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound
 
 
 
 git bisect bad
 
 Output:
 
 Bisecting: 3295 revisions left to test after this (roughly 12 steps)
 [4e241557fc1cb560bd9e77ca1b4a9352732a5427] Merge tag 'for-linus' of 
 git://git.kernel.org/pub/scm/virt/kvm/kvm
 
 
 
 git bisect bad
 
 Output:
 
 Bisecting: 1625 revisions left to test after this (roughly 11 steps)
 [44d21c3f3a2ef2f58b18bda64c52c99e723f3f4a] Merge 
 git://git.kernel.org/pub/scm/linux/kernel/git/herbert/crypto-2.6
 
 
 
 git bisect bad
 
 Output:
 
 Bisecting: 712 revisions left to test after this (roughly 10 steps)
 [e75c73ad64478c12b3a44b86a3e7f62a4f65b93e] Merge branch 
 'x86-fpu-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
 
 
 
 git bisect bad (sometimes the kernel boots but the mouse doesn't work)
 
 Output:
 
 Bisecting: 371 revisions left to test after this (roughly 9 steps)
 [c58267e9fa7b0345dd9006939254701e3622ca6a] Merge branch 
 'perf-core-for-linus' of 
 git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
 
 
 
 git bisect good
 
 Output:
 
 Bisecting: 185 revisions left to test after this (roughly 8 steps)
 [59a36d16be8f9f68410f1bd396577fb7f31ae877] x86/fpu: Factor out 
 fpu/regset.h from fpu/internal.h
 
 
 
 git bisect good
 
 Output:
 
 Bisecting: 93 revisions left to test after this (roughly 7 steps)
 [23b7776290b10297fe2cae0fb5f166a4f2c68121] Merge branch 
 'sched-core-for-linus' of 
 git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
 
 
 
 git bisect good
 
 Output:
 
 Bisecting: 46 revisions left to test after this (roughly 6 steps)
 [0c4109bec0a6cde471bef3a21cd6f8384a614469] x86/fpu/xstate: Fix up bad 
 get_xsave_addr() assumptions
 
 
 
 git bisect good
 
 Output:
 
 Bisecting: 19 revisions left to test after this (roughly 5 steps)
 [cfe3eceb7a2eb91284d5605c5315249bb165e9d3] Merge branch 
 'x86-efi-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
 
 
 
 git bisect good
 
 Output:
 
 Bisecting: 9 revisions left to test after this (roughly 3 steps)
 [cd4996dce18b619bd7b3acf75c91f49c77f05a97] x86/mpx: Trace allocation of 
 new bounds tables
 
 
 
 git bisect good
 
 Output:
 
 Bisecting: 4 revisions left to test after this (roughly 2 steps)
 [613fcb7d3c79ec25b5913a6aa974c9047c31e68c] x86/mpx: Support 32-bit 
 binaries on 64-bit kernels
 
 
 
 git bisect good
 
 Output:
 
 Bisecting: 2 revisions left to test after this (roughly 1 step)
 [bea03c50b871a2fa922f31ad7c9993bb4fc7b192] x86/mpx: Do not count MPX 
 VMAs as neighbors when unmapping
 
 
 
 git bisect bad (sometimes the kernel boots but the mouse doesn't work)
 
 Output:
 
 Bisecting: 0 revisions left to test after this (roughly 0 steps)
 [3ceaccdf92073d193f0bfbe24280dd736e3fed86] x86/mpx: Rewrite the unmap code
 
 
 
 git bisect bad (sometimes the kernel boots but the mouse doesn't work)
 
 Output:
 
 3ceaccdf92073d193f0bfbe24280dd736e3fed86 is the first bad commit
 commit 3ceaccdf92073d193f0bfbe24280dd736e3fed86
 Author: Dave Hansen dave.han...@linux.intel.com
 Date:   Sun Jun 7 11:37:06 2015 -0700
 
  x86/mpx: Rewrite the unmap code
 
  The MPX code needs to clear out bounds tables for memory which
  is no longer in use.  We do this when a userspace mapping is
  torn down (unmapped).
 
  There are two modes:
 
1. An entire bounds table becomes unused, and can be freed
   and its pointer removed from the bounds directory.  This
   happens either when a large mapping is torn down, or when
   a small mapping is torn down and it is the last mapping
   covered by a bounds table.
 
2. Only part of a bounds table becomes unused, in which case
   we free the backing memory as if MADV_DONTNEED was called.
 
  The old code was a spaghetti mess of edge bounds tables
  where the edges were handled specially, even if we were
  unmapping an entire one.  Non-edge 

Re: PASEMI: PA6T board doesn't boot with the RC1 of kernel 4.2 anymore

2015-07-08 Thread Michael Ellerman
On Thu, 2015-07-09 at 08:36 +1000, Benjamin Herrenschmidt wrote:
 On Wed, 2015-07-08 at 20:00 +0200, Christian Zigotzky wrote:
  Dear Linuxppc-dev mailing list,
  
  I used git bisect and found out that the following commit is the problem.
  
  commit 3ceaccdf92073d193f0bfbe24280dd736e3fed86
  Author: Dave Hansen dave.han...@linux.intel.com
 
 There is no way that commit affects anything on that platform, it only
 changes a file in arch/x86 that isn't compiled on a powerpc build. You
 must have made a mistake in your bisection, possibly the one sometimes
 boots should be considered good, but I can't say for sure.
 
 Michael has a PA6T board at work, so I assume he will see if he can
 reproduce.

Yeah I can't reproduce here.

I think you need to bisect again and look for the particular crash you
reported. If you see that crash then mark it bad, else mark it good.

cheers


___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev