Re: [PATCH v2 0/5] stubdom: remove grub-pv

Juergen Gross <[email protected]>
Newsgroups org.xenproject.lists.xen-devel
Message-ID <[email protected]>
Any feedback?

On 20.07.26 10:18, Juergen Gross wrote:
> The grub-pv stubdoms (32- and 64-bit) are disabled by default since
> several years now.
> 
> Remove them in order to enable removing quite some more code from Xen.
> In case someone is really depending on grub-pv, they can easily take it
> from an older Xen build, as there is no Xen version dependency in
> grub-pv (a version built 3 years ago has been tested to still work
> with current 4.23 staging Xen).
> 
> Note that after this series has been committed, some additional
> cleanup is possible by removing stubdom libpci and zlib support, but
> this will require a modification of Mini-OS depending on these patches.
> 
> Changes in V2:
> - moved one hunk from patch 2 to patch 1
> 
> Juergen Gross (5):
>    stubdom: remove support for grub-pv
>    stubdom: remove support for building in 32-bit mode
>    stubdom: remove building of libxenguest and libxenctrl
>    docs: remove stale stubdom entries from stubdom.txt
>    tools/libxenguest: remove Mini-OS specific parts
> 
>   Makefile                                      |    6 -
>   config/Stubdom.mk.in                          |    3 -
>   docs/misc/stubdom.txt                         |   69 -
>   stubdom/.gitignore                            |    1 -
>   stubdom/Makefile                              |   69 +-
>   stubdom/configure                             |   65 -
>   stubdom/configure.ac                          |    2 -
>   stubdom/grub.patches/00cvs                    | 1022 -----
>   stubdom/grub.patches/10graphics.diff          | 2297 -----------
>   stubdom/grub.patches/11graphics-keyboard.diff |   13 -
>   stubdom/grub.patches/20print_func.diff        |   52 -
>   stubdom/grub.patches/30savedefault.diff       |  186 -
>   .../grub.patches/40ext3_256byte_inode.diff    |  114 -
>   stubdom/grub.patches/50fs_fulldisk.diff       |   72 -
>   stubdom/grub.patches/60ext4.diff              |  474 ---
>   stubdom/grub.patches/61btrfs.diff             | 3499 -----------------
>   stubdom/grub.patches/70compiler_warnings.diff |   45 -
>   stubdom/grub.patches/99minios                 | 1570 --------
>   stubdom/grub/Makefile                         |   88 -
>   stubdom/grub/boot-x86_32.S                    |  112 -
>   stubdom/grub/boot-x86_64.S                    |  108 -
>   stubdom/grub/config.h                         |   12 -
>   stubdom/grub/kexec.c                          |  434 --
>   stubdom/grub/mini-os.c                        |  771 ----
>   stubdom/grub/mini-os.h                        |    7 -
>   stubdom/grub/minios.cfg                       |    4 -
>   stubdom/grub/osdep.h                          |   30 -
>   tools/libs/guest/Makefile.common              |   15 -
>   tools/libs/guest/xg_dom_decompress_unsafe.c   |   48 -
>   tools/libs/guest/xg_dom_decompress_unsafe.h   |   28 -
>   .../guest/xg_dom_decompress_unsafe_bzip2.c    |   14 -
>   .../libs/guest/xg_dom_decompress_unsafe_lz4.c |   39 -
>   .../guest/xg_dom_decompress_unsafe_lzma.c     |   14 -
>   .../guest/xg_dom_decompress_unsafe_lzo1x.c    |   44 -
>   .../libs/guest/xg_dom_decompress_unsafe_xz.c  |   46 -
>   .../guest/xg_dom_decompress_unsafe_zstd.c     |   44 -
>   36 files changed, 1 insertion(+), 11416 deletions(-)
>   delete mode 100644 stubdom/grub.patches/00cvs
>   delete mode 100644 stubdom/grub.patches/10graphics.diff
>   delete mode 100644 stubdom/grub.patches/11graphics-keyboard.diff
>   delete mode 100644 stubdom/grub.patches/20print_func.diff
>   delete mode 100644 stubdom/grub.patches/30savedefault.diff
>   delete mode 100644 stubdom/grub.patches/40ext3_256byte_inode.diff
>   delete mode 100644 stubdom/grub.patches/50fs_fulldisk.diff
>   delete mode 100644 stubdom/grub.patches/60ext4.diff
>   delete mode 100644 stubdom/grub.patches/61btrfs.diff
>   delete mode 100644 stubdom/grub.patches/70compiler_warnings.diff
>   delete mode 100644 stubdom/grub.patches/99minios
>   delete mode 100644 stubdom/grub/Makefile
>   delete mode 100644 stubdom/grub/boot-x86_32.S
>   delete mode 100644 stubdom/grub/boot-x86_64.S
>   delete mode 100644 stubdom/grub/config.h
>   delete mode 100644 stubdom/grub/kexec.c
>   delete mode 100644 stubdom/grub/mini-os.c
>   delete mode 100644 stubdom/grub/mini-os.h
>   delete mode 100644 stubdom/grub/minios.cfg
>   delete mode 100644 stubdom/grub/osdep.h
>   delete mode 100644 tools/libs/guest/xg_dom_decompress_unsafe.c
>   delete mode 100644 tools/libs/guest/xg_dom_decompress_unsafe.h
>   delete mode 100644 tools/libs/guest/xg_dom_decompress_unsafe_bzip2.c
>   delete mode 100644 tools/libs/guest/xg_dom_decompress_unsafe_lz4.c
>   delete mode 100644 tools/libs/guest/xg_dom_decompress_unsafe_lzma.c
>   delete mode 100644 tools/libs/guest/xg_dom_decompress_unsafe_lzo1x.c
>   delete mode 100644 tools/libs/guest/xg_dom_decompress_unsafe_xz.c
>   delete mode 100644 tools/libs/guest/xg_dom_decompress_unsafe_zstd.c
>
OpenPGP_0xB0DE9DD628BF132F.asc (application/pgp-keys, 3.6 KB)
-----BEGIN PGP PUBLIC KEY BLOCK-----

xsBNBFOMcBYBCACgGjqjoGvbEouQZw/ToiBg9W98AlM2QHV+iNHsEs7kxWhKMjri
oyspZKOBycWxw3ie3j9uvg9EOB3aN4xiTv4qbnGiTr3oJhkB1gsb6ToJQZ8uxGq2
kaV2KL9650I1SJvedYm8Of8Zd621lSmoKOwlNClALZNew72NjJLEzTalU1OdT7/i
1TXkH09XSSI8mEQ/ouNcMvIJNwQpd369y9bfIhWUiVXEK7MlRgUG6MvIj6Y3Am/B
BLUVbDa4+gmzDC9ezlZkTZG2t14zWPvxXP3FAp2pkW0xqG7/377qptDmrk42GlSK
N4z76ELnLxussxc7I2hx18NUcbP8+uty4bMxABEBAAHNHEp1ZXJnZW4gR3Jvc3Mg
PGpnQHBmdXBmLm5ldD7CwHkEEwECACMFAlOMcBYCGwMHCwkIBwMCAQYVCAIJCgsE
FgIDAQIeAQIXgAAKCRCw3p3WKL8TL0KdB/93FcIZ3GCNwFU0u3EjNbNjmXBKDY4F
UGNQH2lvWAUy+dnyThpwdtF/jQ6j9RwE8VP0+NXcYpGJDWlNb9/JmYqLiX2Q3Tye
vpB0CA3dbBQp0OW0fgCetToGIQrg0MbD1C/sEOv8Mr4NAfbauXjZlvTj30H2jO0u
+6WGM6nHwbh2l5O8ZiHkH32iaSTfN7Eu5RnNVUJbvoPHZ8SlM4KWm8rG+lIkGurq
qu5gu8q8ZMKdsdGC4bBxdQKDKHEFExLJK/nRPFmAuGlId1E3fe10v5QL+qHI3EIP
tyfE7i9Hz6rVwi7lWKgh7pe0ZvatAudZ+JNIlBKptb64FaiIOAWDCx1SzR9KdWVy
Z2VuIEdyb3NzIDxqZ3Jvc3NAc3VzZS5jb20+wsB5BBMBAgAjBQJTjHCvAhsDBwsJ
CAcDAgEGFQgCCQoLBBYCAwECHgECF4AACgkQsN6d1ii/Ey/HmQf/RtI7kv5A2PS4
RF7HoZhPVPogNVbC4YA6lW7DrWf0teC0RR3MzXfy6pJ+7KLgkqMlrAbN/8Dvjoz7
8X+5vhH/rDLa9BuZQlhFmvcGtCF8eR0T1v0nC/nuAFVGy+67q2DH8As3KPu0344T
BDpAvr2uYM4tSqxK4DURx5INz4ZZ0WNFHcqsfvlGJALDeE0LhITTd9jLzdDad1pQ
SToCnLl6SBJZjDOX9QQcyUigZFtCXFst4dlsvddrxyqT1f17+2cFSdu7+ynLmXBK
7abQ3rwJY8SbRO2iRulogc5vr/RLMMlscDAiDkaFQWLoqHHOdfO9rURssHNN8WkM
nQfvUewRz80hSnVlcmdlbiBHcm9zcyA8amdyb3NzQG5vdmVsbC5jb20+wsB5BBMB
AgAjBQJTjHDXAhsDBwsJCAcDAgEGFQgCCQoLBBYCAwECHgECF4AACgkQsN6d1ii/
Ey8PUQf/ehmgCI9jB9hlgexLvgOtf7PJnFOXgMLdBQgBlVPO3/D9R8LtF9DBAFPN
hlrsfIG/SqICoRCqUcJ96Pn3P7UUinFG/I0ECGF4EvTE1jnDkfJZr6jrbjgyoZHi
w/4BNwSTL9rWASyLgqlA8u1mf+c2yUwcGhgkRAd1gOwungxcwzwqgljf0N51N5Jf
VRHRtyfwq/ge+YEkDGcTU6Y0sPOuj4Dyfm8fJzdfHNQsWq3PnczLVELStJNdapwP
OoE+lotufe3AM2vAEYJ9rTz3Cki4JFUsgLkHFqGZarrPGi1eyQcXeluldO3m91NK
/1xMI3/+8jbO0tsn1tqSEUGIJi7ox80eSnVlcmdlbiBHcm9zcyA8amdyb3NzQHN1
c2UuZGU+wsB5BBMBAgAjBQJTjHDrAhsDBwsJCAcDAgEGFQgCCQoLBBYCAwECHgEC
F4AACgkQsN6d1ii/Ey+LhQf9GL45eU5vOowA2u5N3g3OZUEBmDHVVbqMtzwlmNC4
k9Kx39r5s2vcFl4tXqW7g9/ViXYuiDXb0RfUpZiIUW89siKrkzmQ5dM7wRqzgJpJ
wK8Bn2MIxAKArekWpiCKvBOB/Cc+3EXE78XdlxLyOi/NrmSGRIov0karw2RzMNOu
5D+jLRZQd1Sv27AR+IP3I8U4aqnhLpwhK7MEy9oCILlgZ1QZe49kpcumcZKORmzB
TNh30FVKK1EvmV2xAKDoaEOgQB4iFQLhJCdP1I5aSgM5IVFdn7v5YgEYuJYx37Io
N1EblHI//x/e2AaIHpzK5h88NEawQsaNRpNSrcfbFmAg987ATQRTjHAWAQgAyzH6
AOODMBjgfWE9VeCgsrwH3exNAU32gLq2xvjpWnHIs98ndPUDpnoxWQugJ6MpMncr
0xSwFmHEgnSEjK/PAjppgmyc57BwKII3sV4on+gDVFJR6Y8ZRwgnBC5mVM6JjQ5x
Dk8WRXljExRfUX9pNhdE5eBOZJrDRoLUmmjDtKzWaDhIg/+1Hzz93X4fCQkNVbVF
LELU9bMaLPBG/x5q4iYZ2k2ex6d47YE1ZFdMm6YBYMOljGkZKwYde5ldM9mo45mm
we0icXKLkpEdIXKTZeKDO+Hdv1aqFuAcccTg9RXDQjmwhC3yEmrmcfl0+rPghO0I
v3OOImwTEe4co3c1mwARAQABwsBfBBgBAgAJBQJTjHAWAhsMAAoJELDendYovxMv
Q/gH/1ha96vm4P/L+bQpJwrZ/dneZcmEwTbe8YFsw2V/Buv6Z4Mysln3nQK5ZadD
534CF7TDVft7fC4tU4PONxF5D+/tvgkPfDAfF77zy2AH1vJzQ1fOU8lYFpZXTXIH
b+559UqvIB8AdgR3SAJGHHt4RKA0F7f5ipYBBrC6cyXJyyoprT10EMvU8VGiwXvT
yJz3fjoYsdFzpWPlJEBRMedCot60g5dmbdrZ5DWClAr0yau47zpWj3enf1tLWaqc
suylWsviuGjKGw7KHQd3bxALOknAp4dN3QwBYCKuZ7AddY9yjynVaD5X7nF9nO5B
jR/i1DG86lem3iBDXzXsZDn8R3/CwO0EGAEIACAWIQSFEmdy6PYElKXQl/ew3p3W
KL8TLwUCWt3w0AIbAgCBCRCw3p3WKL8TL3YgBBkWCAAdFiEEUy2wekH2OPMeOLge
gFxhu0/YY74FAlrd8NAACgkQgFxhu0/YY75NiwD/fQf/RXpyv9ZX4n8UJrKDq422
bcwkujisT6jix2mOOwYBAKiip9+mAD6W5NPXdhk1XraECcIspcf2ff5kCAlG0DIN
aTUH/RIwNWzXDG58yQoLdD/UPcFgi8GWtNUp0Fhc/GeBxGipXYnvuWxwS+Qs1Qay
7/Nbal/v4/eZZaWs8wl2VtrHTS96/IF6q2o0qMey0dq2AxnZbQIULiEndgR625EF
RFg+IbO4ldSkB3trsF2ypYLij4ZObm2casLIP7iB8NKmQ5PndL8Y07TtiQ+Sb/wn
g4GgV+BJoKdDWLPCAlCMilwbZ88Ijb+HF/aipc9hsqvW/hnXC2GajJSAY3Qs9Mib
4Hm91jzbAjmp7243pQ4bJMfYHemFFBRaoLC7ayqQjcsttN2ufINlqLFPZPR/i3IX
kt+z4drzFUyEjLM1vVvIMjkUoJs=
=eeAB
-----END PGP PUBLIC KEY BLOCK-----
OpenPGP_signature.asc (application/pgp-signature, 495 B)
-----BEGIN PGP SIGNATURE-----

wsB5BAABCAAjFiEEhRJncuj2BJSl0Jf3sN6d1ii/Ey8FAmp8K/oFAwAAAAAACgkQsN6d1ii/Ey9f
awf8D8TO6kxEcF4Ar69AsZ0hsAYl7PT5h0YXLvbF8aB1U0q3xFk/xjZbS37doDYLymDSHBP6JKxg
fmeJe3aXULSSR+mw8r/TyFb89Wv4FS5ucCjrd/MA2ZYaQuTlCmG/4jzdYAd+eC+IYCv33whT5Bqs
PRNjLnOvWioF3dsR7OxYYCAJEnOYuLPlFEvexLZg8VPPlgeXzYsRu0a1CxSm4/0YJptpbQTEQqGw
TYB9T5S4ncb4Usq81exUxqre3KPIWAoT3Y31vYDzXNePgwSPo6rmv97twtDI8zbKf70rCeQDm7hD
TyZg4zaRPKQjkLLJ+IQoVH66vh8WhLTcn+exNSp+xw==
=iw9k
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.