7:04 a.m.
On 01/30/2015 09:31 AM, Ramon Medeiros wrote:
On 01/29/2015 09:20 AM, Daniel Henrique Barboza wrote:
> There was no support for ppc fake ISOs in iso_gen.py, which led
> to several tests crashing in Power systems when the test tries
> to create a template with a fake Intel ISO.
>
> Signed-off-by: Daniel Henrique Barboza <dhbarboza82(a)gmail.com>
> ---
> tests/iso_gen.py | 53
> +++++++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 53 insertions(+)
>
> diff --git a/tests/iso_gen.py b/tests/iso_gen.py
> index 34f091d..72b0290 100644
> --- a/tests/iso_gen.py
> +++ b/tests/iso_gen.py
> @@ -17,6 +17,7 @@
> # License along with this library; if not, write to the Free Software
> # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
> 02110-1301 USA
>
> +import platform
> import struct
>
> from kimchi.isoinfo import IsoImage
> @@ -57,6 +58,17 @@ iso_des = [
>
> class FakeIsoImage(object):
> def _build_iso(self, fd, iso_volid, bootable):
> + if platform.machine().startswith('ppc'):
this will run on ppc64el. Had you tested with this platform?
Yeah. It works on ppc64 and ppc64le
> + self._build_powerpc_bootable_iso(fd, iso_volid)
> + return
> + self._build_intel_iso(fd, iso_volid, bootable)
> +
> + def _build_powerpc_bootable_iso(self, fd, iso_volid):
> + self._build_prim_vol(fd, iso_volid)
> + self._build_bootable_ppc_path_table(fd)
> +
> + def _build_intel_iso(self, fd, iso_volid, bootable):
> + # Do not change the order of the method calls
> self._build_el_boot(fd, bootable)
> self._build_prim_vol(fd, iso_volid)
> self._build_el_torito(fd)
> @@ -125,6 +137,47 @@ class FakeIsoImage(object):
> s = 'a' * IsoImage.SECTOR_SIZE
> fd.write(s)
>
> + def _build_bootable_ppc_path_table(self, fd):
> + # write path table locator
> + PATH_TABLE_LOC_OFFSET = 16 * IsoImage.SECTOR_SIZE + 132
> + PATH_TABLE_SIZE_LOC = struct.Struct("<I 4s I")
> + path_table_size = 64
> + path_table_loc = 18
> + fd.seek(PATH_TABLE_LOC_OFFSET)
> + fmt = PATH_TABLE_SIZE_LOC
> + data = (path_table_size, 4*'0', path_table_loc)
> + s = fmt.pack(*data)
> + fd.write(s)
> + # write path table entry
> + fd.seek(path_table_loc * IsoImage.SECTOR_SIZE)
> + DIR_NAMELEN_LOCATION_PARENT = struct.Struct("<B B I H 3s")
> + dir_struct_size = DIR_NAMELEN_LOCATION_PARENT.size
> + dir_namelen = 3
> + dir_loc = 19
> + dir_parent = 1
> + dir_name = 'ppc'
> + data = (dir_namelen, 0, dir_loc, dir_parent, dir_name)
> + fmt = DIR_NAMELEN_LOCATION_PARENT
> + s = fmt.pack(*data)
> + fd.write(s)
> + # write 'ppc' dir record
> + ppc_dir_offset = dir_loc * IsoImage.SECTOR_SIZE
> + fd.seek(ppc_dir_offset)
> + STATIC_DIR_RECORD_FMT = struct.Struct("<B 9s I 11s B 6s B
12s")
> + dir_rec_len = 1
> + unused1 = 9 * '0'
> + dir_size = 100
> + unused2 = 11 * '0'
> + file_flags = 0
> + unused3 = 6 * '0'
> + file_name_len = 12
> + boot_file_name = "bootinfo.txt"
> + data = (dir_rec_len, unused1, dir_size, unused2, file_flags,
> + unused3, file_name_len, boot_file_name)
> + fmt = STATIC_DIR_RECORD_FMT
> + s = fmt.pack(*data)
> + fd.write(s)
> +
>
> def construct_fake_iso(path, bootable, version, distro):
> iso = FakeIsoImage()