RE: [PATCH V9] powerpc/fsl-pci: Unify pci/pcie initialization code

2012-08-23 Thread Jia Hongtao-B38951
> -Original Message- > From: Wood Scott-B07421 > Sent: Friday, August 24, 2012 8:16 AM > To: Jia Hongtao-B38951 > Cc: linuxppc-dev@lists.ozlabs.org; ga...@kernel.crashing.org; Li Yang- > R58472; Wood Scott-B07421 > Subject: Re: [PATCH V9] powerpc/fsl-pci: Unify pci/pcie initialization > c

Re: [PATCH v4 2/2] powerpc: Uprobes port to powerpc

2012-08-23 Thread Michael Ellerman
On Thu, 2012-08-23 at 11:02 +0530, Srikar Dronamraju wrote: > > > > These seem to be duplicated in kprobes.h, can we consolidate them. > > > > > +struct arch_uprobe { > > > + u8 insn[MAX_UINSN_BYTES]; > > > +}; > > > > Why not uprobe_opcode_t insn ? > > > > insn is updated/accessed in the

Re: [PATCH v4 2/2] powerpc: Uprobes port to powerpc

2012-08-23 Thread Michael Ellerman
On Thu, 2012-08-23 at 11:28 +0530, Ananth N Mavinakayanahalli wrote: > On Thu, Aug 23, 2012 at 02:28:20PM +1000, Michael Ellerman wrote: > > On Wed, 2012-08-22 at 13:57 +0530, Ananth N Mavinakayanahalli wrote: > > > From: Ananth N Mavinakayanahalli > > > > > > This is the port of uprobes to power

Re: [PATCH V9] powerpc/fsl-pci: Unify pci/pcie initialization code

2012-08-23 Thread Scott Wood
On 08/23/2012 04:31 AM, Jia Hongtao wrote: > +static void mpc85xx_cds_pci_assign_primary(void) > +{ > +#ifdef CONFIG_PCI > + struct device_node *np; > + > + if (fsl_pci_primary) > + return; > + > + /* > + * MPC85xx_CDS has ISA bridge but unfortunately there is no > +

Re: [PATCH v4 2/2] powerpc: Uprobes port to powerpc

2012-08-23 Thread Benjamin Herrenschmidt
On Thu, 2012-08-23 at 21:47 +0530, Srikar Dronamraju wrote: > * Benjamin Herrenschmidt [2012-08-23 20:06:18]: > > > On Thu, 2012-08-23 at 11:02 +0530, Srikar Dronamraju wrote: > > > > > > > > > > insn is updated/accessed in the arch independent code. Size of > > > uprobe_opcode_t could be diffe

Re: Build regressions/improvements in v3.6-rc3

2012-08-23 Thread Geert Uytterhoeven
On Thu, Aug 23, 2012 at 10:46 PM, Geert Uytterhoeven wrote: > JFYI, when comparing v3.6-rc3 to v3.6-rc2[3], the summaries are: > - build errors: +13/-9 + arch/x86/include/asm/uaccess_32.h: error: call to 'copy_from_user_overflow' declared with attribute error: copy_from_user() buffer size is

Re: [PATCH 1/2] PCI: leave MEM and IO decoding disabled during 64-bit BAR sizing, too

2012-08-23 Thread Olof Johansson
Hi, On Thu, Aug 23, 2012 at 11:02 AM, Bjorn Helgaas wrote: > On Thu, Aug 23, 2012 at 12:28:23AM -0700, Olof Johansson wrote: >> Hi, >> >> On Mon, Jul 9, 2012 at 11:20 AM, Bjorn Helgaas wrote: >> > After 253d2e5498, we disable MEM and IO decoding for most devices while we >> > size 32-bit BARs.

Re: [PATCH 1/2] PCI: leave MEM and IO decoding disabled during 64-bit BAR sizing, too

2012-08-23 Thread Bjorn Helgaas
On Thu, Aug 23, 2012 at 12:28:23AM -0700, Olof Johansson wrote: > Hi, > > On Mon, Jul 9, 2012 at 11:20 AM, Bjorn Helgaas wrote: > > After 253d2e5498, we disable MEM and IO decoding for most devices while we > > size 32-bit BARs. However, we restore the original COMMAND register before > > we siz

Re: [PATCH v4 2/2] powerpc: Uprobes port to powerpc

2012-08-23 Thread Srikar Dronamraju
* Benjamin Herrenschmidt [2012-08-23 20:06:18]: > On Thu, 2012-08-23 at 11:02 +0530, Srikar Dronamraju wrote: > > > > > > > insn is updated/accessed in the arch independent code. Size of > > uprobe_opcode_t could be different for different archs. > > uprobe_opcode_t > > represents the size of t

Re: [PATCH v4 2/2] powerpc: Uprobes port to powerpc

2012-08-23 Thread Srikar Dronamraju
* Oleg Nesterov [2012-08-23 11:02:09]: > On 08/23, Benjamin Herrenschmidt wrote: > > > > On Thu, 2012-08-23 at 11:02 +0530, Srikar Dronamraju wrote: > > > > > > > > > > insn is updated/accessed in the arch independent code. Size of > > > uprobe_opcode_t could be different for different archs. > >

Re: [git pull] Please pull powerpc.git merge branch (updated)

2012-08-23 Thread Kumar Gala
[ Added defconfig updates from Kim to fix NFS boot on yocto ] The following changes since commit 0d7614f09c1ebdbaa1599a5aba7593f147bf96ee: Linux 3.6-rc1 (2012-08-02 16:38:10 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/galak/powerpc.git merge

Re: [PATCH] powerpc: dts: 52xx: drop critical IRQ from dts files

2012-08-23 Thread Grant Likely
On Thu, Aug 23, 2012 at 4:31 PM, Wolfram Sang wrote: > The wakeup-gpios have been assigned a critical IRQ in current > devicetrees. The PIC driver does not support critical IRQs, though, which > leads to the following error when booting (a PCM030 in this case): > > mpc52xx_irqhost_map: inv

Re: [PATCH 1/2] powerpc/fsl: update defconfigs

2012-08-23 Thread Kumar Gala
On Aug 22, 2012, at 1:43 PM, Kim Phillips wrote: > run make savedefconfig on fsl defconfigs. > > Signed-off-by: Kim Phillips > --- > arch/powerpc/configs/85xx/p1023rds_defconfig | 30 --- > arch/powerpc/configs/corenet32_smp_defconfig | 28 +++-- > arch

Re: [PATCH 2/2 v2] powerpc/fsl: fix "Failed to mount /dev: No such device" errors

2012-08-23 Thread Kumar Gala
On Aug 22, 2012, at 1:43 PM, Kim Phillips wrote: > Yocto (Built by Poky 7.0) 1.2 root filesystems fail to boot, > at least over nfs, with: > > Failed to mount /dev: No such device > > Configuring DEVTMPFS fixes it. > > Signed-off-by: Kim Phillips > --- > v2: separated savedefconfig updates, r

[PATCH] powerpc: dts: 52xx: drop critical IRQ from dts files

2012-08-23 Thread Wolfram Sang
The wakeup-gpios have been assigned a critical IRQ in current devicetrees. The PIC driver does not support critical IRQs, though, which leads to the following error when booting (a PCM030 in this case): mpc52xx_irqhost_map: invalid irq: virq=16, l1=0, l2=3 irq: irq-16==>hwirq-0x3 m

Re: [PATCH v4 2/2] powerpc: Uprobes port to powerpc

2012-08-23 Thread Oleg Nesterov
On 08/23, Benjamin Herrenschmidt wrote: > > On Thu, 2012-08-23 at 11:02 +0530, Srikar Dronamraju wrote: > > > > > > > insn is updated/accessed in the arch independent code. Size of > > uprobe_opcode_t could be different for different archs. > > uprobe_opcode_t > > represents the size of the smalles

Re: [PATCH v4 2/2] powerpc: Uprobes port to powerpc

2012-08-23 Thread Benjamin Herrenschmidt
On Thu, 2012-08-23 at 11:02 +0530, Srikar Dronamraju wrote: > > > > insn is updated/accessed in the arch independent code. Size of > uprobe_opcode_t could be different for different archs. > uprobe_opcode_t > represents the size of the smallest breakpoint instruction for an > arch. > > Hence u8

[PATCH V9] powerpc/fsl-pci: Unify pci/pcie initialization code

2012-08-23 Thread Jia Hongtao
We unified the Freescale pci/pcie initialization by changing the fsl_pci to a platform driver. In previous PCI code architecture the initialization routine is called at board_setup_arch stage. Now the initialization is done in probe function which is architectural better. Also It's convenient for a

Re: [PATCH 1/2] PCI: leave MEM and IO decoding disabled during 64-bit BAR sizing, too

2012-08-23 Thread Olof Johansson
Hi, On Mon, Jul 9, 2012 at 11:20 AM, Bjorn Helgaas wrote: > After 253d2e5498, we disable MEM and IO decoding for most devices while we > size 32-bit BARs. However, we restore the original COMMAND register before > we size the upper 32 bits of 64-bit BARs, so we can still cause a conflict. > > Th