Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 1 | // Copyright 2022, The Android Open Source Project |
| 2 | // |
| 3 | // Licensed under the Apache License, Version 2.0 (the "License"); |
| 4 | // you may not use this file except in compliance with the License. |
| 5 | // You may obtain a copy of the License at |
| 6 | // |
| 7 | // http://www.apache.org/licenses/LICENSE-2.0 |
| 8 | // |
| 9 | // Unless required by applicable law or agreed to in writing, software |
| 10 | // distributed under the License is distributed on an "AS IS" BASIS, |
| 11 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 12 | // See the License for the specific language governing permissions and |
| 13 | // limitations under the License. |
| 14 | |
| 15 | //! Support for the pvmfw configuration data format. |
| 16 | |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 17 | use core::fmt; |
| 18 | use core::mem; |
Jaewan Kim | 3dfc0ab | 2023-09-27 14:00:59 +0900 | [diff] [blame] | 19 | use core::num::NonZeroUsize; |
Pierre-Clément Tosi | 71d7d5b | 2022-12-12 13:15:45 +0000 | [diff] [blame] | 20 | use core::ops::Range; |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 21 | use core::result; |
Pierre-Clément Tosi | ce3e313 | 2023-08-17 10:24:22 +0100 | [diff] [blame] | 22 | use log::{info, warn}; |
Pierre-Clément Tosi | fd694c7 | 2023-08-14 14:24:07 +0100 | [diff] [blame] | 23 | use static_assertions::const_assert_eq; |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 24 | use vmbase::util::RangeExt; |
Alan Stokes | 6561833 | 2023-12-15 14:09:25 +0000 | [diff] [blame] | 25 | use zerocopy::{FromBytes, FromZeroes}; |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 26 | |
Pierre-Clément Tosi | 43592a4 | 2022-12-19 14:17:38 +0000 | [diff] [blame] | 27 | /// Configuration data header. |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 28 | #[repr(C, packed)] |
Frederick Mayle | 2e77994 | 2023-10-15 18:27:31 +0000 | [diff] [blame] | 29 | #[derive(Clone, Copy, Debug, FromZeroes, FromBytes)] |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 30 | struct Header { |
Pierre-Clément Tosi | 43592a4 | 2022-12-19 14:17:38 +0000 | [diff] [blame] | 31 | /// Magic number; must be `Header::MAGIC`. |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 32 | magic: u32, |
Pierre-Clément Tosi | 43592a4 | 2022-12-19 14:17:38 +0000 | [diff] [blame] | 33 | /// Version of the header format. |
Jaewan Kim | 6b7fb3c | 2023-08-08 18:00:43 +0900 | [diff] [blame] | 34 | version: Version, |
Pierre-Clément Tosi | 43592a4 | 2022-12-19 14:17:38 +0000 | [diff] [blame] | 35 | /// Total size of the configuration data. |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 36 | total_size: u32, |
Pierre-Clément Tosi | 43592a4 | 2022-12-19 14:17:38 +0000 | [diff] [blame] | 37 | /// Feature flags; currently reserved and must be zero. |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 38 | flags: u32, |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 39 | } |
| 40 | |
| 41 | #[derive(Debug)] |
| 42 | pub enum Error { |
| 43 | /// Reserved region can't fit configuration header. |
| 44 | BufferTooSmall, |
Alan Stokes | a0e4296 | 2023-04-14 17:59:50 +0100 | [diff] [blame] | 45 | /// Header has the wrong alignment |
| 46 | HeaderMisaligned, |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 47 | /// Header doesn't contain the expect magic value. |
| 48 | InvalidMagic, |
| 49 | /// Version of the header isn't supported. |
Jaewan Kim | 6b7fb3c | 2023-08-08 18:00:43 +0900 | [diff] [blame] | 50 | UnsupportedVersion(Version), |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 51 | /// Header describes configuration data that doesn't fit in the expected buffer. |
| 52 | InvalidSize(usize), |
Pierre-Clément Tosi | 292e099 | 2022-12-12 13:01:27 +0000 | [diff] [blame] | 53 | /// Header entry is missing. |
| 54 | MissingEntry(Entry), |
Pierre-Clément Tosi | fd694c7 | 2023-08-14 14:24:07 +0100 | [diff] [blame] | 55 | /// Range described by entry does not fit within config data. |
| 56 | EntryOutOfBounds(Entry, Range<usize>, Range<usize>), |
Jaewan Kim | b266e0e | 2023-09-27 15:44:53 +0900 | [diff] [blame] | 57 | /// Entries are in out of order |
| 58 | EntryOutOfOrder, |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 59 | } |
| 60 | |
| 61 | impl fmt::Display for Error { |
| 62 | fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result { |
| 63 | match self { |
| 64 | Self::BufferTooSmall => write!(f, "Reserved region is smaller than config header"), |
Alan Stokes | a0e4296 | 2023-04-14 17:59:50 +0100 | [diff] [blame] | 65 | Self::HeaderMisaligned => write!(f, "Reserved region is misaligned"), |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 66 | Self::InvalidMagic => write!(f, "Wrong magic number"), |
Jaewan Kim | 6b7fb3c | 2023-08-08 18:00:43 +0900 | [diff] [blame] | 67 | Self::UnsupportedVersion(v) => write!(f, "Version {v} not supported"), |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 68 | Self::InvalidSize(sz) => write!(f, "Total size ({sz:#x}) overflows reserved region"), |
Pierre-Clément Tosi | 292e099 | 2022-12-12 13:01:27 +0000 | [diff] [blame] | 69 | Self::MissingEntry(entry) => write!(f, "Mandatory {entry:?} entry is missing"), |
Pierre-Clément Tosi | fd694c7 | 2023-08-14 14:24:07 +0100 | [diff] [blame] | 70 | Self::EntryOutOfBounds(entry, range, limits) => { |
| 71 | write!( |
| 72 | f, |
| 73 | "Entry {entry:?} out of bounds: {range:#x?} must be within range {limits:#x?}" |
| 74 | ) |
| 75 | } |
Jaewan Kim | b266e0e | 2023-09-27 15:44:53 +0900 | [diff] [blame] | 76 | Self::EntryOutOfOrder => write!(f, "Entries are out of order"), |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 77 | } |
| 78 | } |
| 79 | } |
| 80 | |
| 81 | pub type Result<T> = result::Result<T, Error>; |
| 82 | |
| 83 | impl Header { |
| 84 | const MAGIC: u32 = u32::from_ne_bytes(*b"pvmf"); |
Jaewan Kim | 6b7fb3c | 2023-08-08 18:00:43 +0900 | [diff] [blame] | 85 | const VERSION_1_0: Version = Version { major: 1, minor: 0 }; |
Jaewan Kim | bf5978c | 2023-07-28 11:30:54 +0000 | [diff] [blame] | 86 | const VERSION_1_1: Version = Version { major: 1, minor: 1 }; |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 87 | |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 88 | pub fn total_size(&self) -> usize { |
| 89 | self.total_size as usize |
| 90 | } |
| 91 | |
Pierre-Clément Tosi | ce3e313 | 2023-08-17 10:24:22 +0100 | [diff] [blame] | 92 | pub fn body_lowest_bound(&self) -> Result<usize> { |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 93 | let entries_offset = mem::size_of::<Self>(); |
| 94 | |
| 95 | // Ensure that the entries are properly aligned and do not require padding. |
| 96 | const_assert_eq!(mem::align_of::<Header>() % mem::align_of::<HeaderEntry>(), 0); |
| 97 | const_assert_eq!(mem::size_of::<Header>() % mem::align_of::<HeaderEntry>(), 0); |
| 98 | |
| 99 | let entries_size = self.entry_count()?.checked_mul(mem::size_of::<HeaderEntry>()).unwrap(); |
| 100 | |
| 101 | Ok(entries_offset.checked_add(entries_size).unwrap()) |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 102 | } |
| 103 | |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 104 | pub fn entry_count(&self) -> Result<usize> { |
| 105 | let last_entry = match self.version { |
| 106 | Self::VERSION_1_0 => Entry::DebugPolicy, |
Jaewan Kim | bf5978c | 2023-07-28 11:30:54 +0000 | [diff] [blame] | 107 | Self::VERSION_1_1 => Entry::VmDtbo, |
Pierre-Clément Tosi | ce3e313 | 2023-08-17 10:24:22 +0100 | [diff] [blame] | 108 | v @ Version { major: 1, .. } => { |
| 109 | const LATEST: Version = Header::VERSION_1_1; |
| 110 | warn!("Parsing unknown config data version {v} as version {LATEST}"); |
| 111 | return Ok(Entry::COUNT); |
| 112 | } |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 113 | v => return Err(Error::UnsupportedVersion(v)), |
Pierre-Clément Tosi | fd694c7 | 2023-08-14 14:24:07 +0100 | [diff] [blame] | 114 | }; |
Pierre-Clément Tosi | 71d7d5b | 2022-12-12 13:15:45 +0000 | [diff] [blame] | 115 | |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 116 | Ok(last_entry as usize + 1) |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 117 | } |
| 118 | } |
| 119 | |
| 120 | #[derive(Clone, Copy, Debug)] |
| 121 | pub enum Entry { |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 122 | Bcc, |
| 123 | DebugPolicy, |
Jaewan Kim | bf5978c | 2023-07-28 11:30:54 +0000 | [diff] [blame] | 124 | VmDtbo, |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 125 | #[allow(non_camel_case_types)] // TODO: Use mem::variant_count once stable. |
| 126 | _VARIANT_COUNT, |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 127 | } |
| 128 | |
| 129 | impl Entry { |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 130 | const COUNT: usize = Self::_VARIANT_COUNT as usize; |
Alan Stokes | 6561833 | 2023-12-15 14:09:25 +0000 | [diff] [blame] | 131 | |
| 132 | const ALL_ENTRIES: [Entry; Self::COUNT] = [Self::Bcc, Self::DebugPolicy, Self::VmDtbo]; |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 133 | } |
| 134 | |
Alan Stokes | d0cf3cd | 2023-12-12 14:36:37 +0000 | [diff] [blame] | 135 | #[derive(Default)] |
| 136 | pub struct Entries<'a> { |
| 137 | pub bcc: &'a mut [u8], |
Alan Stokes | 6561833 | 2023-12-15 14:09:25 +0000 | [diff] [blame] | 138 | pub debug_policy: Option<&'a [u8]>, |
Alan Stokes | d0cf3cd | 2023-12-12 14:36:37 +0000 | [diff] [blame] | 139 | pub vm_dtbo: Option<&'a mut [u8]>, |
| 140 | } |
| 141 | |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 142 | #[repr(packed)] |
Frederick Mayle | 2e77994 | 2023-10-15 18:27:31 +0000 | [diff] [blame] | 143 | #[derive(Clone, Copy, Debug, FromZeroes, FromBytes)] |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 144 | struct HeaderEntry { |
| 145 | offset: u32, |
| 146 | size: u32, |
| 147 | } |
| 148 | |
Jaewan Kim | 6b7fb3c | 2023-08-08 18:00:43 +0900 | [diff] [blame] | 149 | #[repr(C, packed)] |
Frederick Mayle | 2e77994 | 2023-10-15 18:27:31 +0000 | [diff] [blame] | 150 | #[derive(Clone, Copy, Debug, Eq, FromZeroes, FromBytes, PartialEq)] |
Jaewan Kim | 6b7fb3c | 2023-08-08 18:00:43 +0900 | [diff] [blame] | 151 | pub struct Version { |
| 152 | minor: u16, |
| 153 | major: u16, |
| 154 | } |
| 155 | |
| 156 | impl fmt::Display for Version { |
| 157 | fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result { |
| 158 | // Copy the fields to local variables to prevent unaligned access. |
| 159 | let (major, minor) = (self.major, self.minor); |
| 160 | write!(f, "{}.{}", major, minor) |
| 161 | } |
| 162 | } |
| 163 | |
Jaewan Kim | 3dfc0ab | 2023-09-27 14:00:59 +0900 | [diff] [blame] | 164 | /// Range with non-empty length. |
| 165 | #[derive(Debug, Copy, Clone)] |
| 166 | struct NonEmptyRange { |
| 167 | start: usize, |
| 168 | size: NonZeroUsize, |
| 169 | } |
| 170 | |
| 171 | impl NonEmptyRange { |
| 172 | pub fn new(start: usize, size: usize) -> Option<Self> { |
| 173 | // Ensure end() is safe. |
| 174 | start.checked_add(size).unwrap(); |
| 175 | |
| 176 | Some(Self { start, size: NonZeroUsize::new(size)? }) |
| 177 | } |
| 178 | |
| 179 | fn end(&self) -> usize { |
| 180 | self.start + self.len() |
| 181 | } |
| 182 | |
| 183 | fn len(&self) -> usize { |
| 184 | self.size.into() |
| 185 | } |
| 186 | |
| 187 | fn as_range(&self) -> Range<usize> { |
| 188 | self.start..self.end() |
| 189 | } |
| 190 | } |
| 191 | |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 192 | #[derive(Debug)] |
| 193 | pub struct Config<'a> { |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 194 | body: &'a mut [u8], |
Jaewan Kim | 3dfc0ab | 2023-09-27 14:00:59 +0900 | [diff] [blame] | 195 | ranges: [Option<NonEmptyRange>; Entry::COUNT], |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 196 | } |
| 197 | |
| 198 | impl<'a> Config<'a> { |
| 199 | /// Take ownership of a pvmfw configuration consisting of its header and following entries. |
Pierre-Clément Tosi | fd694c7 | 2023-08-14 14:24:07 +0100 | [diff] [blame] | 200 | pub fn new(bytes: &'a mut [u8]) -> Result<Self> { |
| 201 | const HEADER_SIZE: usize = mem::size_of::<Header>(); |
| 202 | if bytes.len() < HEADER_SIZE { |
| 203 | return Err(Error::BufferTooSmall); |
| 204 | } |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 205 | |
Pierre-Clément Tosi | fd694c7 | 2023-08-14 14:24:07 +0100 | [diff] [blame] | 206 | let (header, rest) = |
Alan Stokes | 6561833 | 2023-12-15 14:09:25 +0000 | [diff] [blame] | 207 | zerocopy::Ref::<_, Header>::new_from_prefix(bytes).ok_or(Error::HeaderMisaligned)?; |
Alan Stokes | a0e4296 | 2023-04-14 17:59:50 +0100 | [diff] [blame] | 208 | let header = header.into_ref(); |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 209 | |
| 210 | if header.magic != Header::MAGIC { |
| 211 | return Err(Error::InvalidMagic); |
| 212 | } |
| 213 | |
Pierre-Clément Tosi | ce3e313 | 2023-08-17 10:24:22 +0100 | [diff] [blame] | 214 | let header_flags = header.flags; |
| 215 | if header_flags != 0 { |
| 216 | warn!("Ignoring unknown config flags: {header_flags:#x}"); |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 217 | } |
| 218 | |
Jaewan Kim | bf5978c | 2023-07-28 11:30:54 +0000 | [diff] [blame] | 219 | info!("pvmfw config version: {}", header.version); |
| 220 | |
Pierre-Clément Tosi | fd694c7 | 2023-08-14 14:24:07 +0100 | [diff] [blame] | 221 | // Validate that we won't get an invalid alignment in the following due to padding to u64. |
| 222 | const_assert_eq!(HEADER_SIZE % mem::size_of::<u64>(), 0); |
| 223 | |
| 224 | // Ensure that Header::total_size isn't larger than anticipated by the caller and resize |
| 225 | // the &[u8] to catch OOB accesses to entries/blobs. |
| 226 | let total_size = header.total_size(); |
| 227 | let rest = if let Some(rest_size) = total_size.checked_sub(HEADER_SIZE) { |
| 228 | rest.get_mut(..rest_size).ok_or(Error::InvalidSize(total_size))? |
| 229 | } else { |
| 230 | return Err(Error::InvalidSize(total_size)); |
| 231 | }; |
| 232 | |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 233 | let (header_entries, body) = |
Alan Stokes | 6561833 | 2023-12-15 14:09:25 +0000 | [diff] [blame] | 234 | zerocopy::Ref::<_, [HeaderEntry]>::new_slice_from_prefix(rest, header.entry_count()?) |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 235 | .ok_or(Error::BufferTooSmall)?; |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 236 | |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 237 | // Validate that we won't get an invalid alignment in the following due to padding to u64. |
| 238 | const_assert_eq!(mem::size_of::<HeaderEntry>() % mem::size_of::<u64>(), 0); |
| 239 | |
Jaewan Kim | 3dfc0ab | 2023-09-27 14:00:59 +0900 | [diff] [blame] | 240 | // Ensure entries are in the body. |
Pierre-Clément Tosi | ce3e313 | 2023-08-17 10:24:22 +0100 | [diff] [blame] | 241 | let limits = header.body_lowest_bound()?..total_size; |
Jaewan Kim | 3dfc0ab | 2023-09-27 14:00:59 +0900 | [diff] [blame] | 242 | let mut ranges: [Option<NonEmptyRange>; Entry::COUNT] = [None; Entry::COUNT]; |
Jaewan Kim | b266e0e | 2023-09-27 15:44:53 +0900 | [diff] [blame] | 243 | let mut last_end = 0; |
Alan Stokes | 6561833 | 2023-12-15 14:09:25 +0000 | [diff] [blame] | 244 | for entry in Entry::ALL_ENTRIES { |
Jaewan Kim | 3dfc0ab | 2023-09-27 14:00:59 +0900 | [diff] [blame] | 245 | let Some(header_entry) = header_entries.get(entry as usize) else { continue }; |
| 246 | let entry_offset = header_entry.offset.try_into().unwrap(); |
| 247 | let entry_size = header_entry.size.try_into().unwrap(); |
| 248 | let Some(range) = NonEmptyRange::new(entry_offset, entry_size) else { continue }; |
| 249 | let range = range.as_range(); |
| 250 | if !range.is_within(&limits) { |
| 251 | return Err(Error::EntryOutOfBounds(entry, range, limits)); |
| 252 | } |
| 253 | |
Jaewan Kim | b266e0e | 2023-09-27 15:44:53 +0900 | [diff] [blame] | 254 | if last_end > range.start { |
| 255 | return Err(Error::EntryOutOfOrder); |
| 256 | } |
| 257 | last_end = range.end; |
| 258 | |
Jaewan Kim | 3dfc0ab | 2023-09-27 14:00:59 +0900 | [diff] [blame] | 259 | ranges[entry as usize] = NonEmptyRange::new( |
| 260 | entry_offset - limits.start, // is_within() validates safety of this. |
| 261 | entry_size, |
| 262 | ); |
| 263 | } |
Jaewan Kim | b266e0e | 2023-09-27 15:44:53 +0900 | [diff] [blame] | 264 | // Ensures that BCC exists. |
| 265 | ranges[Entry::Bcc as usize].ok_or(Error::MissingEntry(Entry::Bcc))?; |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 266 | |
| 267 | Ok(Self { body, ranges }) |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 268 | } |
| 269 | |
Alan Stokes | 6561833 | 2023-12-15 14:09:25 +0000 | [diff] [blame] | 270 | /// Locate the various config entries. |
| 271 | pub fn get_entries(self) -> Entries<'a> { |
| 272 | // We require the blobs to be in the same order as the `Entry` enum (and this is checked |
| 273 | // in `new` above) |
| 274 | // So we can just work through the body range and split off the parts we are interested in. |
| 275 | let mut offset = 0; |
| 276 | let mut body = self.body; |
| 277 | |
| 278 | let mut entries: [Option<&mut [u8]>; Entry::COUNT] = Default::default(); |
| 279 | for (i, range) in self.ranges.iter().enumerate() { |
| 280 | if let Some(range) = range { |
| 281 | body = &mut body[range.start - offset..]; |
| 282 | let (chunk, rest) = body.split_at_mut(range.len()); |
| 283 | offset = range.end(); |
| 284 | body = rest; |
| 285 | entries[i] = Some(chunk); |
| 286 | } |
Jaewan Kim | bf5978c | 2023-07-28 11:30:54 +0000 | [diff] [blame] | 287 | } |
Alan Stokes | 6561833 | 2023-12-15 14:09:25 +0000 | [diff] [blame] | 288 | let [bcc, debug_policy, vm_dtbo] = entries; |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 289 | |
Alan Stokes | 6561833 | 2023-12-15 14:09:25 +0000 | [diff] [blame] | 290 | // The platform BCC has always been required. |
| 291 | let bcc = bcc.unwrap(); |
| 292 | |
| 293 | // We have no reason to mutate so drop the `mut`. |
| 294 | let debug_policy = debug_policy.map(|x| &*x); |
Alan Stokes | d0cf3cd | 2023-12-12 14:36:37 +0000 | [diff] [blame] | 295 | Entries { bcc, debug_policy, vm_dtbo } |
Pierre-Clément Tosi | 3300149 | 2023-08-14 17:57:30 +0100 | [diff] [blame] | 296 | } |
Pierre-Clément Tosi | 20b6096 | 2022-10-17 13:35:27 +0100 | [diff] [blame] | 297 | } |