Project

General

Profile

Submit #2764

[PATCH] sys/vfs/hammer: fix comments regarding obsolete "spike" code

Added by tkusumi almost 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
VFS subsystem
Target version:
Start date:
01/08/2015
Due date:
% Done:

100%


Description

This patch fixes comments regarding "spike" code that only existed in early days of hammer development. The spike comments have been left unchanged even after the code was removed in 47197d71536907482d2d215e222600eed3aedc0e.
This patch is kernel cleanup in addition to userspace cleanup http://bugs.dragonflybsd.org/issues/2761. These two patches get rid of spike code at least in grep level.

\# git checkout master
git Already on 'master'
\# git pull
Already up-to-date.
\# grep -nrIi spike sys/vfs/hammer sbin/hammer
sys/vfs/hammer/hammer_cursor.c:645: * a spike then the current node must be a leaf node.
sys/vfs/hammer/hammer_btree.c:1011: * the spike), but ENOSPC is returned.
sys/vfs/hammer/hammer_btree.c:1318: * to a leaf to provide the spike code with a good point
sys/vfs/hammer/hammer_btree.c:1463: * (ENOENT) or spike (ENOSPC) operation.
sys/vfs/hammer/hammer_btree.c:1794: * Spikes are made up of two leaf elements which cannot be
sbin/hammer/cmd_show.c:49: int depth, int spike, hammer_tid_t mirror_tid,
sbin/hammer/cmd_show.c:113: int depth, int spike, hammer_tid_t mirror_tid,
sbin/hammer/cmd_show.c:146: if (spike == 0) {
sbin/hammer/cmd_show.c:225: search, depth + 1, spike,
\# git checkout -b test
Switched to a new branch 'test'
\# git am ./0001-sbin-hammer-remove-obsolete-spike-code-from-hammer-s.patch
Applying: sbin/hammer: remove obsolete "spike" code from hammer show command
\# git am ./0001-sys-vfs-hammer-fix-comments-regarding-obsolete-spike.patch
Applying: sys/vfs/hammer: fix comments regarding obsolete "spike" code
\# grep -nrIi spike sys/vfs/hammer sbin/hammer
\#

Associated revisions

Revision fd1fcc1a (diff)
Added by tkusumi almost 3 years ago

sys/vfs/hammer: fix comments regarding obsolete "spike" code

- This patch fixes comments regarding "spike" code that only existed in
early days of hammer development.
- The spike comments have been left unchanged even after the code was
removed in 47197d71536907482d2d215e222600eed3aedc0e.

Closes: #2764

Revision 46ff9749 (diff)
Added by tkusumi almost 3 years ago

sys/vfs/hammer: fix comments regarding obsolete "spike" code

- This patch fixes comments regarding "spike" code that only existed in
early days of hammer development.
- The spike comments have been left unchanged even after the code was
removed in 47197d71536907482d2d215e222600eed3aedc0e.

Closes: #2764

History

#1 Updated by tuxillo almost 3 years ago

  • Assignee set to tuxillo
  • Target version set to 4.2.x

#2 Updated by tkusumi almost 3 years ago

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

Also available in: Atom PDF