Project

General

Profile

Actions

Bug #12996

closed

bhyve kernel should be wscheck clean

Added by Patrick Mooney over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Category:
bhyve
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
bhyve
Gerrit CR:

Description

When bhyve was imported into illumos, broad portions of it were added to the exception lists including the one for wscheck. Since the kernel side of bhyve is less focused on keeping deltas from upstream in ifdef guards, it should be fine to clean up whitespace nits and remove the exceptions.


Related issues

Related to illumos gate - Bug #13049: cstyle should tolerate freebsd headersClosedPatrick Mooney

Actions
Actions #1

Updated by Electric Monk over 1 year ago

  • Gerrit CR set to 821
Actions #2

Updated by Patrick Mooney over 1 year ago

  • Related to Bug #13049: cstyle should tolerate freebsd headers added
Actions #3

Updated by Patrick Mooney over 1 year ago

Since this is a whitespace-only change, I tested it with wsdiff. The only changes I saw, besides the normal DOF and version string noise, were assertion line number differences, because empty lines had been removed. (This indicating that the resulting binary was functionally identical.)

Actions #4

Updated by Electric Monk over 1 year ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit 84971882a96ac0fecd538b02208054a872ff8af3

commit  84971882a96ac0fecd538b02208054a872ff8af3
Author: Patrick Mooney <pmooney@pfmooney.com>
Date:   2020-08-21T05:03:26.000Z

    12996 bhyve kernel should be wscheck clean
    Reviewed by: Andy Fiddaman <andy@omniosce.org>
    Approved by: Robert Mustacchi <rm@fingolfin.org>

Actions

Also available in: Atom PDF