Project

General

Profile

Actions
Copy link

Bug #3358

closed

[panic] mount_hammer2 a non-existing path

Added by fgudin about 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Low
Assignee:
-
Category:
Kernel
Target version:
6.4
Start date:
09/14/2023
Due date:
% Done:

0%

Estimated time:

Description

Hi,

a foolish attempt at ’mount_hammer2 /non/existing/path /mnt’ leads to:
panic: assertion "strncmp(path, "/dev/", 5) == 0" failed in hammer2_init_devvp at /usr/src/sys/vfs/hammer2/hammer2_ondisk.c:197

For the context, i mis-copypasted a PFS label.

If the dump is useful i can provide it. Just let me know.

Thanks in advance,
--
Francis

Actions
Copy link
 #1

Updated by liweitianux about 1 year ago

This assertion has been removed in commit https://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/8fd12ec on the master branch.
If it works as expected (i.e., the mount command fails with the correct error), we should also cherry-pick it to the release branch.

Actions
Copy link
 #2

Updated by fgudin about 1 year ago

Right, the behaviour is correct on master:

root@aph-mon2:~ # uname -a
DragonFly aph-mon2.asf.pri 6.5-DEVELOPMENT DragonFly v6.5.0.266.g0f9a5-DEVELOPMENT #0: Fri Sep 15 09:16:28 CEST 2023     root@aph-mon2.asf.pri:/usr/obj/usr/src/sys/X86_64_GENERIC  x86_64
root@aph-mon2:~ # mount_hammer2 /non/existing/path /mnt
mount_hammer2: /non/existing/path@DATA not found
root@aph-mon2:~ #

Actions
Copy link
 #3

Updated by liweitianux about 1 year ago

  • Status changed from New to Resolved
  • Target version changed from 6.6 to 6.4

Cherry-picked the commit to 6.4 release branch. Thank you for testing.

Actions
Copy link

Also available in: Atom PDF