Page MenuHomeFreeBSD

New port: sysutils/ltfs: Reference implementation of LTFS for stand alone tape drive
ClosedPublic

Authored by kbowling on Jan 14 2019, 7:04 PM.

Details

Summary

Add a new port for the FUSE based LTFS reference implementation

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

kbowling created this revision.Jan 14 2019, 7:04 PM
ken accepted this revision.Jan 14 2019, 8:06 PM

Looks good. I assume you've made sure this installs correctly. I put a couple of suggestions in the description file, although the comment about MAXPHYS might also go into something that the user will see when he installs the package.

I think most folks won't think about the blocksize, but it will affect whether they can read tapes from other systems.

sysutils/ltfs/files/patch-src_tape__drivers_freebsd_cam_Makefile.am
12

Good catch. I have modified our local source to use DESTDIR there, but forgot to upstream the fix.

sysutils/ltfs/pkg-descr
4

I would add something like this:

This only works with IBM-branded LTO-5 or newer tape drives, although there is a file backend for simulation purposes.

You will only be able to read and write tapes with a blocksize <= MAXPHYS, which is 128KB by default. Put the following in your kernel config file and recompile/reinstall to allow reading and writing tapes up to the maximum LTFS blocksize of 1MB:

options MAXPHYS=(1024*1056)

This revision is now accepted and ready to land.Jan 14 2019, 8:06 PM
timur accepted this revision.Jan 14 2019, 8:18 PM

LGTM

rpokala added inline comments.
sysutils/ltfs/pkg-descr
2

s/user/users/

4

@ken: why (1024*1056) rather than (1024*1024)?

linimon retitled this revision from sysutils/ltfs: New port to New port: sysutils/ltfs: Reference implementation of LTFS for stand alone tape drive.Jan 15 2019, 4:21 PM
ken added inline comments.Jan 15 2019, 4:24 PM
sysutils/ltfs/pkg-descr
4

Just in case the buffer is misaligned. If that happens, you'll still be able to get the full 1MB I/O down to the drive if MAXPHYS is slightly larger than 1MB.

This revision was automatically updated to reflect the committed changes.