linux-kernel - Re: [PATCH 0/6] Add improvements suggested by checkpatch for vmbus

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0870858d-7f92-28f0-e9ca-a2e6f23bcc2a@castello.eng.br>
Date:   Tue, 24 Nov 2020 21:53:02 -0300
From:   Matheus Castello <matheus@...tello.eng.br>
To:     Wei Liu <wei.liu@...nel.org>, mikelley@...rosoft.com
Cc:     kys@...rosoft.com, haiyangz@...rosoft.com, sthemmin@...rosoft.com,
        sashal@...nel.org, Tianyu.Lan@...rosoft.com, decui@...rosoft.com,
        sunilmut@...rosoft.com, linux-hyperv@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/6] Add improvements suggested by checkpatch for
 vmbus_drv

Hi Wei,

Em 11/17/2020 7:58 AM, Wei Liu escreveu:
> On Sun, Nov 15, 2020 at 04:57:28PM -0300, Matheus Castello wrote:
>> This series fixes some warnings edmited by the checkpatch in the file
>> vmbus_drv.c. I thought it would be good to split each fix into a commit to
>> help with the review.
>>
>> Matheus Castello (6):
>>    drivers: hv: Fix hyperv_record_panic_msg path on comment
>>    drivers: hv: vmbus: Replace symbolic permissions by octal permissions
>>    drivers: hv: vmbus: Fix checkpatch LINE_SPACING
>>    drivers: hv: vmbus: Fix checkpatch SPLIT_STRING
>>    drivers: hv: vmbus: Fix unnecessary OOM_MESSAGE
>>    drivers: hv: vmbus: Fix call msleep using < 20ms
> 
> I've pushed patch 1-3 and 6 to hyperv-next.
> 
> Patch 4 has a pending comment from Michael. Patch 5 can be dropped.
>

Thanks for the review, got it and I agree with the drop of patch 5 
thanks to point it. I will send the v2 for the patch 4.

> Wei.
>