Re: [PATCH] staging: android: ion: Initialize dma_address of new sg list
From: Liam Mark
Date: Fri Feb 16 2018 - 12:57:19 EST
On Fri, 16 Feb 2018, Greg KH wrote:
> On Fri, Feb 09, 2018 at 10:16:56PM -0800, Liam Mark wrote:
> > Fix the dup_sg_table function to initialize the dma_address of the new
> > sg list entries instead of the source dma_address entries.
> >
> > Fixes: 17fd283f3870 ("staging: android: ion: Duplicate sg_table")
>
> So this should be sent to the stable trees as well, right?
>
> Please add that when you resend...
My understanding was that I should only send this to the stable branch if
it fixes a real bug.
Both myself and Laura can't see any actual problems that result from this
issue, the change is more to help future proof the code.
My commit message wasn't clear about this and could have mislead people
into thinking there was a bug.
I will update the commit message to make it clear that this issue doesn't
currently result in any problem.
Do you still want me to send it to the stable trees?
Thanks,
Liam
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project