Thread (7 messages) 7 messages, 3 authors, 2015-02-28

Re: [PATCH v2] mm: cma: fix CMA aligned offset calculation

From: Danesh Petigara <hidden>
Date: 2015-02-27 23:53:07
Also in: lkml, stable

On 2/27/2015 1:24 PM, Andrew Morton wrote:
On Tue, 24 Feb 2015 15:39:45 -0800 Danesh Petigara [off-list ref] wrote:
quoted
The CMA aligned offset calculation is incorrect for
non-zero order_per_bit values.

For example, if cma->order_per_bit=1, cma->base_pfn=
0x2f800000 and align_order=12, the function returns
a value of 0x17c00 instead of 0x400.

This patch fixes the CMA aligned offset calculation.
When fixing a bug please always describe the end-user visible effects
of that bug.

Without that information others are unable to understand why you are
recommending a -stable backport.
Thank you for the feedback. I had no crash logs to show, nevertheless, I
agree that a sentence describing potential effects of the bug would've
helped.

I'll keep that in mind for future submissions.

Best Regards,
Danesh

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help