Re: [PATCH RESEND 1/2] mtd: bcm47: convert kzalloc to avoid invalid access
On Thu, May 30, 2013 at 10:22:12AM +0800, Libo Chen wrote: > > mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its > member. > So I use devm_kazlloc instead of kazlloc to avoid it. > > * Changelog: >convert to devm_kzalloc > > Signed-off-by: Libo chen > Suggested-by: Andy Shevchenko Resolved a conflict and pushed to l2-mtd.git. Thanks! Brian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH RESEND 1/2] mtd: bcm47: convert kzalloc to avoid invalid access
On Thu, May 30, 2013 at 10:22:12AM +0800, Libo Chen wrote: mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its member. So I use devm_kazlloc instead of kazlloc to avoid it. * Changelog: convert to devm_kzalloc Signed-off-by: Libo chen libo.c...@huawei.com Suggested-by: Andy Shevchenko andy.shevche...@gmail.com Resolved a conflict and pushed to l2-mtd.git. Thanks! Brian -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH RESEND 1/2] mtd: bcm47: convert kzalloc to avoid invalid access
2013/5/30 Libo Chen : > mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its > member. > So I use devm_kazlloc instead of kazlloc to avoid it. I think you should use mtd: bcm47xxsflash: prefix for both patches, but Artem or David may want to comment on that. Nice trick with that devm_kzalloc, I didn't know it. -- Rafał -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH RESEND 1/2] mtd: bcm47: convert kzalloc to avoid invalid access
2013/5/30 Libo Chen clbchenlibo.c...@huawei.com: mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its member. So I use devm_kazlloc instead of kazlloc to avoid it. I think you should use mtd: bcm47xxsflash: prefix for both patches, but Artem or David may want to comment on that. Nice trick with that devm_kzalloc, I didn't know it. -- Rafał -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH RESEND 1/2] mtd: bcm47: convert kzalloc to avoid invalid access
On Thu, May 30, 2013 at 5:22 AM, Libo Chen wrote: > mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its > member. > So I use devm_kazlloc instead of kazlloc to avoid it. You may amend commit message, but technically you have my Reviewed-by: Andy Shevchenko > diff --git a/drivers/mtd/devices/bcm47xxsflash.c > b/drivers/mtd/devices/bcm47xxsflash.c > index 18e7761..2630d5c 100644 > --- a/drivers/mtd/devices/bcm47xxsflash.c > +++ b/drivers/mtd/devices/bcm47xxsflash.c > @@ -54,11 +54,9 @@ static int bcm47xxsflash_bcma_probe(struct platform_device > *pdev) > struct bcm47xxsflash *b47s; > int err; > > - b47s = kzalloc(sizeof(*b47s), GFP_KERNEL); > - if (!b47s) { > - err = -ENOMEM; > - goto out; > - } > + b47s = devm_kzalloc(>dev, sizeof(*b47s), GFP_KERNEL); > + if (!b47s) > + return -ENOMEM; > sflash->priv = b47s; > > b47s->bcma_cc = container_of(sflash, struct bcma_drv_cc, sflash); > @@ -81,15 +79,10 @@ static int bcm47xxsflash_bcma_probe(struct > platform_device *pdev) > err = mtd_device_parse_register(>mtd, probes, NULL, NULL, 0); > if (err) { > pr_err("Failed to register MTD device: %d\n", err); > - goto err_dev_reg; > + return err; > } > > return 0; > - > -err_dev_reg: > - kfree(>mtd); > -out: > - return err; > } > > static int bcm47xxsflash_bcma_remove(struct platform_device *pdev) > @@ -98,7 +91,6 @@ static int bcm47xxsflash_bcma_remove(struct platform_device > *pdev) > struct bcm47xxsflash *b47s = sflash->priv; > > mtd_device_unregister(>mtd); > - kfree(b47s); > > return 0; > } > -- > 1.7.1 > > -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH RESEND 1/2] mtd: bcm47: convert kzalloc to avoid invalid access
On Thu, May 30, 2013 at 5:22 AM, Libo Chen clbchenlibo.c...@huawei.com wrote: mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its member. So I use devm_kazlloc instead of kazlloc to avoid it. You may amend commit message, but technically you have my Reviewed-by: Andy Shevchenko andy.shevche...@gmail.com diff --git a/drivers/mtd/devices/bcm47xxsflash.c b/drivers/mtd/devices/bcm47xxsflash.c index 18e7761..2630d5c 100644 --- a/drivers/mtd/devices/bcm47xxsflash.c +++ b/drivers/mtd/devices/bcm47xxsflash.c @@ -54,11 +54,9 @@ static int bcm47xxsflash_bcma_probe(struct platform_device *pdev) struct bcm47xxsflash *b47s; int err; - b47s = kzalloc(sizeof(*b47s), GFP_KERNEL); - if (!b47s) { - err = -ENOMEM; - goto out; - } + b47s = devm_kzalloc(pdev-dev, sizeof(*b47s), GFP_KERNEL); + if (!b47s) + return -ENOMEM; sflash-priv = b47s; b47s-bcma_cc = container_of(sflash, struct bcma_drv_cc, sflash); @@ -81,15 +79,10 @@ static int bcm47xxsflash_bcma_probe(struct platform_device *pdev) err = mtd_device_parse_register(b47s-mtd, probes, NULL, NULL, 0); if (err) { pr_err(Failed to register MTD device: %d\n, err); - goto err_dev_reg; + return err; } return 0; - -err_dev_reg: - kfree(b47s-mtd); -out: - return err; } static int bcm47xxsflash_bcma_remove(struct platform_device *pdev) @@ -98,7 +91,6 @@ static int bcm47xxsflash_bcma_remove(struct platform_device *pdev) struct bcm47xxsflash *b47s = sflash-priv; mtd_device_unregister(b47s-mtd); - kfree(b47s); return 0; } -- 1.7.1 -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/