[lustre-devel] [PATCH 10/11] lustre: ldlm: checkpatch cleanup

James Simmons jsimmons at infradead.org
Wed Jul 24 18:54:04 PDT 2019


> Le 24/07/2019 05:38, « lustre-devel au nom de James Simmons » <lustre-devel-bounces at lists.lustre.org au nom de jsimmons at infradead.org> a écrit :
>     
>     From checkpatch - "ENOSYS means 'invalid syscall nr' and nothing else"
>     So using ENOSYS is no no. We get away with this because people rarely
>     actually handle each error code individually.
> .
>     
> As a general comment about this, I think we should improve this in the futur. Some people asked me "what's the error code when this or that happens in Lustre?". And I've got no real answer for that.
> Indeed people, most of the time, only checks returned code against 0. That does not help when looking at specific Lustre issues.

perf probe 'lustre_function%return $retval'

Does the same thing as lctl set_param debug=+trace except dynamic probes 
can work on any kernel function without code modification!!!


More information about the lustre-devel mailing list