[Qemu-devel] [PATCH] Fix the RARP protocol ID

[Qemu-devel] [PATCH] Fix the RARP protocol ID

[Stefan Berger]

The packet(s) sent out after migration are supposed to be RAPR type of
packets. If they are supposed to go anywhere useful, the RAPR ethernet
identifier needs to be fix.

Also see http://www.iana.org/assignments/ethernet-numbers for 0x8035 for
RARP.

Signed-off-by: Stefan Berger <stefanb@us.ibm.com>

---
 savevm.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: qemu-git/savevm.c
===================================================================
--- qemu-git.orig/savevm.c
+++ qemu-git/savevm.c
@@ -89,7 +89,7 @@ static BlockDriverState *bs_snapshots;
 #define SELF_ANNOUNCE_ROUNDS 5
 
 #ifndef ETH_P_RARP
-#define ETH_P_RARP 0x0835
+#define ETH_P_RARP 0x8035
 #endif
 #define ARP_HTYPE_ETH 0x0001
 #define ARP_PTYPE_IP 0x0800

Re: [Qemu-devel] [PATCH] Fix the RARP protocol ID

[Stefan Berger]

[-- Attachment #1: Type: text/plain, Size: 1228 bytes --]

qemu-devel-bounces+stefanb=us.ibm.com@nongnu.org wrote on 04/24/2010 
08:54:07 AM:


> The packet(s) sent out after migration are supposed to be RAPR type of
> packets. If they are supposed to go anywhere useful, the RAPR ethernet
> identifier needs to be fix.
> 
> Also see http://www.iana.org/assignments/ethernet-numbers for 0x8035 for
> RARP.

Hi! It's probably not high on the agenda, but any reason why the created 
packet
looks like a RAPR packet and yet has the wrong ethernet protocol ID? If 
this 
packet is sent  to tell the local switch that the VM is now on a different 

port, why not just send an ARP request packet asking for an IP address of 
0.0.0.0 ?

   Stefan

> 
> Signed-off-by: Stefan Berger <stefanb@us.ibm.com>
> 
> ---
>  savevm.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Index: qemu-git/savevm.c
> ===================================================================
> --- qemu-git.orig/savevm.c
> +++ qemu-git/savevm.c
> @@ -89,7 +89,7 @@ static BlockDriverState *bs_snapshots;
>  #define SELF_ANNOUNCE_ROUNDS 5
> 
>  #ifndef ETH_P_RARP
> -#define ETH_P_RARP 0x0835
> +#define ETH_P_RARP 0x8035
>  #endif
>  #define ARP_HTYPE_ETH 0x0001
>  #define ARP_PTYPE_IP 0x0800
> 
> 
> 

[-- Attachment #2: Type: text/html, Size: 1871 bytes --]

Re: [Qemu-devel] [PATCH] Fix the RARP protocol ID

[Anthony Liguori]

On 04/24/2010 07:54 AM, Stefan Berger wrote:
> The packet(s) sent out after migration are supposed to be RAPR type of
> packets. If they are supposed to go anywhere useful, the RAPR ethernet
> identifier needs to be fix.
>
> Also see http://www.iana.org/assignments/ethernet-numbers for 0x8035 for
> RARP.
>
> Signed-off-by: Stefan Berger<stefanb@us.ibm.com>
>    

Applied.  Thanks.

Regards,

Anthony Liguori
> ---
>   savevm.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> Index: qemu-git/savevm.c
> ===================================================================
> --- qemu-git.orig/savevm.c
> +++ qemu-git/savevm.c
> @@ -89,7 +89,7 @@ static BlockDriverState *bs_snapshots;
>   #define SELF_ANNOUNCE_ROUNDS 5
>
>   #ifndef ETH_P_RARP
> -#define ETH_P_RARP 0x0835
> +#define ETH_P_RARP 0x8035
>   #endif
>   #define ARP_HTYPE_ETH 0x0001
>   #define ARP_PTYPE_IP 0x0800
>
>
>
>
>