* [Qemu-devel] [PULL] slirp updates @ 2016-03-31 22:21 Samuel Thibault 2016-03-31 22:21 ` [Qemu-devel] [PULL] slirp: Fix migration from older versions of QEMU to the current one Samuel Thibault 2016-04-01 11:42 ` [Qemu-devel] [PULL] slirp updates Peter Maydell 0 siblings, 2 replies; 5+ messages in thread From: Samuel Thibault @ 2016-03-31 22:21 UTC (permalink / raw) To: qemu-devel, peter.maydell; +Cc: Samuel Thibault, jan.kiszka MAINTAINERS: Delete invalid maintainer entries of the Exynos section (2016-03-31 18:21:01 +0100) are available in the git repository at: http://people.debian.org/~sthibault/qemu.git tags/samuel-thibault-2 for you to fetch changes up to eaf136f9a21e02a2f55346e44d2d88df37b2cde3: slirp: Fix migration from older versions of QEMU to the current one (2016-04-01 00:05:06 +0200) ---------------------------------------------------------------- slirp updates (2) ---------------------------------------------------------------- Thomas Huth (1): slirp: Fix migration from older versions of QEMU to the current one slirp/slirp.c | 44 ++++++++++++++++++++++++++------------------ 1 file changed, 26 insertions(+), 18 deletions(-) ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Qemu-devel] [PULL] slirp: Fix migration from older versions of QEMU to the current one 2016-03-31 22:21 [Qemu-devel] [PULL] slirp updates Samuel Thibault @ 2016-03-31 22:21 ` Samuel Thibault 2016-04-01 11:42 ` [Qemu-devel] [PULL] slirp updates Peter Maydell 1 sibling, 0 replies; 5+ messages in thread From: Samuel Thibault @ 2016-03-31 22:21 UTC (permalink / raw) To: qemu-devel, peter.maydell; +Cc: jan.kiszka, Thomas Huth, Samuel Thibault From: Thomas Huth <thuth@redhat.com> While adding the IPv6 support, the commit eae303ff23f51259eddc8856c71453d8 ("slirp: Make Socket structure IPv6 compatible") changed the format of the migration stream, without taking into account that we might still receive an old migration stream layout when upgrading from QEMU version 2.5 (or older) to QEMU 2.6. Currently, QEMU bails out when doing a migration from QEMU 2.5 to the recent master version when it has been started with a "-net user,guestfwd=..." network. So let's fix this by checking the version ID of the migration stream and by using the old behavior if we've detected version 3 or less. Signed-off-by: Thomas Huth <thuth@redhat.com> Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org> --- slirp/slirp.c | 44 ++++++++++++++++++++++++++------------------ 1 file changed, 26 insertions(+), 18 deletions(-) diff --git a/slirp/slirp.c b/slirp/slirp.c index 3481fcc..998f278 100644 --- a/slirp/slirp.c +++ b/slirp/slirp.c @@ -1233,31 +1233,39 @@ static int slirp_sbuf_load(QEMUFile *f, struct sbuf *sbuf) return 0; } -static int slirp_socket_load(QEMUFile *f, struct socket *so) +static int slirp_socket_load(QEMUFile *f, struct socket *so, int version_id) { if (tcp_attach(so) < 0) return -ENOMEM; so->so_urgc = qemu_get_be32(f); - so->so_ffamily = qemu_get_be16(f); - switch (so->so_ffamily) { - case AF_INET: + if (version_id <= 3) { + so->so_ffamily = AF_INET; so->so_faddr.s_addr = qemu_get_be32(f); - so->so_fport = qemu_get_be16(f); - break; - default: - error_report( - "so_ffamily unknown, unable to restore so_faddr and so_lport\n"); - } - so->so_lfamily = qemu_get_be16(f); - switch (so->so_lfamily) { - case AF_INET: so->so_laddr.s_addr = qemu_get_be32(f); + so->so_fport = qemu_get_be16(f); so->so_lport = qemu_get_be16(f); - break; - default: - error_report( - "so_ffamily unknown, unable to restore so_laddr and so_lport\n"); + } else { + so->so_ffamily = qemu_get_be16(f); + switch (so->so_ffamily) { + case AF_INET: + so->so_faddr.s_addr = qemu_get_be32(f); + so->so_fport = qemu_get_be16(f); + break; + default: + error_report( + "so_ffamily unknown, unable to restore so_faddr and so_lport"); + } + so->so_lfamily = qemu_get_be16(f); + switch (so->so_lfamily) { + case AF_INET: + so->so_laddr.s_addr = qemu_get_be32(f); + so->so_lport = qemu_get_be16(f); + break; + default: + error_report( + "so_ffamily unknown, unable to restore so_laddr and so_lport"); + } } so->so_iptos = qemu_get_byte(f); so->so_emu = qemu_get_byte(f); @@ -1294,7 +1302,7 @@ static int slirp_state_load(QEMUFile *f, void *opaque, int version_id) if (!so) return -ENOMEM; - ret = slirp_socket_load(f, so); + ret = slirp_socket_load(f, so, version_id); if (ret < 0) return ret; -- 2.8.0.rc3 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PULL] slirp updates 2016-03-31 22:21 [Qemu-devel] [PULL] slirp updates Samuel Thibault 2016-03-31 22:21 ` [Qemu-devel] [PULL] slirp: Fix migration from older versions of QEMU to the current one Samuel Thibault @ 2016-04-01 11:42 ` Peter Maydell 1 sibling, 0 replies; 5+ messages in thread From: Peter Maydell @ 2016-04-01 11:42 UTC (permalink / raw) To: Samuel Thibault; +Cc: J. Kiszka, QEMU Developers On 31 March 2016 at 23:21, Samuel Thibault <samuel.thibault@ens-lyon.org> wrote: > MAINTAINERS: Delete invalid maintainer entries of the Exynos section (2016-03-31 18:21:01 +0100) > > are available in the git repository at: > > http://people.debian.org/~sthibault/qemu.git tags/samuel-thibault-2 > > for you to fetch changes up to eaf136f9a21e02a2f55346e44d2d88df37b2cde3: > > slirp: Fix migration from older versions of QEMU to the current one (2016-04-01 00:05:06 +0200) > > ---------------------------------------------------------------- > slirp updates (2) > Applied, thanks. -- PMM ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Qemu-devel] [PULL] slirp updates @ 2016-04-01 15:53 Samuel Thibault 2016-04-04 12:29 ` Peter Maydell 0 siblings, 1 reply; 5+ messages in thread From: Samuel Thibault @ 2016-04-01 15:53 UTC (permalink / raw) To: qemu-devel, peter.maydell; +Cc: Samuel Thibault, jan.kiszka The following changes since commit de1d099a448beb2ec39af4bd9ce4dd6452a18cb5: Merge remote-tracking branch 'remotes/thibault/tags/samuel-thibault-2' into staging (2016-04-01 11:15:20 +0100) are available in the git repository at: http://people.debian.org/~sthibault/qemu.git tags/samuel-thibault-2 for you to fetch changes up to 0b11c036620251f8f831ab8f0d9a7280ec1c4813: slirp: Allow disabling IPv4 or IPv6 (2016-04-01 17:51:55 +0200) ---------------------------------------------------------------- slirp updates (2) ---------------------------------------------------------------- Samuel Thibault (1): slirp: Allow disabling IPv4 or IPv6 net/slirp.c | 36 ++++++++++++++++++++++++++++++------ qapi-schema.json | 8 ++++++++ qemu-options.hx | 8 ++++++-- slirp/ip6_icmp.c | 8 ++++++++ slirp/ip6_input.c | 5 +++++ slirp/ip_input.c | 4 ++++ slirp/libslirp.h | 3 ++- slirp/slirp.c | 10 +++++++++- slirp/slirp.h | 2 ++ 9 files changed, 74 insertions(+), 10 deletions(-) ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PULL] slirp updates 2016-04-01 15:53 Samuel Thibault @ 2016-04-04 12:29 ` Peter Maydell 0 siblings, 0 replies; 5+ messages in thread From: Peter Maydell @ 2016-04-04 12:29 UTC (permalink / raw) To: Samuel Thibault; +Cc: J. Kiszka, QEMU Developers On 1 April 2016 at 16:53, Samuel Thibault <samuel.thibault@ens-lyon.org> wrote: > The following changes since commit de1d099a448beb2ec39af4bd9ce4dd6452a18cb5: > > Merge remote-tracking branch 'remotes/thibault/tags/samuel-thibault-2' into staging (2016-04-01 11:15:20 +0100) > > are available in the git repository at: > > http://people.debian.org/~sthibault/qemu.git tags/samuel-thibault-2 > > for you to fetch changes up to 0b11c036620251f8f831ab8f0d9a7280ec1c4813: > > slirp: Allow disabling IPv4 or IPv6 (2016-04-01 17:51:55 +0200) > > ---------------------------------------------------------------- > slirp updates (2) > > ---------------------------------------------------------------- > Samuel Thibault (1): > slirp: Allow disabling IPv4 or IPv6 Applied, thanks. -- PMM ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-04-04 12:30 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-03-31 22:21 [Qemu-devel] [PULL] slirp updates Samuel Thibault 2016-03-31 22:21 ` [Qemu-devel] [PULL] slirp: Fix migration from older versions of QEMU to the current one Samuel Thibault 2016-04-01 11:42 ` [Qemu-devel] [PULL] slirp updates Peter Maydell 2016-04-01 15:53 Samuel Thibault 2016-04-04 12:29 ` Peter Maydell
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.