From: Dominik Csapak <d.csapak@proxmox.com>
To: pbs-devel@lists.proxmox.com
Subject: [pbs-devel] [PATCH proxmox-backup 1/3] tape: refactor setting the mode page
Date: Mon, 10 Jun 2024 13:45:03 +0200 [thread overview]
Message-ID: <20240610114505.2211336-1-d.csapak@proxmox.com> (raw)
we'll reuse that code later for a different page/subpage
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
pbs-tape/src/sg_tape.rs | 59 ++++++++++++++++++++++++-----------------
1 file changed, 35 insertions(+), 24 deletions(-)
diff --git a/pbs-tape/src/sg_tape.rs b/pbs-tape/src/sg_tape.rs
index 50f5c079..1aa7d492 100644
--- a/pbs-tape/src/sg_tape.rs
+++ b/pbs-tape/src/sg_tape.rs
@@ -804,35 +804,17 @@ impl SgTape {
Ok(())
}
- /// Set important drive options
- #[allow(clippy::vec_init_then_push)]
- pub fn set_drive_options(
+ // tries to set the given page with either mode sense 6 or 10 depending on
+ // the given header size
+ fn drive_mode_select<T: Endian>(
&mut self,
- compression: Option<bool>,
- block_length: Option<u32>,
- buffer_mode: Option<bool>,
+ head: ModeParameterHeader,
+ block_descriptor: ModeBlockDescriptor,
+ page: T,
) -> Result<(), Error> {
- // Note: Read/Modify/Write
-
- let (mut head, mut block_descriptor, mut page) = self.read_compression_page()?;
-
let mut sg_raw = SgRaw::new(&mut self.file, 0)?;
sg_raw.set_timeout(Self::SCSI_TAPE_DEFAULT_TIMEOUT);
- head.reset_mode_data_len(); // mode_data_len need to be zero
-
- if let Some(compression) = compression {
- page.set_compression(compression);
- }
-
- if let Some(block_length) = block_length {
- block_descriptor.set_block_length(block_length)?;
- }
-
- if let Some(buffer_mode) = buffer_mode {
- head.set_buffer_mode(buffer_mode);
- }
-
match head {
ModeParameterHeader::Long(head) => {
let mut data = Vec::new();
@@ -878,8 +860,37 @@ impl SgTape {
format_err!("set drive options (mode select(6)) failed - {err}")
})?;
}
+ };
+ Ok(())
+ }
+
+ /// Set important drive options
+ #[allow(clippy::vec_init_then_push)]
+ pub fn set_drive_options(
+ &mut self,
+ compression: Option<bool>,
+ block_length: Option<u32>,
+ buffer_mode: Option<bool>,
+ ) -> Result<(), Error> {
+ // Note: Read/Modify/Write
+ let (mut head, mut block_descriptor, mut page) = self.read_compression_page()?;
+
+ head.reset_mode_data_len(); // mode_data_len need to be zero
+
+ if let Some(compression) = compression {
+ page.set_compression(compression);
}
+ if let Some(block_length) = block_length {
+ block_descriptor.set_block_length(block_length)?;
+ }
+
+ if let Some(buffer_mode) = buffer_mode {
+ head.set_buffer_mode(buffer_mode);
+ }
+
+ self.drive_mode_select(head, block_descriptor, page)?;
+
Ok(())
}
--
2.39.2
_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel
next reply other threads:[~2024-06-10 11:44 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-10 11:45 Dominik Csapak [this message]
2024-06-10 11:45 ` [pbs-devel] [PATCH proxmox-backup 2/3] tape: disable Programmable Early Warning Zone (PEWZ) Dominik Csapak
2024-06-10 11:45 ` [pbs-devel] [PATCH proxmox-backup 3/3] tape: handle PEWZ like regular early warning Dominik Csapak
2024-06-11 8:34 ` [pbs-devel] applied: [PATCH proxmox-backup 1/3] tape: refactor setting the mode page Dietmar Maurer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240610114505.2211336-1-d.csapak@proxmox.com \
--to=d.csapak@proxmox.com \
--cc=pbs-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox