Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 31 Oct 2018 22:12:04 +0800
From:      Marcelo Araujo <araujobsdport@gmail.com>
To:        Shawn Webb <shawn.webb@hardenedbsd.org>
Cc:        Marcelo Araujo <araujo@freebsd.org>, src-committers <src-committers@freebsd.org>,  svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r339936 - head/sys/amd64/vmm/amd
Message-ID:  <CAOfEmZg%2BxOWE1dCnhMP3%2BXboX6Mfnsb8iFrLRKqOUkOi6m6h%2Bg@mail.gmail.com>
In-Reply-To: <20181031140631.ipwlwh3rlm7ma4xk@mutt-hbsd>
References:  <201810310127.w9V1RiFX025463@repo.freebsd.org> <20181031140631.ipwlwh3rlm7ma4xk@mutt-hbsd>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Oct 31, 2018, 10:07 PM Shawn Webb <shawn.webb@hardenedbsd.org>
wrote:

> On Wed, Oct 31, 2018 at 01:27:44AM +0000, Marcelo Araujo wrote:
> > Author: araujo
> > Date: Wed Oct 31 01:27:44 2018
> > New Revision: 339936
> > URL: https://svnweb.freebsd.org/changeset/base/339936
> >
> > Log:
> >   Merge cases with upper block.
> >   This is a cosmetic change only to simplify code.
> >
> >   Reported by:        anish
> >   Sponsored by:       iXsystems Inc.
> >
> > Modified:
> >   head/sys/amd64/vmm/amd/svm_msr.c
> >
> > Modified: head/sys/amd64/vmm/amd/svm_msr.c
> >
> ==============================================================================
> > --- head/sys/amd64/vmm/amd/svm_msr.c  Tue Oct 30 23:09:04 2018
> (r339935)
> > +++ head/sys/amd64/vmm/amd/svm_msr.c  Wed Oct 31 01:27:44 2018
> (r339936)
> > @@ -122,11 +122,7 @@ svm_rdmsr(struct svm_softc *sc, int vcpu, u_int
> num, u
> >       case MSR_MTRR16kBase ... MSR_MTRR16kBase + 1:
> >       case MSR_MTRR64kBase:
> >       case MSR_SYSCFG:
> > -             *result = 0;
> > -             break;
> >       case MSR_AMDK8_IPM:
> > -             *result = 0;
> > -             break;
> >       case MSR_EXTFEATURES:
> >               *result = 0;
> >               break;
>
> Does this need a /* FALLTHROUGH */ comment to appease the Coverity
> Gods?
>

Wow yeah, probably yes!!! I will check tomorrow!

Thanks.



> Thanks,
>
> --
> Shawn Webb
> Cofounder and Security Engineer
> HardenedBSD
>
> Tor-ified Signal:    +1 443-546-8752
> Tor+XMPP+OTR:        lattera@is.a.hacker.sx
> GPG Key ID:          0x6A84658F52456EEE
> GPG Key Fingerprint: 2ABA B6BD EF6A F486 BE89  3D9E 6A84 658F 5245 6EEE
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAOfEmZg%2BxOWE1dCnhMP3%2BXboX6Mfnsb8iFrLRKqOUkOi6m6h%2Bg>