Page MenuHomeFreeBSD

Add an option to gnop to allow rejecting I/O after allowing some through.
ClosedPublic

Authored by chs on Sep 10 2019, 8:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 11 2024, 9:48 PM
Unknown Object (File)
Feb 8 2024, 11:25 AM
Unknown Object (File)
Dec 20 2023, 7:04 AM
Unknown Object (File)
Nov 19 2023, 7:41 AM
Unknown Object (File)
Nov 7 2023, 4:53 PM
Unknown Object (File)
Nov 4 2023, 3:44 AM
Unknown Object (File)
Oct 30 2023, 12:29 PM
Unknown Object (File)
Oct 10 2023, 2:42 AM
Subscribers

Details

Summary

Add a "count_until_fail" option to gnop, which says to start failing
I/O requests after the given number have been allowed though.

Test Plan

I've written a test that uses this feature to exercise FFS I/O error handling.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rpokala added inline comments.
lib/geom/nop/gnop.8
125 ↗(On Diff #61910)

I/O requests

kib added inline comments.
sys/geom/nop/g_nop.c
198 ↗(On Diff #61910)

if (sc->sc_count_until_fail != 0 && ...

This revision is now accepted and ready to land.Sep 11 2019, 5:21 AM
0mp requested changes to this revision.Sep 11 2019, 11:10 AM
0mp added a subscriber: 0mp.
0mp added inline comments.
lib/geom/nop/gnop.8
27 ↗(On Diff #61910)

Remember to bump the date.

123 ↗(On Diff #61910)

.Bl -tag -width "-r count_until_fail" is enough here. Macros are not expanded here anyway (the whole .Fl r Ar count_until_fail string is treated literally).

This revision now requires changes to proceed.Sep 11 2019, 11:10 AM

With the minor changes requested, looks good to me.

Modulo kib's quibble, this is ready to go. No need to repost the review if all you change is the .Dd date and kib's thing.

This revision was not accepted when it landed; it landed in state Needs Revision.Sep 13 2019, 11:04 PM
This revision was automatically updated to reflect the committed changes.