On 08/20/2013 01:30 PM, Cyrill Gorcunov wrote: > On Tue, Aug 20, 2013 at 11:56:15AM +0800, Chen Gang wrote: >> For the implementation (patch 1/3), need fill buffer as full as >> possible when buffer space is not enough. >> >> For the caller (patch 2/3, 3/3), need check the return value of >> mpol_to_str(). >> >> Signed-off-by: Chen Gang <gang.c...@asianux.com> > > Won't simple check for mpol_to_str() < 0 be enough? IOW fix all > callers to check that mpol_to_str exit without errors. As far > as I see here are only two users. Something like > > show_numa_map > ret = mpol_to_str(); > if (ret) > return ret; > > shmem_show_mpol > ret = mpol_to_str(); > if (ret) > return ret; >
need "if (ret < 0)" instead of. ;-) > sure you'll have to change shmem_show_mpol statement to return int code. > Won't this be more short and convenient? > > Hmm... if return -ENOSPC, in common processing, it still need continue (but need let outside know about the string truncation). So I still suggest to give more check for it. Thanks. -- Chen Gang -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
