[pbs-devel] applied: [PATCH proxmox-backup] lto/sg_tape/encryption: remove non lto-4 supported byte

Dietmar Maurer dietmar at proxmox.com
Mon May 31 12:33:32 CEST 2021


applied

> On 05/31/2021 10:29 AM Dominik Csapak <d.csapak at proxmox.com> wrote:
> 
>  
> from the SspDataEncryptionCapabilityPage
> 
> it seems we do not need it, since the EXTDECC flag is only used for
> determining if the drive is capable to be configured via
> ADI (Automation/Drive Interface) which we do not use at all.
> 
> this makes the call work with LTO-4 again
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  src/tape/drive/lto/sg_tape/encryption.rs | 15 ++-------------
>  1 file changed, 2 insertions(+), 13 deletions(-)
> 
> diff --git a/src/tape/drive/lto/sg_tape/encryption.rs b/src/tape/drive/lto/sg_tape/encryption.rs
> index 7b687f2c..5ce571e4 100644
> --- a/src/tape/drive/lto/sg_tape/encryption.rs
> +++ b/src/tape/drive/lto/sg_tape/encryption.rs
> @@ -195,8 +195,7 @@ struct DataEncryptionStatus {
>  struct SspDataEncryptionCapabilityPage {
>      page_code: u16,
>      page_len: u16,
> -    extdecc_cfgp_byte: u8,
> -    reserved: [u8; 15],
> +    reserved: [u8; 16],
>  }
>  
>  #[derive(Endian)]
> @@ -222,17 +221,7 @@ fn decode_spin_data_encryption_caps(data: &[u8]) -> Result<u8, Error> {
>  
>      proxmox::try_block!({
>          let mut reader = &data[..];
> -        let page: SspDataEncryptionCapabilityPage = unsafe { reader.read_be_value()? };
> -
> -        let extdecc = (page.extdecc_cfgp_byte & 0b00001100) >> 2;
> -        if extdecc != 2 {
> -            bail!("not external data encryption control capable");
> -        }
> -
> -        let cfg_p = page.extdecc_cfgp_byte & 0b00000011;
> -        if cfg_p != 1 {
> -            bail!("not allow to change logical block encryption parameters");
> -        }
> +        let _page: SspDataEncryptionCapabilityPage = unsafe { reader.read_be_value()? };
>  
>          let mut aes_cgm_index = None;
>  
> -- 
> 2.20.1
> 
> 
> 
> _______________________________________________
> pbs-devel mailing list
> pbs-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel





More information about the pbs-devel mailing list