This reply comments on the patch to fixes the missing a blank line warning

[David Hildenbrand <david@redhat.com>]

^

Please create proper patch subjects. Nobody has a glue what you are 
doing when looking at the subject.

"mm/util: fix ??? warning"

Which raises the question, what is ???

Compiler? static code checker? ... ?


Thanks

On 11.02.21 08:29, Adithya Chandrakasan wrote:
> FILE: mm/util.c:930: WARNING: Missing a blank line after declarations
> 
> Signed-off-by: Adithya Chandrakasan <adithya.chandrakasan@gmail.com>
> ---
>   mm/util.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/mm/util.c b/mm/util.c
> index 8c9b7d1e7c49..60286876636d 100644
> --- a/mm/util.c
> +++ b/mm/util.c
> @@ -927,6 +927,7 @@ int get_cmdline(struct task_struct *task, char *buffer, int buflen)
>   	unsigned int len;
>   	struct mm_struct *mm = get_task_mm(task);
>   	unsigned long arg_start, arg_end, env_start, env_end;
> +
>   	if (!mm)
>   		goto out;
>   	if (!mm->arg_end)
> 


-- 
Thanks,

David / dhildenb