Page MenuHomeFreeBSD

mckusick (Marshall Kirk McKusick)
User

Projects

User Details

User Since
Oct 24 2014, 7:17 PM (267 w, 3 d)

Recent Activity

Tue, Dec 3

mckusick committed rS355371: Currently the breadn_flags() and getblkx() interfaces are passed.
Currently the breadn_flags() and getblkx() interfaces are passed
Tue, Dec 3, 11:07 PM

Wed, Nov 20

mckusick committed rS354873: White space cleanup. No functional change..
White space cleanup. No functional change.
Wed, Nov 20, 1:14 AM
mckusick committed rS354872: Add some KASSERTs. Reacquire a mutex after a kernel printf rather.
Add some KASSERTs. Reacquire a mutex after a kernel printf rather
Wed, Nov 20, 1:10 AM

Oct 27 2019

mckusick committed rS354125: MFC of 354050.
MFC of 354050
Oct 27 2019, 4:11 AM
mckusick committed rS354124: MFC of 354050.
MFC of 354050
Oct 27 2019, 4:09 AM
mckusick committed rS354123: MFC of 353903.
MFC of 353903
Oct 27 2019, 3:47 AM
mckusick committed rS354122: MFC of 353903.
MFC of 353903
Oct 27 2019, 3:46 AM

Oct 24 2019

mckusick committed rS354050: After the unlink() of one name of a file with multiple links, a.
After the unlink() of one name of a file with multiple links, a
Oct 24 2019, 9:28 PM
mckusick committed rS354036: Soft updates needs to keep an on-disk linked list of inodes that.
Soft updates needs to keep an on-disk linked list of inodes that
Oct 24 2019, 7:47 PM

Oct 22 2019

mckusick committed rS353903: Replace an uninitialized variable with the correct element from the.
Replace an uninitialized variable with the correct element from the
Oct 22 2019, 10:24 PM

Oct 17 2019

mckusick accepted D22042: Abbreviate softdep lock names..

This request is entirely reasonable.

Oct 17 2019, 11:31 PM

Oct 11 2019

mckusick accepted D21897: mountroot: Run statfs after mounting devfs.

I concur that VFS_STATFS should be done here.

Oct 11 2019, 4:30 AM

Oct 4 2019

mckusick committed rS353099: Update ffs_getcg() function to accept a flags parameter to be passed.
Update ffs_getcg() function to accept a flags parameter to be passed
Oct 4 2019, 5:28 AM

Oct 1 2019

mckusick committed rS352947: MFC of 352453.
MFC of 352453
Oct 1 2019, 11:28 PM
mckusick committed rS352946: MFC of 352453.
MFC of 352453
Oct 1 2019, 11:27 PM

Sep 17 2019

mckusick committed rS352453: The VFS-level clustering code collects together sequential blocks.
The VFS-level clustering code collects together sequential blocks
Sep 17 2019, 5:45 PM

Sep 12 2019

mckusick accepted D21593: Add an option to gnop to allow rejecting I/O after allowing some through..

With the minor changes requested, looks good to me.

Sep 12 2019, 9:57 PM

Sep 9 2019

mckusick added a comment to D21560: Collection of fixes for exec/v_writecount handling..

What scenario would ever have v_writecount be less than zero?

Sep 9 2019, 4:12 AM

Aug 28 2019

D21412: Rework v_object lifecycle for vnodes. is now accepted and ready to land.

Removing the function is right approach. All looks good.

Aug 28 2019, 8:00 PM

Aug 27 2019

mckusick added inline comments to D21412: Rework v_object lifecycle for vnodes..
Aug 27 2019, 6:49 PM
mckusick added a comment to D21412: Rework v_object lifecycle for vnodes..

My inline comment seems to have been lost. I requested that ufs_prepare_reclaim() be made static.

Aug 27 2019, 6:47 PM
D21412: Rework v_object lifecycle for vnodes. now requires changes to proceed.

This change looks good to me modulo minor inline comment.

Aug 27 2019, 6:45 PM

Aug 17 2019

mckusick committed rS351164: MFC of 351002.
MFC of 351002
Aug 17 2019, 6:07 AM

Aug 15 2019

mckusick committed rS351098: Add missing comma in calendar.freebsd.
Add missing comma in calendar.freebsd
Aug 15 2019, 6:46 PM

Aug 14 2019

mckusick added a comment to D2844: Bail out of corrupt directory entries during boot.

Any progress on getting this done?

Aug 14 2019, 6:05 AM

Aug 13 2019

mckusick committed rS351002: Clarify comment that describes how the FS_METACKHASH is managed..
Clarify comment that describes how the FS_METACKHASH is managed.
Aug 13 2019, 8:56 PM

Aug 10 2019

mckusick committed rS350852: MFC of 350682.
MFC of 350682
Aug 10 2019, 11:08 PM
mckusick committed rS350851: MFC of 350682.
MFC of 350682
Aug 10 2019, 11:07 PM
mckusick committed rS350850: MFC of 350490.
MFC of 350490
Aug 10 2019, 11:03 PM

Aug 7 2019

mckusick committed rS350682: Correct the location of the first backup superblock in fsck_ffs.8..
Correct the location of the first backup superblock in fsck_ffs.8.
Aug 7 2019, 4:56 PM

Aug 6 2019

mckusick committed rS350653: Ignore UFS/FFS superblock check hash failures so as to allow a higher.
Ignore UFS/FFS superblock check hash failures so as to allow a higher
Aug 6 2019, 6:29 PM
mckusick committed rS350651: A race condition existed between the time a UFS/FFS superblock check.
A race condition existed between the time a UFS/FFS superblock check
Aug 6 2019, 6:10 PM

Jul 31 2019

mckusick committed rS350490: When updating the user or group disk quotas for the return of inodes or.
When updating the user or group disk quotas for the return of inodes or
Jul 31 2019, 10:45 PM
mckusick committed rS350460: MFC of 349589, 350070, 350071, 350096, and 350187.
MFC of 349589, 350070, 350071, 350096, and 350187
Jul 31 2019, 12:16 AM

Jul 30 2019

mckusick committed rS350459: MFC of 349032.
MFC of 349032
Jul 30 2019, 11:50 PM

Jul 26 2019

mckusick added a comment to D21039: Enable witness blessings..

This is long overdue. I would love to have these known LORs ignored.

Jul 26 2019, 12:44 AM

Jul 20 2019

mckusick committed rS350187: Treat any inode with bad content as unknown (i.e., ask if it should.
Treat any inode with bad content as unknown (i.e., ask if it should
Jul 20 2019, 9:39 PM
mckusick committed rS350186: When running with journaled soft updates, some updated inodes were not.
When running with journaled soft updates, some updated inodes were not
Jul 20 2019, 9:20 PM

Jul 18 2019

mckusick added a comment to D2844: Bail out of corrupt directory entries during boot.

I concur that taking evasive action is preferable to looping forever. However, we can do better than just bailing out. The kernel code checks for a bad d_reclen and if found skips up to the beginning of the next block within the directory using this code:

Jul 18 2019, 6:24 PM

Jul 17 2019

mckusick committed rS350096: The error reported in FS-14-UFS-3 can only happen on UFS/FFS.
The error reported in FS-14-UFS-3 can only happen on UFS/FFS
Jul 17 2019, 10:07 PM

Jul 16 2019

mckusick committed rS350071: Style..
Style.
Jul 16 2019, 11:39 PM
mckusick committed rS350070: When a process attempts to allocate space on a full filesystem, a.
When a process attempts to allocate space on a full filesystem, a
Jul 16 2019, 11:12 PM

Jul 3 2019

D18586: mount(8) does not handle "update" option in the fstab(5) correctly is now accepted and ready to land.

Looks good.

Jul 3 2019, 4:15 AM

Jul 1 2019

mckusick committed rS349589: Add a new "untrusted" option to the mount command. Its purpose.
Add a new "untrusted" option to the mount command. Its purpose
Jul 1 2019, 11:23 PM
Herald added a reviewer for D20786: Add option "untrusted" to mount(8): manpages.
Jul 1 2019, 11:23 PM

Jun 29 2019

D20695: Fix VOP_PUTPAGES(9) in regards to the use of VM_PAGER_CLUSTER_OK is now accepted and ready to land.

Looks good to go.

Jun 29 2019, 2:49 AM

Jun 28 2019

mckusick added a comment to D20786: Add option "untrusted" to mount(8).

Some of the checks that I am contemplating are expensive in that they may require extra I/O operations. Another possible thing is that I may spawn a process to run a full filesystem check before proceeding with the mount. These are not things that should or need to be done in the case of a trusted filesystem. I believe that the name "untrusted" well and succinctly describes this option.

Jun 28 2019, 5:52 PM

Jun 27 2019

mckusick created D20786: Add option "untrusted" to mount(8).
Jun 27 2019, 10:40 PM
D20695: Fix VOP_PUTPAGES(9) in regards to the use of VM_PAGER_CLUSTER_OK now requires changes to proceed.

Doing asynchronous writes prevents clustering. The correct fix is to say:

Jun 27 2019, 9:09 PM

Jun 24 2019

D20741: Remove references to splbio(). is now accepted and ready to land.

Simply deleting the comments about splbio is fine. It might be helpful to change the comment to say `Called with the soft update lock held.' In many places the lock ownership is asserted which may be sufficient, or perhaps the assertion should be added where it is not already there.

Jun 24 2019, 11:42 PM

Jun 20 2019

D20705: Add FIOBMAP2 ioctl is now accepted and ready to land.

Changing ufs_bamparray() to return errors rather than panicing is a helpful move forward.

Jun 20 2019, 6:08 AM
D20704: Add a VOP_BMAP(9) man page is now accepted and ready to land.

Looks good to me.

Jun 20 2019, 6:00 AM

Jun 17 2019

mckusick committed rS349155: MFC of 349032.
MFC of 349032
Jun 17 2019, 9:31 PM

Jun 14 2019

mckusick committed rS349032: Clarify that the -r and -R options can be used only to create level 0 dumps..
Clarify that the -r and -R options can be used only to create level 0 dumps.
Jun 14 2019, 6:10 AM

Jun 12 2019

mckusick added a comment to D18351: Add BIO_SPEEDUP.

Has this been resolved / committed?

Jun 12 2019, 8:48 PM
mckusick added a comment to D18586: mount(8) does not handle "update" option in the fstab(5) correctly.

Has this been resolved / committed?

Jun 12 2019, 8:47 PM

Jun 7 2019

mckusick committed rS348792: MFC of 348259.
MFC of 348259
Jun 7 2019, 10:25 PM

Jun 4 2019

mckusick accepted D20518: Add myself to committers-src.dot and calendar.freebsd.
Jun 4 2019, 7:49 PM

May 28 2019

mckusick committed rS348330: Add a missing bresle() in seldom-used error return..
Add a missing bresle() in seldom-used error return.
May 28 2019, 5:31 PM
mckusick committed rS348329: Convert use of UFS-specific #ifdef DEBUG to DIAGNOSTIC or INVARIANTS.
Convert use of UFS-specific #ifdef DEBUG to DIAGNOSTIC or INVARIANTS
May 28 2019, 4:32 PM

May 27 2019

mckusick committed rS348300: Add function name and line number debugging information to softupdates.
Add function name and line number debugging information to softupdates
May 27 2019, 6:23 AM

May 25 2019

mckusick committed rS348260: MFC of 348074.
MFC of 348074
May 25 2019, 12:22 AM
mckusick committed rS348259: When using the destroy option to shut down a nop GEOM module, I/O.
When using the destroy option to shut down a nop GEOM module, I/O
May 25 2019, 12:08 AM

May 24 2019

mckusick committed rS348256: MFC of 348074.
MFC of 348074
May 24 2019, 11:07 PM

May 21 2019

mckusick committed rS348074: This revision began as a simple change to eliminate an uninitialized warning.
This revision began as a simple change to eliminate an uninitialized warning
May 21 2019, 10:25 PM

May 20 2019

mckusick accepted D20304: NDFREE(): Fix unlocking for LOCKPARENT|LOCKLEAF and ndp->ni_dvp == ndp->ni_vp.

This change looks correct to me.

May 20 2019, 9:44 PM

May 10 2019

mckusick committed rS347475: MFC of 347064, 347066, and 347130.
MFC of 347064, 347066, and 347130
May 10 2019, 11:46 PM
mckusick committed rS347474: MFC of 347064, 347066, and 347130.
MFC of 347064, 347066, and 347130
May 10 2019, 11:45 PM

May 6 2019

mckusick committed rS347200: MFC of 346185.
MFC of 346185
May 6 2019, 7:16 PM
mckusick committed rS347199: MFC of 345758.
MFC of 345758
May 6 2019, 7:13 PM

May 4 2019

mckusick committed rS347130: Zero out the file directory entry metadata to reduce disk.
Zero out the file directory entry metadata to reduce disk
May 4 2019, 6:01 PM

May 3 2019

mckusick committed rS347066: This update eliminates a kernel stack disclosure bug in UFS/FFS.
This update eliminates a kernel stack disclosure bug in UFS/FFS
May 3 2019, 9:54 PM
mckusick committed rS347064: Simplify calculation of DIRECTSIZ. No functional change intended..
Simplify calculation of DIRECTSIZ. No functional change intended.
May 3 2019, 9:46 PM

Apr 27 2019

mckusick accepted D20066: Stop checking for TD_IDLETHREAD() in the buffer cache..

A useful cleanup.

Apr 27 2019, 12:49 PM

Apr 15 2019

mckusick committed rS346222: MFC of 345758.
MFC of 345758
Apr 15 2019, 12:10 PM
mckusick committed rS346221: MFC of 345758.
MFC of 345758
Apr 15 2019, 12:08 PM

Apr 13 2019

mckusick committed rS346185: Followup to -r344552 in which fsck_ffs checks for a size past the.
Followup to -r344552 in which fsck_ffs checks for a size past the
Apr 13 2019, 1:31 PM

Apr 9 2019

mckusick added a comment to D19599: Add a UFS-specific implementation for lseek(SEEK_DATA)..

Should this now be closed since it was resolved in D19811?

Apr 9 2019, 7:12 PM

Mar 31 2019

mckusick committed rS345758: When using the force option to shut down a memory-disk device,.
When using the force option to shut down a memory-disk device,
Mar 31 2019, 9:35 PM

Mar 29 2019

mckusick committed rS345667: MFC of 343536, 345077, and 345352.
MFC of 343536, 345077, and 345352
Mar 29 2019, 1:17 AM
mckusick committed rS345666: MFC of 343536, 345077, and 345352.
MFC of 343536, 345077, and 345352
Mar 29 2019, 1:15 AM

Mar 20 2019

mckusick committed rS345352: This is an additional and hopefully final fix for bug report 230962..
This is an additional and hopefully final fix for bug report 230962.
Mar 20 2019, 11:11 PM

Mar 17 2019

mckusick accepted D19599: Add a UFS-specific implementation for lseek(SEEK_DATA)..

This seems like a very useful improvement.

Mar 17 2019, 12:32 AM

Mar 16 2019

mckusick added a comment to D19598: Remove incorrect comments..

I would preface the actual condition "The nbp parameter is non-NULL when the mapping is for a block that contains data, one of an external data block, a direct block, or the final block in a chain of indirect blocks." before the correct clarification "If mapping an extended attribute block, nbp must point to a buffer for that block."

Mar 16 2019, 11:49 PM

Mar 12 2019

mckusick committed rS345077: This is an additional fix for bug report 230962. When using.
This is an additional fix for bug report 230962. When using
Mar 12 2019, 7:09 PM
mckusick committed rS345044: Add KASSERT to the softdep_disk_write_complete() function in the.
Add KASSERT to the softdep_disk_write_complete() function in the
Mar 12 2019, 12:10 AM

Mar 11 2019

mckusick committed rS345043: Give more complete information in INVARIANTS panic messages at end of.
Give more complete information in INVARIANTS panic messages at end of
Mar 11 2019, 11:54 PM
mckusick committed rS345037: Update the main loop in the flushbuflist() routine to properly select.
Update the main loop in the flushbuflist() routine to properly select
Mar 11 2019, 10:42 PM
mckusick committed rS345032: Augment the UFS filesystem specific print function (called by the.
Augment the UFS filesystem specific print function (called by the
Mar 11 2019, 10:05 PM
mckusick committed rS345029: Augment DDB "show buffer" command to print the buffer's referenced.
Augment DDB "show buffer" command to print the buffer's referenced
Mar 11 2019, 9:49 PM

Mar 7 2019

mckusick accepted D19437: explain fsck -f more in detail.

Looks good, thanks for clarifying -f.

Mar 7 2019, 12:15 AM
mckusick committed rS344864: MFC of 344731.
MFC of 344731
Mar 7 2019, 12:04 AM
mckusick committed rS344863: MFC of 344731.
MFC of 344731
Mar 7 2019, 12:02 AM
mckusick committed rS344861: MFC of 344552 and 344732.
MFC of 344552 and 344732
Mar 7 2019, 12:00 AM

Mar 6 2019

mckusick committed rS344860: MFC of 344552 and 344732.
MFC of 344552 and 344732
Mar 6 2019, 11:55 PM

Mar 3 2019

mckusick added a comment to D19437: explain fsck -f more in detail.

Your change looks good. I'll approve once you update the suggested mandoc refinement.

Mar 3 2019, 9:28 PM

Mar 2 2019

mckusick committed rS344732: Properly calculate the last used logical block of a file when checking.
Properly calculate the last used logical block of a file when checking
Mar 2 2019, 9:30 PM
mckusick committed rS344731: The size of the UFS soft-updates journal must be a multiple of the.
The size of the UFS soft-updates journal must be a multiple of the
Mar 2 2019, 9:23 PM

Mar 1 2019

mckusick added a comment to D19325: Do not panic if inode bitmap is corrupted..

I recently added the new error EINTEGRITY which is intended for use when a cylinder group or other filesystem structure has an integrity error. My initial plan is to use it for check-hash failures, but this is another good place to use it. It means that the callers of the functions that return this error need to be prepared to handle it. Specifically when allocating blocks or files trying to allocate from a different cylinder group. When deleting blocks or files, not immediately giving up, but trying to release all the remaining blocks of the file. A similar analysis would need to be used for this error (notably trying to allocate from a different cylinder group).

Mar 1 2019, 1:15 AM
mckusick added a comment to D18586: mount(8) does not handle "update" option in the fstab(5) correctly.

The addition to fstab.5 is helpful, but slightly incomplete in that it is not clear in what context the update keyword would be used.I would add text at the end of your new block that says something like:
The
.Dq update
option is typically used in conjuction with two
.Nm
files.
The first
.Nm
file is used to set up the initial set of file systems.
The second
.Nm
file is then run to update the initial set of file systems and
to add additional file systems.

Mar 1 2019, 12:54 AM