Re: [PATCHv2 01/02] block: removes dynamic allocation on stack

[Jens Axboe]

On Tue, Mar 26 2013, Philip J. Kelleher wrote:
> From: Philip J Kelleher <pjk1939@xxxxxxxxxxxxxxxxxx>
> 
> Removing the dynamic allocation on the stack.
> 
> Signed-off-by: Philip J Kelleher <pjk1939@xxxxxxxxxxxxxxxxxx>
> -------------------------------------------------------------------------------
> o Version 2 consists of the error checking that was foolishly left out.
> o Added error checking to multiple functions to support the dynamically
>   allocated list_head array.

I've been fixing up your patch formats for a while now, lets please just
improve on it a little bit. First of all, your subject line should be
prefixed with the driver name. So just use rsxx or similar. Secondly,
for this patch, you are using the same subject line as the original
patch. This is a fixup patch, it should be labelled as such. A proper
subject line for this patch would be:

rsxx: enable error return of rsxx_eeh_save_issued_dmas()

and the body could be:

Commit d8d595df introduced a bug where we did not check for a NULL
return from kmalloc(). Make rsxx_eeh_save_issued_dmas() return an
error for that case, and make the callers handle that.

You appear to put the description of the patch in a section below your
signed-off-by, which doesn't really work. Your --- divider gets broken
with git am as well.


-- 
Jens Axboe

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/