Project

General

Profile

Bug #9328

zap code can take advantage of c99

Added by Brad Lewis over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Start date:
2018-03-22
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
needs-triage

Description

The ZAP code was written before we allowed c99 in the Solaris kernel. We should change it to take advantage of being able to declare variables where they are first used. This reduces variable scope and means less scrolling to find the type of variables.

History

#1

Updated by Electric Monk about 1 year ago

  • % Done changed from 0 to 100
  • Status changed from New to Closed

git commit bf26014c5541b6119f34e0d95294b7f2eb105ac2

commit  bf26014c5541b6119f34e0d95294b7f2eb105ac2
Author: Matthew Ahrens <mahrens@delphix.com>
Date:   2018-05-09T21:18:42.000Z

    9328 zap code can take advantage of c99
    9329 panic in zap_leaf_lookup() due to concurrent zapification
    Reviewed by: Steve Gonczi <steve.gonczi@delphix.com>
    Reviewed by: George Wilson <george.wilson@delphix.com>
    Reviewed by: Pavel Zakharov <pavel.zakharov@delphix.com>
    Reviewed by: Brad Lewis <brad.lewis@delphix.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom PDF