[lkml]   [2019]   [Dec]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] KVM: vmx: remove unreachable statement in vmx_get_msr_feature()
On 06/12/19 04:41, linmiaohe wrote:
> Wanpeng Li <> wrote:
>>>> I personally just prefer to remove the “default” case and change this “return 0;” to “return 1;”.
>>>> But it’s a matter of taste of course.
>>> Yes. As what " Turnip greens, all have love " said. ^_^
>> Actually it is a great appreciated to introduce something more useful instead of tons of cleanups, I saw guys did one cleanup and can incur several bugs before.
> I'd like to introduce something more useful, but side corner cleanups may be hard to
> found out something to introduce. And such cleanups can also be validated by code inspection
> to avoid something bad. Many thanks.

Yeah, I think you have been doing a good job. Usually, when the
cleanups introduce bugs there are many other "suspicious" things. For
me it's clear that you're learning the code and not just messing around.


 \ /
  Last update: 2019-12-06 10:59    [W:0.034 / U:2.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site