Messages in this thread | | | Date | Thu, 4 Oct 2012 07:44:42 -0700 | From | Stephen Hemminger <> | Subject | Re: [PATCH 19/20] drivers/net/ethernet/marvell/skge.c: fix error return code |
| |
On Thu, 4 Oct 2012 11:05:55 +0200 Peter Senna Tschudin <peter.senna@gmail.com> wrote:
> > > > Thanks for looking into these kind of problems. The contents > > of the patch are correct, but the automated commit message is useless. > > You shouldn't just blindly say what the automated > > script was looking for, you should describe what the bug is so that evaluators > > can decide what the impact is and if it should be backported to stable > > and vendor kernels. > > > > Please resubmit the patchs with a reasonable analysis in the commit message. > > Something like: > > > > There is a bug in skge driver. If alloc_etherdev() fails, then > > skge_devinit() will return NULL, and the skge_probe function incorrectly > > returns success 0. It should return -ENOMEM instead. > > > > > > Stephen, I do not want to include function names on the commit > message. What do you think about this updated message, is it > acceptable? >
No still to generic, it needs to be written by a human examining the file and understanding what the cause and effect of the bug is.
| |