Re: [PATCH 2/2] clean up to just return ERR_PTR

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 17 Mar 2015, denc716@xxxxxxxxx wrote:

> Signed-off-by: Derek Che <crquan@xxxxxxxxx>
> Acked-by: "Kirill A. Shutemov" <kirill@xxxxxxxxxxxxx>
> Acked-by: David Rientjes <rientjes@xxxxxxxxxx>

I don't believe Kirill ever acked this patch, you need to wait for the 
person to respond to your email with the line themself before you can add 
it to your commit message.  Please see Documentation/SubmittingPatches.

He also suggested that this patch be done, so you need to add

Suggested-by: Kirill A. Shutemov <kirill@xxxxxxxxxxxxx>

You need a proper subject line for the patch, in this case it would be 
something like "mm, mremap: avoid unnecessary gotos in vma_to_resize()" 
and you also need a proper commit message such as:

	The "goto"s in vma_to_resize() aren't necessary since they just
	return a specific value.  Switch these statements to explicit
	return statements.

My acked-by was never given before this, but it can be added because it 
looks good:

Acked-by: David Rientjes <rientjes@xxxxxxxxxx>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]