Project

General

Profile

Bug #10205

Mounting zfs filesystems on startup shows incorrect data

Added by Alexander Pyhalov 8 months ago. Updated 8 months ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Start date:
2019-01-10
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
needs-triage

Description

It seems we are incrementing sms->sm_done twice in share_mount_one_cb() -> report_mount_progress() in usr/src/cmd/zfs/zfs_main.c, so
a) header is never set (current == 2) initially
b) sometimes we see figures like (35/34).

History

#2

Updated by Alexander Pyhalov 8 months ago

  • Description updated (diff)
#3

Updated by Alexander Pyhalov 8 months ago

Tested change by copying zfs binary to /sbin/zfs.
With new zfs binary I see
correct "Mounting ZFS filesystems (x/y)" string on boot instead of "(null) (x/y)".

#4

Updated by Electric Monk 8 months ago

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

git commit 247b7da039fd88350c50e3d7fef15bdab6bef215

commit  247b7da039fd88350c50e3d7fef15bdab6bef215
Author: Alexander Pyhalov <apyhalov@gmail.com>
Date:   2019-01-11T06:14:24.000Z

    10205 Mounting zfs filesystems on startup shows incorrect data
    Reviewed by: Andy Stormont <astormont@racktopsystems.com>
    Reviewed by: Toomas Soome <tsoome@me.com>
    Approved by: Robert Mustacchi <rm@joyent.com>

Also available in: Atom PDF