* Re: [PATCH] dmaengine: bcm-sba-raid: Handle mbox_request_channel failure [not found] <1547100464-7020-1-git-send-email-rayagonda.kokatanur@broadcom.com> @ 2019-01-10 17:36 ` Ray Jui 2019-09-27 4:08 ` Rayagonda Kokatanur 0 siblings, 1 reply; 3+ messages in thread From: Ray Jui @ 2019-01-10 17:36 UTC (permalink / raw) To: Rayagonda Kokatanur, Vinod Koul Cc: Dan Williams, dmaengine, linux-kernel, bcm-kernel-feedback-list On 1/9/2019 10:07 PM, Rayagonda Kokatanur wrote: > Fix kernel NULL pointer dereference error when mbox_request_channel() > fails to allocate channel. > > Fixes: 4e9f8187aecb ("dmaengine: bcm-sba-raid: Use only single mailbox channel") > Signed-off-by: Rayagonda Kokatanur <rayagonda.kokatanur@broadcom.com> > --- > drivers/dma/bcm-sba-raid.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/dma/bcm-sba-raid.c b/drivers/dma/bcm-sba-raid.c > index 72878ac5c78d..77ae74663a45 100644 > --- a/drivers/dma/bcm-sba-raid.c > +++ b/drivers/dma/bcm-sba-raid.c > @@ -1690,7 +1690,7 @@ static int sba_probe(struct platform_device *pdev) > sba->mchan = mbox_request_channel(&sba->client, 0); > if (IS_ERR(sba->mchan)) { > ret = PTR_ERR(sba->mchan); > - goto fail_free_mchan; > + goto fail_exit; > } > > /* Find-out underlying mailbox device */ > @@ -1747,6 +1747,7 @@ static int sba_probe(struct platform_device *pdev) > sba_freeup_channel_resources(sba); > fail_free_mchan: > mbox_free_channel(sba->mchan); > +fail_exit: > return ret; > } > > Looks good to me. Reviewed-by: Ray Jui <ray.jui@broadcom.com> ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dmaengine: bcm-sba-raid: Handle mbox_request_channel failure 2019-01-10 17:36 ` [PATCH] dmaengine: bcm-sba-raid: Handle mbox_request_channel failure Ray Jui @ 2019-09-27 4:08 ` Rayagonda Kokatanur 2019-10-14 8:30 ` Vinod Koul 0 siblings, 1 reply; 3+ messages in thread From: Rayagonda Kokatanur @ 2019-09-27 4:08 UTC (permalink / raw) To: Vinod Koul Cc: Dan Williams, dmaengine, Linux Kernel Mailing List, BCM Kernel Feedback, Ray Jui Hi Vinod, Did you get chance to review this fix? Best regards, Rayagonda On Thu, Jan 10, 2019 at 11:06 PM Ray Jui <ray.jui@broadcom.com> wrote: > > > > On 1/9/2019 10:07 PM, Rayagonda Kokatanur wrote: > > Fix kernel NULL pointer dereference error when mbox_request_channel() > > fails to allocate channel. > > > > Fixes: 4e9f8187aecb ("dmaengine: bcm-sba-raid: Use only single mailbox channel") > > Signed-off-by: Rayagonda Kokatanur <rayagonda.kokatanur@broadcom.com> > > --- > > drivers/dma/bcm-sba-raid.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/dma/bcm-sba-raid.c b/drivers/dma/bcm-sba-raid.c > > index 72878ac5c78d..77ae74663a45 100644 > > --- a/drivers/dma/bcm-sba-raid.c > > +++ b/drivers/dma/bcm-sba-raid.c > > @@ -1690,7 +1690,7 @@ static int sba_probe(struct platform_device *pdev) > > sba->mchan = mbox_request_channel(&sba->client, 0); > > if (IS_ERR(sba->mchan)) { > > ret = PTR_ERR(sba->mchan); > > - goto fail_free_mchan; > > + goto fail_exit; > > } > > > > /* Find-out underlying mailbox device */ > > @@ -1747,6 +1747,7 @@ static int sba_probe(struct platform_device *pdev) > > sba_freeup_channel_resources(sba); > > fail_free_mchan: > > mbox_free_channel(sba->mchan); > > +fail_exit: > > return ret; > > } > > > > > > Looks good to me. > > Reviewed-by: Ray Jui <ray.jui@broadcom.com> ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dmaengine: bcm-sba-raid: Handle mbox_request_channel failure 2019-09-27 4:08 ` Rayagonda Kokatanur @ 2019-10-14 8:30 ` Vinod Koul 0 siblings, 0 replies; 3+ messages in thread From: Vinod Koul @ 2019-10-14 8:30 UTC (permalink / raw) To: Rayagonda Kokatanur Cc: Vinod Koul, Dan Williams, dmaengine, Linux Kernel Mailing List, BCM Kernel Feedback, Ray Jui Hi Rayagonda, On 27-09-19, 09:38, Rayagonda Kokatanur wrote: > Hi Vinod, > > Did you get chance to review this fix? Please do *not* top post And on your question, sorry this is not in my queue somehow, please rebase and repost > > Best regards, > Rayagonda > > > On Thu, Jan 10, 2019 at 11:06 PM Ray Jui <ray.jui@broadcom.com> wrote: > > > > > > > > On 1/9/2019 10:07 PM, Rayagonda Kokatanur wrote: > > > Fix kernel NULL pointer dereference error when mbox_request_channel() > > > fails to allocate channel. > > > > > > Fixes: 4e9f8187aecb ("dmaengine: bcm-sba-raid: Use only single mailbox channel") > > > Signed-off-by: Rayagonda Kokatanur <rayagonda.kokatanur@broadcom.com> > > > --- > > > drivers/dma/bcm-sba-raid.c | 3 ++- > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/dma/bcm-sba-raid.c b/drivers/dma/bcm-sba-raid.c > > > index 72878ac5c78d..77ae74663a45 100644 > > > --- a/drivers/dma/bcm-sba-raid.c > > > +++ b/drivers/dma/bcm-sba-raid.c > > > @@ -1690,7 +1690,7 @@ static int sba_probe(struct platform_device *pdev) > > > sba->mchan = mbox_request_channel(&sba->client, 0); > > > if (IS_ERR(sba->mchan)) { > > > ret = PTR_ERR(sba->mchan); > > > - goto fail_free_mchan; > > > + goto fail_exit; > > > } > > > > > > /* Find-out underlying mailbox device */ > > > @@ -1747,6 +1747,7 @@ static int sba_probe(struct platform_device *pdev) > > > sba_freeup_channel_resources(sba); > > > fail_free_mchan: > > > mbox_free_channel(sba->mchan); > > > +fail_exit: > > > return ret; > > > } > > > > > > > > > > Looks good to me. > > > > Reviewed-by: Ray Jui <ray.jui@broadcom.com> -- ~Vinod ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-10-14 8:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1547100464-7020-1-git-send-email-rayagonda.kokatanur@broadcom.com>
2019-01-10 17:36 ` [PATCH] dmaengine: bcm-sba-raid: Handle mbox_request_channel failure Ray Jui
2019-09-27 4:08 ` Rayagonda Kokatanur
2019-10-14 8:30 ` Vinod Koul
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®