Fix the RARP protocol ID

Submitted by Stefan Berger on April 24, 2010, 12:54 p.m.

Details

Message ID 1272113647.8527.16.camel@d941e-10
State New
Headers show

Commit Message

Stefan Berger April 24, 2010, 12:54 p.m.
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(-)

Comments

Stefan Berger April 29, 2010, 6:32 p.m.
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
> 
> 
>
Anthony Liguori May 3, 2010, 5:13 p.m.
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
>
>
>
>
>

Patch hide | download patch | download mbox

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