* [PATCH 0/2] SH Mobile IOMMU fixes @ 2013-12-17 17:18 ` Laurent Pinchart 0 siblings, 0 replies; 14+ messages in thread From: Laurent Pinchart @ 2013-12-17 17:18 UTC (permalink / raw) To: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA Cc: Simon Horman, Nguyen Viet Dung, linux-sh-u79uwXL29TY76Z2rM5mHXA Hello, These two small patches fix bugs in the SH Mobile IOMMU driver that result in crashes at runtime. Given that the bugs have been present in the driver since v3.9 there's no urgency to push the fixes to v3.13. Laurent Pinchart (2): iommu/shmobile: Allocate archdata with kzalloc() iommu/shmobile: Turn the flush_lock mutex into a spinlock drivers/iommu/shmobile-iommu.c | 3 +-- drivers/iommu/shmobile-ipmmu.c | 10 +++++----- drivers/iommu/shmobile-ipmmu.h | 2 +- 3 files changed, 7 insertions(+), 8 deletions(-) -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 0/2] SH Mobile IOMMU fixes @ 2013-12-17 17:18 ` Laurent Pinchart 0 siblings, 0 replies; 14+ messages in thread From: Laurent Pinchart @ 2013-12-17 17:18 UTC (permalink / raw) To: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA Cc: Simon Horman, Nguyen Viet Dung, linux-sh-u79uwXL29TY76Z2rM5mHXA Hello, These two small patches fix bugs in the SH Mobile IOMMU driver that result in crashes at runtime. Given that the bugs have been present in the driver since v3.9 there's no urgency to push the fixes to v3.13. Laurent Pinchart (2): iommu/shmobile: Allocate archdata with kzalloc() iommu/shmobile: Turn the flush_lock mutex into a spinlock drivers/iommu/shmobile-iommu.c | 3 +-- drivers/iommu/shmobile-ipmmu.c | 10 +++++----- drivers/iommu/shmobile-ipmmu.h | 2 +- 3 files changed, 7 insertions(+), 8 deletions(-) -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 2/2] iommu/shmobile: Turn the flush_lock mutex into a spinlock 2013-12-17 17:18 ` Laurent Pinchart @ 2013-12-17 17:18 ` Laurent Pinchart -1 siblings, 0 replies; 14+ messages in thread From: Laurent Pinchart @ 2013-12-17 17:18 UTC (permalink / raw) To: iommu; +Cc: linux-sh, Simon Horman, Nguyen Viet Dung The lock is taken in atomic context, replace it with a spinlock. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- drivers/iommu/shmobile-ipmmu.c | 10 +++++----- drivers/iommu/shmobile-ipmmu.h | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/iommu/shmobile-ipmmu.c b/drivers/iommu/shmobile-ipmmu.c index 8321f89..e3bc2e1 100644 --- a/drivers/iommu/shmobile-ipmmu.c +++ b/drivers/iommu/shmobile-ipmmu.c @@ -35,12 +35,12 @@ void ipmmu_tlb_flush(struct shmobile_ipmmu *ipmmu) if (!ipmmu) return; - mutex_lock(&ipmmu->flush_lock); + spin_lock(&ipmmu->flush_lock); if (ipmmu->tlb_enabled) ipmmu_reg_write(ipmmu, IMCTR1, IMCTR1_FLUSH | IMCTR1_TLBEN); else ipmmu_reg_write(ipmmu, IMCTR1, IMCTR1_FLUSH); - mutex_unlock(&ipmmu->flush_lock); + spin_unlock(&ipmmu->flush_lock); } void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, @@ -49,7 +49,7 @@ void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, if (!ipmmu) return; - mutex_lock(&ipmmu->flush_lock); + spin_lock(&ipmmu->flush_lock); switch (size) { default: ipmmu->tlb_enabled = 0; @@ -85,7 +85,7 @@ void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, } ipmmu_reg_write(ipmmu, IMTTBR, phys); ipmmu_reg_write(ipmmu, IMASID, asid); - mutex_unlock(&ipmmu->flush_lock); + spin_unlock(&ipmmu->flush_lock); } static int ipmmu_probe(struct platform_device *pdev) @@ -104,7 +104,7 @@ static int ipmmu_probe(struct platform_device *pdev) dev_err(&pdev->dev, "cannot allocate device data\n"); return -ENOMEM; } - mutex_init(&ipmmu->flush_lock); + spin_lock_init(&ipmmu->flush_lock); ipmmu->dev = &pdev->dev; ipmmu->ipmmu_base = devm_ioremap_nocache(&pdev->dev, res->start, resource_size(res)); diff --git a/drivers/iommu/shmobile-ipmmu.h b/drivers/iommu/shmobile-ipmmu.h index 4d53684..9524743 100644 --- a/drivers/iommu/shmobile-ipmmu.h +++ b/drivers/iommu/shmobile-ipmmu.h @@ -14,7 +14,7 @@ struct shmobile_ipmmu { struct device *dev; void __iomem *ipmmu_base; int tlb_enabled; - struct mutex flush_lock; + spinlock_t flush_lock; const char * const *dev_names; unsigned int num_dev_names; }; -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] iommu/shmobile: Turn the flush_lock mutex into a spinlock @ 2013-12-17 17:18 ` Laurent Pinchart 0 siblings, 0 replies; 14+ messages in thread From: Laurent Pinchart @ 2013-12-17 17:18 UTC (permalink / raw) To: iommu; +Cc: linux-sh, Simon Horman, Nguyen Viet Dung The lock is taken in atomic context, replace it with a spinlock. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- drivers/iommu/shmobile-ipmmu.c | 10 +++++----- drivers/iommu/shmobile-ipmmu.h | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/iommu/shmobile-ipmmu.c b/drivers/iommu/shmobile-ipmmu.c index 8321f89..e3bc2e1 100644 --- a/drivers/iommu/shmobile-ipmmu.c +++ b/drivers/iommu/shmobile-ipmmu.c @@ -35,12 +35,12 @@ void ipmmu_tlb_flush(struct shmobile_ipmmu *ipmmu) if (!ipmmu) return; - mutex_lock(&ipmmu->flush_lock); + spin_lock(&ipmmu->flush_lock); if (ipmmu->tlb_enabled) ipmmu_reg_write(ipmmu, IMCTR1, IMCTR1_FLUSH | IMCTR1_TLBEN); else ipmmu_reg_write(ipmmu, IMCTR1, IMCTR1_FLUSH); - mutex_unlock(&ipmmu->flush_lock); + spin_unlock(&ipmmu->flush_lock); } void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, @@ -49,7 +49,7 @@ void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, if (!ipmmu) return; - mutex_lock(&ipmmu->flush_lock); + spin_lock(&ipmmu->flush_lock); switch (size) { default: ipmmu->tlb_enabled = 0; @@ -85,7 +85,7 @@ void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, } ipmmu_reg_write(ipmmu, IMTTBR, phys); ipmmu_reg_write(ipmmu, IMASID, asid); - mutex_unlock(&ipmmu->flush_lock); + spin_unlock(&ipmmu->flush_lock); } static int ipmmu_probe(struct platform_device *pdev) @@ -104,7 +104,7 @@ static int ipmmu_probe(struct platform_device *pdev) dev_err(&pdev->dev, "cannot allocate device data\n"); return -ENOMEM; } - mutex_init(&ipmmu->flush_lock); + spin_lock_init(&ipmmu->flush_lock); ipmmu->dev = &pdev->dev; ipmmu->ipmmu_base = devm_ioremap_nocache(&pdev->dev, res->start, resource_size(res)); diff --git a/drivers/iommu/shmobile-ipmmu.h b/drivers/iommu/shmobile-ipmmu.h index 4d53684..9524743 100644 --- a/drivers/iommu/shmobile-ipmmu.h +++ b/drivers/iommu/shmobile-ipmmu.h @@ -14,7 +14,7 @@ struct shmobile_ipmmu { struct device *dev; void __iomem *ipmmu_base; int tlb_enabled; - struct mutex flush_lock; + spinlock_t flush_lock; const char * const *dev_names; unsigned int num_dev_names; }; -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] iommu/shmobile: Turn the flush_lock mutex into a spinlock 2013-12-17 17:18 ` Laurent Pinchart @ 2013-12-18 1:16 ` Simon Horman -1 siblings, 0 replies; 14+ messages in thread From: Simon Horman @ 2013-12-18 1:16 UTC (permalink / raw) To: Laurent Pinchart; +Cc: iommu, linux-sh, Nguyen Viet Dung On Tue, Dec 17, 2013 at 06:18:50PM +0100, Laurent Pinchart wrote: > The lock is taken in atomic context, replace it with a spinlock. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Reviewed-by: Simon Horman <horms+renesas@verge.net.au> > --- > drivers/iommu/shmobile-ipmmu.c | 10 +++++----- > drivers/iommu/shmobile-ipmmu.h | 2 +- > 2 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/iommu/shmobile-ipmmu.c b/drivers/iommu/shmobile-ipmmu.c > index 8321f89..e3bc2e1 100644 > --- a/drivers/iommu/shmobile-ipmmu.c > +++ b/drivers/iommu/shmobile-ipmmu.c > @@ -35,12 +35,12 @@ void ipmmu_tlb_flush(struct shmobile_ipmmu *ipmmu) > if (!ipmmu) > return; > > - mutex_lock(&ipmmu->flush_lock); > + spin_lock(&ipmmu->flush_lock); > if (ipmmu->tlb_enabled) > ipmmu_reg_write(ipmmu, IMCTR1, IMCTR1_FLUSH | IMCTR1_TLBEN); > else > ipmmu_reg_write(ipmmu, IMCTR1, IMCTR1_FLUSH); > - mutex_unlock(&ipmmu->flush_lock); > + spin_unlock(&ipmmu->flush_lock); > } > > void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, > @@ -49,7 +49,7 @@ void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, > if (!ipmmu) > return; > > - mutex_lock(&ipmmu->flush_lock); > + spin_lock(&ipmmu->flush_lock); > switch (size) { > default: > ipmmu->tlb_enabled = 0; > @@ -85,7 +85,7 @@ void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, > } > ipmmu_reg_write(ipmmu, IMTTBR, phys); > ipmmu_reg_write(ipmmu, IMASID, asid); > - mutex_unlock(&ipmmu->flush_lock); > + spin_unlock(&ipmmu->flush_lock); > } > > static int ipmmu_probe(struct platform_device *pdev) > @@ -104,7 +104,7 @@ static int ipmmu_probe(struct platform_device *pdev) > dev_err(&pdev->dev, "cannot allocate device data\n"); > return -ENOMEM; > } > - mutex_init(&ipmmu->flush_lock); > + spin_lock_init(&ipmmu->flush_lock); > ipmmu->dev = &pdev->dev; > ipmmu->ipmmu_base = devm_ioremap_nocache(&pdev->dev, res->start, > resource_size(res)); > diff --git a/drivers/iommu/shmobile-ipmmu.h b/drivers/iommu/shmobile-ipmmu.h > index 4d53684..9524743 100644 > --- a/drivers/iommu/shmobile-ipmmu.h > +++ b/drivers/iommu/shmobile-ipmmu.h > @@ -14,7 +14,7 @@ struct shmobile_ipmmu { > struct device *dev; > void __iomem *ipmmu_base; > int tlb_enabled; > - struct mutex flush_lock; > + spinlock_t flush_lock; > const char * const *dev_names; > unsigned int num_dev_names; > }; > -- > 1.8.3.2 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-sh" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] iommu/shmobile: Turn the flush_lock mutex into a spinlock @ 2013-12-18 1:16 ` Simon Horman 0 siblings, 0 replies; 14+ messages in thread From: Simon Horman @ 2013-12-18 1:16 UTC (permalink / raw) To: Laurent Pinchart; +Cc: iommu, linux-sh, Nguyen Viet Dung On Tue, Dec 17, 2013 at 06:18:50PM +0100, Laurent Pinchart wrote: > The lock is taken in atomic context, replace it with a spinlock. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Reviewed-by: Simon Horman <horms+renesas@verge.net.au> > --- > drivers/iommu/shmobile-ipmmu.c | 10 +++++----- > drivers/iommu/shmobile-ipmmu.h | 2 +- > 2 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/iommu/shmobile-ipmmu.c b/drivers/iommu/shmobile-ipmmu.c > index 8321f89..e3bc2e1 100644 > --- a/drivers/iommu/shmobile-ipmmu.c > +++ b/drivers/iommu/shmobile-ipmmu.c > @@ -35,12 +35,12 @@ void ipmmu_tlb_flush(struct shmobile_ipmmu *ipmmu) > if (!ipmmu) > return; > > - mutex_lock(&ipmmu->flush_lock); > + spin_lock(&ipmmu->flush_lock); > if (ipmmu->tlb_enabled) > ipmmu_reg_write(ipmmu, IMCTR1, IMCTR1_FLUSH | IMCTR1_TLBEN); > else > ipmmu_reg_write(ipmmu, IMCTR1, IMCTR1_FLUSH); > - mutex_unlock(&ipmmu->flush_lock); > + spin_unlock(&ipmmu->flush_lock); > } > > void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, > @@ -49,7 +49,7 @@ void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, > if (!ipmmu) > return; > > - mutex_lock(&ipmmu->flush_lock); > + spin_lock(&ipmmu->flush_lock); > switch (size) { > default: > ipmmu->tlb_enabled = 0; > @@ -85,7 +85,7 @@ void ipmmu_tlb_set(struct shmobile_ipmmu *ipmmu, unsigned long phys, int size, > } > ipmmu_reg_write(ipmmu, IMTTBR, phys); > ipmmu_reg_write(ipmmu, IMASID, asid); > - mutex_unlock(&ipmmu->flush_lock); > + spin_unlock(&ipmmu->flush_lock); > } > > static int ipmmu_probe(struct platform_device *pdev) > @@ -104,7 +104,7 @@ static int ipmmu_probe(struct platform_device *pdev) > dev_err(&pdev->dev, "cannot allocate device data\n"); > return -ENOMEM; > } > - mutex_init(&ipmmu->flush_lock); > + spin_lock_init(&ipmmu->flush_lock); > ipmmu->dev = &pdev->dev; > ipmmu->ipmmu_base = devm_ioremap_nocache(&pdev->dev, res->start, > resource_size(res)); > diff --git a/drivers/iommu/shmobile-ipmmu.h b/drivers/iommu/shmobile-ipmmu.h > index 4d53684..9524743 100644 > --- a/drivers/iommu/shmobile-ipmmu.h > +++ b/drivers/iommu/shmobile-ipmmu.h > @@ -14,7 +14,7 @@ struct shmobile_ipmmu { > struct device *dev; > void __iomem *ipmmu_base; > int tlb_enabled; > - struct mutex flush_lock; > + spinlock_t flush_lock; > const char * const *dev_names; > unsigned int num_dev_names; > }; > -- > 1.8.3.2 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-sh" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 14+ messages in thread
[parent not found: <1387300730-28783-1-git-send-email-laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>]
* [PATCH 1/2] iommu/shmobile: Allocate archdata with kzalloc() [not found] ` <1387300730-28783-1-git-send-email-laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org> @ 2013-12-17 17:18 ` Laurent Pinchart 2014-01-07 14:36 ` Joerg Roedel 1 sibling, 0 replies; 14+ messages in thread From: Laurent Pinchart @ 2013-12-17 17:18 UTC (permalink / raw) To: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA Cc: Simon Horman, Nguyen Viet Dung, linux-sh-u79uwXL29TY76Z2rM5mHXA The archdata attached_list field isn't initialized, leading to random crashes when accessed. Use kzalloc() to allocate the whole structure and make sure all fields get initialized properly. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org> --- drivers/iommu/shmobile-iommu.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/iommu/shmobile-iommu.c b/drivers/iommu/shmobile-iommu.c index d572863..7a3b928 100644 --- a/drivers/iommu/shmobile-iommu.c +++ b/drivers/iommu/shmobile-iommu.c @@ -380,14 +380,13 @@ int ipmmu_iommu_init(struct shmobile_ipmmu *ipmmu) kmem_cache_destroy(l1cache); return -ENOMEM; } - archdata = kmalloc(sizeof(*archdata), GFP_KERNEL); + archdata = kzalloc(sizeof(*archdata), GFP_KERNEL); if (!archdata) { kmem_cache_destroy(l1cache); kmem_cache_destroy(l2cache); return -ENOMEM; } spin_lock_init(&archdata->attach_lock); - archdata->attached = NULL; archdata->ipmmu = ipmmu; ipmmu_archdata = archdata; bus_set_iommu(&platform_bus_type, &shmobile_iommu_ops); -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 1/2] iommu/shmobile: Allocate archdata with kzalloc() @ 2013-12-17 17:18 ` Laurent Pinchart 0 siblings, 0 replies; 14+ messages in thread From: Laurent Pinchart @ 2013-12-17 17:18 UTC (permalink / raw) To: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA Cc: Simon Horman, Nguyen Viet Dung, linux-sh-u79uwXL29TY76Z2rM5mHXA The archdata attached_list field isn't initialized, leading to random crashes when accessed. Use kzalloc() to allocate the whole structure and make sure all fields get initialized properly. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- drivers/iommu/shmobile-iommu.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/iommu/shmobile-iommu.c b/drivers/iommu/shmobile-iommu.c index d572863..7a3b928 100644 --- a/drivers/iommu/shmobile-iommu.c +++ b/drivers/iommu/shmobile-iommu.c @@ -380,14 +380,13 @@ int ipmmu_iommu_init(struct shmobile_ipmmu *ipmmu) kmem_cache_destroy(l1cache); return -ENOMEM; } - archdata = kmalloc(sizeof(*archdata), GFP_KERNEL); + archdata = kzalloc(sizeof(*archdata), GFP_KERNEL); if (!archdata) { kmem_cache_destroy(l1cache); kmem_cache_destroy(l2cache); return -ENOMEM; } spin_lock_init(&archdata->attach_lock); - archdata->attached = NULL; archdata->ipmmu = ipmmu; ipmmu_archdata = archdata; bus_set_iommu(&platform_bus_type, &shmobile_iommu_ops); -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] iommu/shmobile: Allocate archdata with kzalloc() 2013-12-17 17:18 ` Laurent Pinchart @ 2013-12-18 1:15 ` Simon Horman -1 siblings, 0 replies; 14+ messages in thread From: Simon Horman @ 2013-12-18 1:15 UTC (permalink / raw) To: Laurent Pinchart; +Cc: iommu, linux-sh, Nguyen Viet Dung On Tue, Dec 17, 2013 at 06:18:49PM +0100, Laurent Pinchart wrote: > The archdata attached_list field isn't initialized, leading to random > crashes when accessed. Use kzalloc() to allocate the whole structure and > make sure all fields get initialized properly. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Reviewed-by: Simon Horman <horms+renesas@verge.net.au> > --- > drivers/iommu/shmobile-iommu.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/iommu/shmobile-iommu.c b/drivers/iommu/shmobile-iommu.c > index d572863..7a3b928 100644 > --- a/drivers/iommu/shmobile-iommu.c > +++ b/drivers/iommu/shmobile-iommu.c > @@ -380,14 +380,13 @@ int ipmmu_iommu_init(struct shmobile_ipmmu *ipmmu) > kmem_cache_destroy(l1cache); > return -ENOMEM; > } > - archdata = kmalloc(sizeof(*archdata), GFP_KERNEL); > + archdata = kzalloc(sizeof(*archdata), GFP_KERNEL); > if (!archdata) { > kmem_cache_destroy(l1cache); > kmem_cache_destroy(l2cache); > return -ENOMEM; > } > spin_lock_init(&archdata->attach_lock); > - archdata->attached = NULL; > archdata->ipmmu = ipmmu; > ipmmu_archdata = archdata; > bus_set_iommu(&platform_bus_type, &shmobile_iommu_ops); > -- > 1.8.3.2 > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] iommu/shmobile: Allocate archdata with kzalloc() @ 2013-12-18 1:15 ` Simon Horman 0 siblings, 0 replies; 14+ messages in thread From: Simon Horman @ 2013-12-18 1:15 UTC (permalink / raw) To: Laurent Pinchart; +Cc: iommu, linux-sh, Nguyen Viet Dung On Tue, Dec 17, 2013 at 06:18:49PM +0100, Laurent Pinchart wrote: > The archdata attached_list field isn't initialized, leading to random > crashes when accessed. Use kzalloc() to allocate the whole structure and > make sure all fields get initialized properly. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Reviewed-by: Simon Horman <horms+renesas@verge.net.au> > --- > drivers/iommu/shmobile-iommu.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/iommu/shmobile-iommu.c b/drivers/iommu/shmobile-iommu.c > index d572863..7a3b928 100644 > --- a/drivers/iommu/shmobile-iommu.c > +++ b/drivers/iommu/shmobile-iommu.c > @@ -380,14 +380,13 @@ int ipmmu_iommu_init(struct shmobile_ipmmu *ipmmu) > kmem_cache_destroy(l1cache); > return -ENOMEM; > } > - archdata = kmalloc(sizeof(*archdata), GFP_KERNEL); > + archdata = kzalloc(sizeof(*archdata), GFP_KERNEL); > if (!archdata) { > kmem_cache_destroy(l1cache); > kmem_cache_destroy(l2cache); > return -ENOMEM; > } > spin_lock_init(&archdata->attach_lock); > - archdata->attached = NULL; > archdata->ipmmu = ipmmu; > ipmmu_archdata = archdata; > bus_set_iommu(&platform_bus_type, &shmobile_iommu_ops); > -- > 1.8.3.2 > ^ permalink raw reply [flat|nested] 14+ messages in thread
[parent not found: <20131218011521.GI18992-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>]
* Re: [PATCH 1/2] iommu/shmobile: Allocate archdata with kzalloc() [not found] ` <20131218011521.GI18992-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org> @ 2013-12-18 14:14 ` Laurent Pinchart 0 siblings, 0 replies; 14+ messages in thread From: Laurent Pinchart @ 2013-12-18 14:14 UTC (permalink / raw) To: Joerg Roedel Cc: Nguyen Viet Dung, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA, Simon Horman, Laurent Pinchart, linux-sh-u79uwXL29TY76Z2rM5mHXA On Wednesday 18 December 2013 10:15:21 Simon Horman wrote: > On Tue, Dec 17, 2013 at 06:18:49PM +0100, Laurent Pinchart wrote: > > The archdata attached_list field isn't initialized, leading to random > > crashes when accessed. Use kzalloc() to allocate the whole structure and > > make sure all fields get initialized properly. > > > > Signed-off-by: Laurent Pinchart > > <laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org> > > Reviewed-by: Simon Horman <horms+renesas-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org> Thank you Simon. Joerg, could you please pick those two patches up for v3.14 ? > > --- > > > > drivers/iommu/shmobile-iommu.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/drivers/iommu/shmobile-iommu.c > > b/drivers/iommu/shmobile-iommu.c index d572863..7a3b928 100644 > > --- a/drivers/iommu/shmobile-iommu.c > > +++ b/drivers/iommu/shmobile-iommu.c > > @@ -380,14 +380,13 @@ int ipmmu_iommu_init(struct shmobile_ipmmu *ipmmu) > > kmem_cache_destroy(l1cache); > > return -ENOMEM; > > } > > - archdata = kmalloc(sizeof(*archdata), GFP_KERNEL); > > + archdata = kzalloc(sizeof(*archdata), GFP_KERNEL); > > if (!archdata) { > > kmem_cache_destroy(l1cache); > > kmem_cache_destroy(l2cache); > > return -ENOMEM; > > } > > spin_lock_init(&archdata->attach_lock); > > - archdata->attached = NULL; > > archdata->ipmmu = ipmmu; > > ipmmu_archdata = archdata; > > bus_set_iommu(&platform_bus_type, &shmobile_iommu_ops); -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] iommu/shmobile: Allocate archdata with kzalloc() @ 2013-12-18 14:14 ` Laurent Pinchart 0 siblings, 0 replies; 14+ messages in thread From: Laurent Pinchart @ 2013-12-18 14:14 UTC (permalink / raw) To: Joerg Roedel Cc: Nguyen Viet Dung, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA, Simon Horman, Laurent Pinchart, linux-sh-u79uwXL29TY76Z2rM5mHXA On Wednesday 18 December 2013 10:15:21 Simon Horman wrote: > On Tue, Dec 17, 2013 at 06:18:49PM +0100, Laurent Pinchart wrote: > > The archdata attached_list field isn't initialized, leading to random > > crashes when accessed. Use kzalloc() to allocate the whole structure and > > make sure all fields get initialized properly. > > > > Signed-off-by: Laurent Pinchart > > <laurent.pinchart+renesas@ideasonboard.com> > > Reviewed-by: Simon Horman <horms+renesas@verge.net.au> Thank you Simon. Joerg, could you please pick those two patches up for v3.14 ? > > --- > > > > drivers/iommu/shmobile-iommu.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/drivers/iommu/shmobile-iommu.c > > b/drivers/iommu/shmobile-iommu.c index d572863..7a3b928 100644 > > --- a/drivers/iommu/shmobile-iommu.c > > +++ b/drivers/iommu/shmobile-iommu.c > > @@ -380,14 +380,13 @@ int ipmmu_iommu_init(struct shmobile_ipmmu *ipmmu) > > kmem_cache_destroy(l1cache); > > return -ENOMEM; > > } > > - archdata = kmalloc(sizeof(*archdata), GFP_KERNEL); > > + archdata = kzalloc(sizeof(*archdata), GFP_KERNEL); > > if (!archdata) { > > kmem_cache_destroy(l1cache); > > kmem_cache_destroy(l2cache); > > return -ENOMEM; > > } > > spin_lock_init(&archdata->attach_lock); > > - archdata->attached = NULL; > > archdata->ipmmu = ipmmu; > > ipmmu_archdata = archdata; > > bus_set_iommu(&platform_bus_type, &shmobile_iommu_ops); -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] SH Mobile IOMMU fixes [not found] ` <1387300730-28783-1-git-send-email-laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org> @ 2014-01-07 14:36 ` Joerg Roedel 2014-01-07 14:36 ` Joerg Roedel 1 sibling, 0 replies; 14+ messages in thread From: Joerg Roedel @ 2014-01-07 14:36 UTC (permalink / raw) To: Laurent Pinchart Cc: linux-sh-u79uwXL29TY76Z2rM5mHXA, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA, Simon Horman, Nguyen Viet Dung On Tue, Dec 17, 2013 at 06:18:48PM +0100, Laurent Pinchart wrote: > Hello, > > These two small patches fix bugs in the SH Mobile IOMMU driver that result in > crashes at runtime. Given that the bugs have been present in the driver since > v3.9 there's no urgency to push the fixes to v3.13. > > Laurent Pinchart (2): > iommu/shmobile: Allocate archdata with kzalloc() > iommu/shmobile: Turn the flush_lock mutex into a spinlock > > drivers/iommu/shmobile-iommu.c | 3 +-- > drivers/iommu/shmobile-ipmmu.c | 10 +++++----- > drivers/iommu/shmobile-ipmmu.h | 2 +- > 3 files changed, 7 insertions(+), 8 deletions(-) Applied both, thanks. ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] SH Mobile IOMMU fixes @ 2014-01-07 14:36 ` Joerg Roedel 0 siblings, 0 replies; 14+ messages in thread From: Joerg Roedel @ 2014-01-07 14:36 UTC (permalink / raw) To: Laurent Pinchart Cc: linux-sh-u79uwXL29TY76Z2rM5mHXA, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA, Simon Horman, Nguyen Viet Dung On Tue, Dec 17, 2013 at 06:18:48PM +0100, Laurent Pinchart wrote: > Hello, > > These two small patches fix bugs in the SH Mobile IOMMU driver that result in > crashes at runtime. Given that the bugs have been present in the driver since > v3.9 there's no urgency to push the fixes to v3.13. > > Laurent Pinchart (2): > iommu/shmobile: Allocate archdata with kzalloc() > iommu/shmobile: Turn the flush_lock mutex into a spinlock > > drivers/iommu/shmobile-iommu.c | 3 +-- > drivers/iommu/shmobile-ipmmu.c | 10 +++++----- > drivers/iommu/shmobile-ipmmu.h | 2 +- > 3 files changed, 7 insertions(+), 8 deletions(-) Applied both, thanks. ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2014-01-07 14:36 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-17 17:18 [PATCH 0/2] SH Mobile IOMMU fixes Laurent Pinchart
2013-12-17 17:18 ` Laurent Pinchart
2013-12-17 17:18 ` [PATCH 2/2] iommu/shmobile: Turn the flush_lock mutex into a spinlock Laurent Pinchart
2013-12-17 17:18 ` Laurent Pinchart
2013-12-18 1:16 ` Simon Horman
2013-12-18 1:16 ` Simon Horman
[not found] ` <1387300730-28783-1-git-send-email-laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
2013-12-17 17:18 ` [PATCH 1/2] iommu/shmobile: Allocate archdata with kzalloc() Laurent Pinchart
2013-12-17 17:18 ` Laurent Pinchart
2013-12-18 1:15 ` Simon Horman
2013-12-18 1:15 ` Simon Horman
[not found] ` <20131218011521.GI18992-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
2013-12-18 14:14 ` Laurent Pinchart
2013-12-18 14:14 ` Laurent Pinchart
2014-01-07 14:36 ` [PATCH 0/2] SH Mobile IOMMU fixes Joerg Roedel
2014-01-07 14:36 ` Joerg Roedel
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.