[PATCH] parisc iommu: fix panic due to trying to allocate too large region

2015-11-30 Thread Mikulas Patocka
When using the Promise TX2+ SATA controller on PA-RISC, the system often crashes with kernel panic, for example just writing data with the dd utility will make it crash. Kernel panic - not syncing: drivers/parisc/sba_iommu.c: I/O MMU @ a000 is out of mapping resources CPU: 0 PID: 184

Re: [PATCH 0/4] Cleanup legacy OMAP IOMMU device creation

2015-11-30 Thread Tony Lindgren
* Suman Anna [151022 10:16]: > Hi Tony, > > On 09/16/2015 06:48 PM, Suman Anna wrote: > > Hi Tony, > > > > The following series removes the legacy platform device creation > > logic for OMAP IOMMU devices. I will cleanup the legacy support > > from the OMAP IOMMU driver in a subsequent merge win

Re: [Patch v2 7/9] iommu/amd: copy old tables and do not update dev tables before driver init

2015-11-30 Thread Baoquan He
On 11/27/15 at 12:35pm, Joerg Roedel wrote: > On Fri, Nov 06, 2015 at 08:10:49PM +0800, Baoquan He wrote: > > Signed-off-by: Baoquan He > > Missing patch description. > > > --- > > drivers/iommu/amd_iommu.c | 19 +-- > > drivers/iommu/amd_iommu_init.c | 71 > >

Re: [Patch v2 6/9] iommu/amd: Add functions copy_command_buffer/copy_event_buffer

2015-11-30 Thread Baoquan He
On 11/27/15 at 12:24pm, Joerg Roedel wrote: > On Fri, Nov 06, 2015 at 08:10:48PM +0800, Baoquan He wrote: > > > > +static void copy_command_buffer(void) > > + > > +static void copy_event_buffer(void) > > +{ > > There is no need to copy any of these buffers. For the command buffer > you just need

Re: [Patch v2 5/9] iommu/amd: Add function copy_dev_tables

2015-11-30 Thread Baoquan He
On 11/27/15 at 12:21pm, Joerg Roedel wrote: > On Fri, Nov 06, 2015 at 08:10:47PM +0800, Baoquan He wrote: > > +static void copy_dev_tables(void) > > +{ > > +u64 entry; > > +u32 lo, hi; > > +phys_addr_t old_devtb_phys; > > +struct dev_table_entry *old_devtb; > > +

Re: [Patch v2 4/9] iommu/amd: add copy_irq_table function

2015-11-30 Thread Baoquan He
On 11/27/15 at 12:13pm, Joerg Roedel wrote: > On Fri, Nov 06, 2015 at 08:10:46PM +0800, Baoquan He wrote: > > +static void copy_irq_table(u16 devid) > > +{ > > + struct irq_remap_table *table = NULL; > > + u16 alias; > > + u64 dte; > > + u64 old_intr_virt; > > + > > + alias = amd_iommu_al

Re: [Patch v2 3/9] iommu/amd: make several functions globally seen

2015-11-30 Thread Baoquan He
On 11/27/15 at 12:06pm, Joerg Roedel wrote: > On Fri, Nov 06, 2015 at 08:10:45PM +0800, Baoquan He wrote: > > They will be called later when copy old dev/irq tables. It's better to use > > them > > then call iommu_flush_all_caches() since iommu_flush_all_caches() will > > iterate many empty table

Re: [Patch v2 2/9] iommu/amd: Detect pre enabled translation

2015-11-30 Thread Baoquan He
On 11/27/15 at 12:03pm, Joerg Roedel wrote: > On Fri, Nov 06, 2015 at 08:10:44PM +0800, Baoquan He wrote: > > Add functions to check whether translation is already enabled in IOMMU. > > > > Signed-off-by: Baoquan He > > --- > > drivers/iommu/amd_iommu_init.c | 26 ++ > >

Re: [Patch v2 0/9] Fix AMD IOMMU faults in kdump kernel

2015-11-30 Thread Baoquan He
Hi Joerg, Thanks a lot for reviewing this patchset. On 11/27/15 at 12:38pm, Joerg Roedel wrote: > On Fri, Nov 06, 2015 at 08:10:42PM +0800, Baoquan He wrote: > > This is v2 draft patch set. It mainly functions as the following steps: > > > > 1. Checking if it's in kdump kernel and previously ena