Re: [PATCH] staging: android: ion: Initialize dma_address of new sg list

From: Greg KH
Date: Fri Feb 16 2018 - 14:06:16 EST


On Fri, Feb 16, 2018 at 09:57:12AM -0800, Liam Mark wrote:
> 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?

If it's not a bugfix, no need to. But please clean up your changelog
text as it implies that it is a bugfix...

thanks,

greg k-h