Project

General

Profile

Actions
Copy link

Bug #9328

closed

zap code can take advantage of c99

Added by Brad Lewis over 4 years ago. Updated about 4 years ago.

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

100%

Estimated time:
Difficulty:
Medium
Tags:
needs-triage
Gerrit CR:

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.

Actions
Copy link
 #1

Updated by Electric Monk about 4 years ago

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

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>

Actions
Copy link

Also available in: Atom PDF