Re: [PATCH] PowerPC: kernel: need return the related error code whenfailure occurs.

From: Paul Mackerras
Date: Tue May 21 2013 - 04:11:07 EST


On Tue, May 21, 2013 at 01:48:58PM +0800, Chen Gang wrote:
>
> When error occurs, need return the related error code to let upper
> caller know about it.
>
> ppc_md.nvram_size() can return the error code (e.g. core99_nvram_size()
> in 'arch/powerpc/platforms/powermac/nvram.c').
>
> And when '*ppos >= size', need return -ESPIPE (Illegal seek)

Why? When *ppos >= size, it should return 0 (end of file) in my opinion.
ESPIPE means that any seek would be ineffective, not that a particular
seek went out of bounds.

Paul.
--
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/