Page MenuHomeFreeBSD

gnop: add the option for adding a suffix to the name of the device
ClosedPublic

Authored by oshogbo on Nov 10 2019, 11:48 AM.
Tags
None
Referenced Files
F103127243: D22304.id.diff
Thu, Nov 21, 8:59 AM
Unknown Object (File)
Thu, Nov 14, 1:17 PM
Unknown Object (File)
Oct 20 2024, 1:41 PM
Unknown Object (File)
Oct 5 2024, 4:56 PM
Unknown Object (File)
Oct 5 2024, 4:27 AM
Unknown Object (File)
Oct 4 2024, 11:27 PM
Unknown Object (File)
Sep 27 2024, 5:41 AM
Unknown Object (File)
Sep 24 2024, 11:55 AM
Subscribers

Details

Summary

Thanks to this option we can create more then one gnop provider from
single provider. This may be useful for temporary labeling some data
on the disk.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 27613
Build 25826: arc lint + arc unit

Event Timeline

Why don't we just have a "name" parameter that determines the entire gnop provider name?

Why don't we just have a "name" parameter that determines the entire gnop provider name?

I like Mark's idea, change the flag to just let you specify the entire name of the resulting provider (the parameter passed to g_new_geomf())

I'm fine with naming nop devices any name as long as there is a nop suffix.
Otherwise somebody may do some weird stuff in /dev/.

lib/geom/nop/gnop.8
158

Should be "the new provider".

159

I think it would be clearer to refer to it as a suffix. For example, "The suffix .Dq .nop will be appended to the provider name."

sys/geom/nop/g_nop.c
33

param.h is special and should come first in the include order.

91

Why not write these as boolean expressions? if (!islower(name[i]) && !isdigit(name[i])) ...

I do not see why upper case characters should be prohibited.

392

Perhaps include the name in the error message.

oshogbo marked 3 inline comments as done.

Address @markj comments.

sys/geom/nop/g_nop.c
33

Fixed.

91

style(9) defines so.
I added upper cases.

This revision is now accepted and ready to land.Dec 25 2019, 11:17 AM
This revision was automatically updated to reflect the committed changes.