On 24/05/2021 23:12, samirweng1979 wrote: > From: wengjianfeng > > In some functions, use goto exit statement, which just return r. This is not parseable. Imperative "use goto" suggest that it's the change you make. > and exit label only used once in each funciton, so we use return Every sentence in English starts with capital letter. They however cannto start with "and" because it joins two parts of sentences. s/funciton/use the spellcheck please/ The code looks good, but please spend some time to make the description understandable. You can also make it very short, to avoid complicated sentences. Look for examples in Linux kernel how other people wrote it. Best regards, Krzysztof