Wolfgang Denk | d79de1d | 2013-07-08 09:37:19 +0200 | [diff] [blame] | 1 | # SPDX-License-Identifier: GPL-2.0+ |
Tom Rini | 10e4779 | 2018-05-06 17:58:06 -0400 | [diff] [blame] | 2 | # Copyright (c) 2013, Google Inc. |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 3 | # |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 4 | # Sanity check of the FIT handling in U-Boot |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 5 | |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 6 | import os |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 7 | import pytest |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 8 | import struct |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 9 | import u_boot_utils as util |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 10 | |
| 11 | # Define a base ITS which we can adjust using % and a dictionary |
| 12 | base_its = ''' |
| 13 | /dts-v1/; |
| 14 | |
| 15 | / { |
| 16 | description = "Chrome OS kernel image with one or more FDT blobs"; |
| 17 | #address-cells = <1>; |
| 18 | |
| 19 | images { |
Simon Glass | 4433890 | 2021-02-15 17:08:06 -0700 | [diff] [blame] | 20 | kernel-1 { |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 21 | data = /incbin/("%(kernel)s"); |
| 22 | type = "kernel"; |
| 23 | arch = "sandbox"; |
| 24 | os = "linux"; |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 25 | compression = "%(compression)s"; |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 26 | load = <0x40000>; |
| 27 | entry = <0x8>; |
| 28 | }; |
Simon Glass | 4433890 | 2021-02-15 17:08:06 -0700 | [diff] [blame] | 29 | kernel-2 { |
Karl Apsite | aad8dd7 | 2015-05-21 09:52:50 -0400 | [diff] [blame] | 30 | data = /incbin/("%(loadables1)s"); |
| 31 | type = "kernel"; |
| 32 | arch = "sandbox"; |
| 33 | os = "linux"; |
| 34 | compression = "none"; |
| 35 | %(loadables1_load)s |
| 36 | entry = <0x0>; |
| 37 | }; |
Simon Glass | 4433890 | 2021-02-15 17:08:06 -0700 | [diff] [blame] | 38 | fdt-1 { |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 39 | description = "snow"; |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 40 | data = /incbin/("%(fdt)s"); |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 41 | type = "flat_dt"; |
| 42 | arch = "sandbox"; |
| 43 | %(fdt_load)s |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 44 | compression = "%(compression)s"; |
Simon Glass | 4433890 | 2021-02-15 17:08:06 -0700 | [diff] [blame] | 45 | signature-1 { |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 46 | algo = "sha1,rsa2048"; |
| 47 | key-name-hint = "dev"; |
| 48 | }; |
| 49 | }; |
Simon Glass | 4433890 | 2021-02-15 17:08:06 -0700 | [diff] [blame] | 50 | ramdisk-1 { |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 51 | description = "snow"; |
| 52 | data = /incbin/("%(ramdisk)s"); |
| 53 | type = "ramdisk"; |
| 54 | arch = "sandbox"; |
| 55 | os = "linux"; |
| 56 | %(ramdisk_load)s |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 57 | compression = "%(compression)s"; |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 58 | }; |
Simon Glass | 4433890 | 2021-02-15 17:08:06 -0700 | [diff] [blame] | 59 | ramdisk-2 { |
Karl Apsite | aad8dd7 | 2015-05-21 09:52:50 -0400 | [diff] [blame] | 60 | description = "snow"; |
| 61 | data = /incbin/("%(loadables2)s"); |
| 62 | type = "ramdisk"; |
| 63 | arch = "sandbox"; |
| 64 | os = "linux"; |
| 65 | %(loadables2_load)s |
| 66 | compression = "none"; |
| 67 | }; |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 68 | }; |
| 69 | configurations { |
Simon Glass | 4433890 | 2021-02-15 17:08:06 -0700 | [diff] [blame] | 70 | default = "conf-1"; |
| 71 | conf-1 { |
| 72 | kernel = "kernel-1"; |
| 73 | fdt = "fdt-1"; |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 74 | %(ramdisk_config)s |
Karl Apsite | aad8dd7 | 2015-05-21 09:52:50 -0400 | [diff] [blame] | 75 | %(loadables_config)s |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 76 | }; |
| 77 | }; |
| 78 | }; |
| 79 | ''' |
| 80 | |
| 81 | # Define a base FDT - currently we don't use anything in this |
| 82 | base_fdt = ''' |
| 83 | /dts-v1/; |
| 84 | |
| 85 | / { |
Heinrich Schuchardt | dca1056 | 2019-12-18 11:05:59 +0100 | [diff] [blame] | 86 | #address-cells = <1>; |
| 87 | #size-cells = <0>; |
| 88 | |
| 89 | model = "Sandbox Verified Boot Test"; |
| 90 | compatible = "sandbox"; |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 91 | |
Philippe Reynes | 462d163 | 2022-03-28 22:56:53 +0200 | [diff] [blame] | 92 | binman { |
| 93 | }; |
| 94 | |
Simon Glass | 546bab1 | 2016-02-24 09:14:44 -0700 | [diff] [blame] | 95 | reset@0 { |
| 96 | compatible = "sandbox,reset"; |
Heinrich Schuchardt | dca1056 | 2019-12-18 11:05:59 +0100 | [diff] [blame] | 97 | reg = <0>; |
Simon Glass | 546bab1 | 2016-02-24 09:14:44 -0700 | [diff] [blame] | 98 | }; |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 99 | }; |
| 100 | ''' |
| 101 | |
Robert P. J. Day | 8445f73 | 2017-03-13 06:50:55 -0400 | [diff] [blame] | 102 | # This is the U-Boot script that is run for each test. First load the FIT, |
| 103 | # then run the 'bootm' command, then save out memory from the places where |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 104 | # we expect 'bootm' to write things. Then quit. |
| 105 | base_script = ''' |
Simon Glass | f250d47 | 2018-11-15 18:44:02 -0700 | [diff] [blame] | 106 | host load hostfs 0 %(fit_addr)x %(fit)s |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 107 | fdt addr %(fit_addr)x |
| 108 | bootm start %(fit_addr)x |
| 109 | bootm loados |
Simon Glass | f250d47 | 2018-11-15 18:44:02 -0700 | [diff] [blame] | 110 | host save hostfs 0 %(kernel_addr)x %(kernel_out)s %(kernel_size)x |
| 111 | host save hostfs 0 %(fdt_addr)x %(fdt_out)s %(fdt_size)x |
| 112 | host save hostfs 0 %(ramdisk_addr)x %(ramdisk_out)s %(ramdisk_size)x |
| 113 | host save hostfs 0 %(loadables1_addr)x %(loadables1_out)s %(loadables1_size)x |
| 114 | host save hostfs 0 %(loadables2_addr)x %(loadables2_out)s %(loadables2_size)x |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 115 | ''' |
| 116 | |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 117 | @pytest.mark.boardspec('sandbox') |
| 118 | @pytest.mark.buildconfigspec('fit_signature') |
Stephen Warren | 2079db3 | 2017-09-18 11:11:49 -0600 | [diff] [blame] | 119 | @pytest.mark.requiredtool('dtc') |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 120 | def test_fit(u_boot_console): |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 121 | def make_fname(leaf): |
| 122 | """Make a temporary filename |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 123 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 124 | Args: |
| 125 | leaf: Leaf name of file to create (within temporary directory) |
| 126 | Return: |
| 127 | Temporary filename |
| 128 | """ |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 129 | |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 130 | return os.path.join(cons.config.build_dir, leaf) |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 131 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 132 | def filesize(fname): |
| 133 | """Get the size of a file |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 134 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 135 | Args: |
| 136 | fname: Filename to check |
| 137 | Return: |
| 138 | Size of file in bytes |
| 139 | """ |
| 140 | return os.stat(fname).st_size |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 141 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 142 | def read_file(fname): |
| 143 | """Read the contents of a file |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 144 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 145 | Args: |
| 146 | fname: Filename to read |
| 147 | Returns: |
| 148 | Contents of file as a string |
| 149 | """ |
Paul Burton | 4c35e97 | 2017-09-14 14:34:48 -0700 | [diff] [blame] | 150 | with open(fname, 'rb') as fd: |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 151 | return fd.read() |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 152 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 153 | def make_dtb(): |
| 154 | """Make a sample .dts file and compile it to a .dtb |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 155 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 156 | Returns: |
| 157 | Filename of .dtb file created |
| 158 | """ |
| 159 | src = make_fname('u-boot.dts') |
| 160 | dtb = make_fname('u-boot.dtb') |
| 161 | with open(src, 'w') as fd: |
Tom Rini | 6a99041 | 2019-10-24 11:59:21 -0400 | [diff] [blame] | 162 | fd.write(base_fdt) |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 163 | util.run_and_log(cons, ['dtc', src, '-O', 'dtb', '-o', dtb]) |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 164 | return dtb |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 165 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 166 | def make_its(params): |
| 167 | """Make a sample .its file with parameters embedded |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 168 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 169 | Args: |
| 170 | params: Dictionary containing parameters to embed in the %() strings |
| 171 | Returns: |
| 172 | Filename of .its file created |
| 173 | """ |
| 174 | its = make_fname('test.its') |
| 175 | with open(its, 'w') as fd: |
Paul Burton | 00f2d20 | 2017-09-14 14:34:43 -0700 | [diff] [blame] | 176 | print(base_its % params, file=fd) |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 177 | return its |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 178 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 179 | def make_fit(mkimage, params): |
| 180 | """Make a sample .fit file ready for loading |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 181 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 182 | This creates a .its script with the selected parameters and uses mkimage to |
| 183 | turn this into a .fit image. |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 184 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 185 | Args: |
| 186 | mkimage: Filename of 'mkimage' utility |
| 187 | params: Dictionary containing parameters to embed in the %() strings |
| 188 | Return: |
| 189 | Filename of .fit file created |
| 190 | """ |
| 191 | fit = make_fname('test.fit') |
| 192 | its = make_its(params) |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 193 | util.run_and_log(cons, [mkimage, '-f', its, fit]) |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 194 | with open(make_fname('u-boot.dts'), 'w') as fd: |
Tom Rini | 6a99041 | 2019-10-24 11:59:21 -0400 | [diff] [blame] | 195 | fd.write(base_fdt) |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 196 | return fit |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 197 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 198 | def make_kernel(filename, text): |
| 199 | """Make a sample kernel with test data |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 200 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 201 | Args: |
| 202 | filename: the name of the file you want to create |
| 203 | Returns: |
| 204 | Full path and filename of the kernel it created |
| 205 | """ |
| 206 | fname = make_fname(filename) |
| 207 | data = '' |
| 208 | for i in range(100): |
| 209 | data += 'this %s %d is unlikely to boot\n' % (text, i) |
| 210 | with open(fname, 'w') as fd: |
Paul Burton | 00f2d20 | 2017-09-14 14:34:43 -0700 | [diff] [blame] | 211 | print(data, file=fd) |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 212 | return fname |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 213 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 214 | def make_ramdisk(filename, text): |
| 215 | """Make a sample ramdisk with test data |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 216 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 217 | Returns: |
| 218 | Filename of ramdisk created |
| 219 | """ |
| 220 | fname = make_fname(filename) |
| 221 | data = '' |
| 222 | for i in range(100): |
| 223 | data += '%s %d was seldom used in the middle ages\n' % (text, i) |
| 224 | with open(fname, 'w') as fd: |
Paul Burton | 00f2d20 | 2017-09-14 14:34:43 -0700 | [diff] [blame] | 225 | print(data, file=fd) |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 226 | return fname |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 227 | |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 228 | def make_compressed(filename): |
| 229 | util.run_and_log(cons, ['gzip', '-f', '-k', filename]) |
| 230 | return filename + '.gz' |
| 231 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 232 | def find_matching(text, match): |
| 233 | """Find a match in a line of text, and return the unmatched line portion |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 234 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 235 | This is used to extract a part of a line from some text. The match string |
| 236 | is used to locate the line - we use the first line that contains that |
| 237 | match text. |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 238 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 239 | Once we find a match, we discard the match string itself from the line, |
| 240 | and return what remains. |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 241 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 242 | TODO: If this function becomes more generally useful, we could change it |
| 243 | to use regex and return groups. |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 244 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 245 | Args: |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 246 | text: Text to check (list of strings, one for each command issued) |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 247 | match: String to search for |
| 248 | Return: |
| 249 | String containing unmatched portion of line |
| 250 | Exceptions: |
| 251 | ValueError: If match is not found |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 252 | |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 253 | >>> find_matching(['first line:10', 'second_line:20'], 'first line:') |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 254 | '10' |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 255 | >>> find_matching(['first line:10', 'second_line:20'], 'second line') |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 256 | Traceback (most recent call last): |
| 257 | ... |
| 258 | ValueError: Test aborted |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 259 | >>> find_matching('first line:10\', 'second_line:20'], 'second_line:') |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 260 | '20' |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 261 | >>> find_matching('first line:10\', 'second_line:20\nthird_line:30'], |
| 262 | 'third_line:') |
| 263 | '30' |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 264 | """ |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 265 | __tracebackhide__ = True |
| 266 | for line in '\n'.join(text).splitlines(): |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 267 | pos = line.find(match) |
| 268 | if pos != -1: |
| 269 | return line[:pos] + line[pos + len(match):] |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 270 | |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 271 | pytest.fail("Expected '%s' but not found in output") |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 272 | |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 273 | def check_equal(expected_fname, actual_fname, failure_msg): |
| 274 | """Check that a file matches its expected contents |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 275 | |
Julius Werner | 8804002 | 2019-08-02 15:52:28 -0700 | [diff] [blame] | 276 | This is always used on out-buffers whose size is decided by the test |
| 277 | script anyway, which in some cases may be larger than what we're |
| 278 | actually looking for. So it's safe to truncate it to the size of the |
| 279 | expected data. |
| 280 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 281 | Args: |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 282 | expected_fname: Filename containing expected contents |
| 283 | actual_fname: Filename containing actual contents |
| 284 | failure_msg: Message to print on failure |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 285 | """ |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 286 | expected_data = read_file(expected_fname) |
| 287 | actual_data = read_file(actual_fname) |
Julius Werner | 8804002 | 2019-08-02 15:52:28 -0700 | [diff] [blame] | 288 | if len(expected_data) < len(actual_data): |
| 289 | actual_data = actual_data[:len(expected_data)] |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 290 | assert expected_data == actual_data, failure_msg |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 291 | |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 292 | def check_not_equal(expected_fname, actual_fname, failure_msg): |
| 293 | """Check that a file does not match its expected contents |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 294 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 295 | Args: |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 296 | expected_fname: Filename containing expected contents |
| 297 | actual_fname: Filename containing actual contents |
| 298 | failure_msg: Message to print on failure |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 299 | """ |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 300 | expected_data = read_file(expected_fname) |
| 301 | actual_data = read_file(actual_fname) |
| 302 | assert expected_data != actual_data, failure_msg |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 303 | |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 304 | def run_fit_test(mkimage): |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 305 | """Basic sanity check of FIT loading in U-Boot |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 306 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 307 | TODO: Almost everything: |
| 308 | - hash algorithms - invalid hash/contents should be detected |
| 309 | - signature algorithms - invalid sig/contents should be detected |
| 310 | - compression |
| 311 | - checking that errors are detected like: |
| 312 | - image overwriting |
| 313 | - missing images |
| 314 | - invalid configurations |
| 315 | - incorrect os/arch/type fields |
| 316 | - empty data |
| 317 | - images too large/small |
| 318 | - invalid FDT (e.g. putting a random binary in instead) |
| 319 | - default configuration selection |
| 320 | - bootm command line parameters should have desired effect |
| 321 | - run code coverage to make sure we are testing all the code |
| 322 | """ |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 323 | # Set up invariant files |
| 324 | control_dtb = make_dtb() |
| 325 | kernel = make_kernel('test-kernel.bin', 'kernel') |
| 326 | ramdisk = make_ramdisk('test-ramdisk.bin', 'ramdisk') |
| 327 | loadables1 = make_kernel('test-loadables1.bin', 'lenrek') |
| 328 | loadables2 = make_ramdisk('test-loadables2.bin', 'ksidmar') |
| 329 | kernel_out = make_fname('kernel-out.bin') |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 330 | fdt = make_fname('u-boot.dtb') |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 331 | fdt_out = make_fname('fdt-out.dtb') |
| 332 | ramdisk_out = make_fname('ramdisk-out.bin') |
| 333 | loadables1_out = make_fname('loadables1-out.bin') |
| 334 | loadables2_out = make_fname('loadables2-out.bin') |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 335 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 336 | # Set up basic parameters with default values |
| 337 | params = { |
| 338 | 'fit_addr' : 0x1000, |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 339 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 340 | 'kernel' : kernel, |
| 341 | 'kernel_out' : kernel_out, |
| 342 | 'kernel_addr' : 0x40000, |
| 343 | 'kernel_size' : filesize(kernel), |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 344 | |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 345 | 'fdt' : fdt, |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 346 | 'fdt_out' : fdt_out, |
| 347 | 'fdt_addr' : 0x80000, |
| 348 | 'fdt_size' : filesize(control_dtb), |
| 349 | 'fdt_load' : '', |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 350 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 351 | 'ramdisk' : ramdisk, |
| 352 | 'ramdisk_out' : ramdisk_out, |
| 353 | 'ramdisk_addr' : 0xc0000, |
| 354 | 'ramdisk_size' : filesize(ramdisk), |
| 355 | 'ramdisk_load' : '', |
| 356 | 'ramdisk_config' : '', |
Karl Apsite | aad8dd7 | 2015-05-21 09:52:50 -0400 | [diff] [blame] | 357 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 358 | 'loadables1' : loadables1, |
| 359 | 'loadables1_out' : loadables1_out, |
| 360 | 'loadables1_addr' : 0x100000, |
| 361 | 'loadables1_size' : filesize(loadables1), |
| 362 | 'loadables1_load' : '', |
Karl Apsite | aad8dd7 | 2015-05-21 09:52:50 -0400 | [diff] [blame] | 363 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 364 | 'loadables2' : loadables2, |
| 365 | 'loadables2_out' : loadables2_out, |
| 366 | 'loadables2_addr' : 0x140000, |
| 367 | 'loadables2_size' : filesize(loadables2), |
| 368 | 'loadables2_load' : '', |
Karl Apsite | aad8dd7 | 2015-05-21 09:52:50 -0400 | [diff] [blame] | 369 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 370 | 'loadables_config' : '', |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 371 | 'compression' : 'none', |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 372 | } |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 373 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 374 | # Make a basic FIT and a script to load it |
| 375 | fit = make_fit(mkimage, params) |
| 376 | params['fit'] = fit |
| 377 | cmd = base_script % params |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 378 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 379 | # First check that we can load a kernel |
| 380 | # We could perhaps reduce duplication with some loss of readability |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 381 | cons.config.dtb = control_dtb |
| 382 | cons.restart_uboot() |
| 383 | with cons.log.section('Kernel load'): |
| 384 | output = cons.run_command_list(cmd.splitlines()) |
| 385 | check_equal(kernel, kernel_out, 'Kernel not loaded') |
| 386 | check_not_equal(control_dtb, fdt_out, |
| 387 | 'FDT loaded but should be ignored') |
| 388 | check_not_equal(ramdisk, ramdisk_out, |
| 389 | 'Ramdisk loaded but should not be') |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 390 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 391 | # Find out the offset in the FIT where U-Boot has found the FDT |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 392 | line = find_matching(output, 'Booting using the fdt blob at ') |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 393 | fit_offset = int(line, 16) - params['fit_addr'] |
| 394 | fdt_magic = struct.pack('>L', 0xd00dfeed) |
| 395 | data = read_file(fit) |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 396 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 397 | # Now find where it actually is in the FIT (skip the first word) |
| 398 | real_fit_offset = data.find(fdt_magic, 4) |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 399 | assert fit_offset == real_fit_offset, ( |
| 400 | 'U-Boot loaded FDT from offset %#x, FDT is actually at %#x' % |
| 401 | (fit_offset, real_fit_offset)) |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 402 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 403 | # Now a kernel and an FDT |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 404 | with cons.log.section('Kernel + FDT load'): |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 405 | params['fdt_load'] = 'load = <%#x>;' % params['fdt_addr'] |
| 406 | fit = make_fit(mkimage, params) |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 407 | cons.restart_uboot() |
| 408 | output = cons.run_command_list(cmd.splitlines()) |
| 409 | check_equal(kernel, kernel_out, 'Kernel not loaded') |
| 410 | check_equal(control_dtb, fdt_out, 'FDT not loaded') |
| 411 | check_not_equal(ramdisk, ramdisk_out, |
| 412 | 'Ramdisk loaded but should not be') |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 413 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 414 | # Try a ramdisk |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 415 | with cons.log.section('Kernel + FDT + Ramdisk load'): |
Simon Glass | 4433890 | 2021-02-15 17:08:06 -0700 | [diff] [blame] | 416 | params['ramdisk_config'] = 'ramdisk = "ramdisk-1";' |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 417 | params['ramdisk_load'] = 'load = <%#x>;' % params['ramdisk_addr'] |
| 418 | fit = make_fit(mkimage, params) |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 419 | cons.restart_uboot() |
| 420 | output = cons.run_command_list(cmd.splitlines()) |
| 421 | check_equal(ramdisk, ramdisk_out, 'Ramdisk not loaded') |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 422 | |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 423 | # Configuration with some Loadables |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 424 | with cons.log.section('Kernel + FDT + Ramdisk load + Loadables'): |
Simon Glass | 4433890 | 2021-02-15 17:08:06 -0700 | [diff] [blame] | 425 | params['loadables_config'] = 'loadables = "kernel-2", "ramdisk-2";' |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 426 | params['loadables1_load'] = ('load = <%#x>;' % |
| 427 | params['loadables1_addr']) |
| 428 | params['loadables2_load'] = ('load = <%#x>;' % |
| 429 | params['loadables2_addr']) |
Simon Glass | 6867ec0 | 2017-08-05 10:28:39 -0600 | [diff] [blame] | 430 | fit = make_fit(mkimage, params) |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 431 | cons.restart_uboot() |
| 432 | output = cons.run_command_list(cmd.splitlines()) |
| 433 | check_equal(loadables1, loadables1_out, |
| 434 | 'Loadables1 (kernel) not loaded') |
| 435 | check_equal(loadables2, loadables2_out, |
| 436 | 'Loadables2 (ramdisk) not loaded') |
Simon Glass | db74919 | 2013-05-16 13:53:28 +0000 | [diff] [blame] | 437 | |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 438 | # Kernel, FDT and Ramdisk all compressed |
| 439 | with cons.log.section('(Kernel + FDT + Ramdisk) compressed'): |
| 440 | params['compression'] = 'gzip' |
| 441 | params['kernel'] = make_compressed(kernel) |
| 442 | params['fdt'] = make_compressed(fdt) |
| 443 | params['ramdisk'] = make_compressed(ramdisk) |
| 444 | fit = make_fit(mkimage, params) |
| 445 | cons.restart_uboot() |
| 446 | output = cons.run_command_list(cmd.splitlines()) |
| 447 | check_equal(kernel, kernel_out, 'Kernel not loaded') |
| 448 | check_equal(control_dtb, fdt_out, 'FDT not loaded') |
Julius Werner | 8804002 | 2019-08-02 15:52:28 -0700 | [diff] [blame] | 449 | check_not_equal(ramdisk, ramdisk_out, 'Ramdisk got decompressed?') |
| 450 | check_equal(ramdisk + '.gz', ramdisk_out, 'Ramdist not loaded') |
Julius Werner | 97b09cd | 2019-07-24 19:37:55 -0700 | [diff] [blame] | 451 | |
| 452 | |
Simon Glass | fe39411 | 2017-08-05 10:28:40 -0600 | [diff] [blame] | 453 | cons = u_boot_console |
| 454 | try: |
| 455 | # We need to use our own device tree file. Remember to restore it |
| 456 | # afterwards. |
| 457 | old_dtb = cons.config.dtb |
| 458 | mkimage = cons.config.build_dir + '/tools/mkimage' |
| 459 | run_fit_test(mkimage) |
| 460 | finally: |
| 461 | # Go back to the original U-Boot with the correct dtb. |
| 462 | cons.config.dtb = old_dtb |
| 463 | cons.restart_uboot() |