Skip to content

Fixes issues with Tandberg LTO 6 drive documented in #645 - #646

Open
simonwaid wants to merge 16 commits into
LinearTapeFileSystem:release/v2.4.9.1from
simonwaid:fix_tandberg_lto6
Open

Fixes issues with Tandberg LTO 6 drive documented in #645#646
simonwaid wants to merge 16 commits into
LinearTapeFileSystem:release/v2.4.9.1from
simonwaid:fix_tandberg_lto6

Conversation

@simonwaid

@simonwaid simonwaid commented Sep 11, 2026 •
edited
Loading

Copy link
Copy Markdown

Summary of changes

This pull request addresses issue #645

Description

The detailed changes are outlined in issue #645

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have confirmed my fix is effective or that my feature works

XV02 and others added 15 commits July 27, 2026 13:34
@XV02
Docs: MD updates (LinearTapeFileSystem#584)
31830b4
@asivery
fix: fixes and improvements to the ltfs_ordered_copy tool (LinearTape…
d8595bc 
…FileSystem#586)
@hugo-hur
fix: resolve ltfs_ordered_copy issue when processing a single file. (L…
ba1695e 
…inearTapeFileSystem#599)
@mhorimoto
feat: introduce support for TANDBERG LTO-6 HH drive. (LinearTapeFileS…
aa05125 
…ystem#605)
@matejk
fix: prevent data loss on writes larger than the tape block size. (Li…
135edb1 
…nearTapeFileSystem#607)
@matejk
fix: resolve build and CLI issues. (LinearTapeFileSystem#608)
f5c51b1
@rangelmx
fix: correct inverted status checks in the SCSI tape backend. (Linear…
56baaf9 
…TapeFileSystem#627)

Authored-by: Matej Kenda <matejken@gmail.com>
@hugo-hur
configure: detect ICU via icu-uc/icu-i18n pkg-config modules to easil…
42f5884 
…y build on Debian (LinearTapeFileSystem#597)
@Rolaxgar97 Rolando Garcia Arreguin @vandelvan
fix: Drive Sense Key error (LinearTapeFileSystem#625)
413c932 
Co-authored-by: Rolando Garcia Arreguin <Rolando.Garcia.Arreguin@ibm.com>
Co-authored-by: Ivan Orozco <vandel_van@hotmail.com>
@vandelvan
update: release version from 2.4.8.4 to 2.4.9.0 (LinearTapeFileSystem…
f60d45b 
…#632)
@madjesc
fix: rhel10 netsnmp faulty flags (LinearTapeFileSystem#633)
30fad32
@vandelvan
fix: HP LTO 6 remaining capacity (LinearTapeFileSystem#634)
2a13b53
@vandelvan
fix: remove vendor condition from lintape (LinearTapeFileSystem#642)
5e84c19
Simon Waid
Fixes issues with Tandberg LTO 6 drive documented in LinearTapeFileSy…
dea5496 
…stem#645
Simon Waid
Add missing parenthesis
52d1318
@simonwaid

simonwaid commented Sep 14, 2026

Copy link
Copy Markdown
Author

Building on Debian 11 seems to fail due to apt not being able to install packages, not due to the change in code.

@Piloalucard

Piloalucard commented Sep 15, 2026

Copy link
Copy Markdown
Member

Hello, thanks for this contribution!
For me, it looks good, but I am curious where did you get missing code on Sense key 8?

@simonwaid

simonwaid commented Sep 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Author

The sense key is defined by IBM here: https://www.ibm.com/docs/en/ts2900tapeautoloader?topic=STCAPQJ%2Fref_3572_appD_drv_sense.htm

@Piloalucard

Piloalucard commented Sep 23, 2026

Copy link
Copy Markdown
Member

Hmm that is interesting. Documentation does not explicitly say is on the Sense Key 8, however your log demonstrates it, I am just taking a note of this, but for me this looks good.

Piloalucard
Piloalucard reviewed Sep 23, 2026
View reviewed changes
Comment thread src/tape_drivers/hp_tape.c
@vandelvan
vandelvan changed the base branch from release/v2.4.9.0 to release/v2.4.9.1 September 28, 2026 18:01
Rolaxgar97
Rolaxgar97 reviewed Sep 29, 2026
View reviewed changes
Comment thread src/libltfs/ltfs_thread.h
Rolaxgar97
Rolaxgar97 reviewed Sep 29, 2026
View reviewed changes

@Rolaxgar97 Rolaxgar97 left a comment •
edited
Loading

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your effort. Generally, it looks good to me. I just have one comment/question. @simonwaid

@Rolaxgar97
Rolaxgar97 self-requested a review September 29, 2026 21:59
Rolaxgar97
Rolaxgar97 approved these changes Sep 29, 2026
View reviewed changes

@Rolaxgar97 Rolaxgar97 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Rolaxgar97 Rolaxgar97 Rolaxgar97 approved these changes
@Piloalucard Piloalucard Piloalucard approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.