From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] Work around dhclient brokenness Date: Mon, 18 Aug 2008 13:56:57 +0300 Message-ID: <48A95579.6080004@qumranet.com> References: <1218829632-19037-1-git-send-email-aliguori@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org, Mark McLoughlin , Rusty Russell , Herbert Xu To: Anthony Liguori Return-path: Received: from il.qumranet.com ([212.179.150.194]:56949 "EHLO il.qumranet.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750943AbYHRK46 (ORCPT ); Mon, 18 Aug 2008 06:56:58 -0400 In-Reply-To: <1218829632-19037-1-git-send-email-aliguori@us.ibm.com> Sender: kvm-owner@vger.kernel.org List-ID: Anthony Liguori wrote: > With the latest GSO/csum offload patches, any guest using an unpatched version > of dhclient (any Ubuntu guest, for instance), will no longer be able to get > a DHCP address. > > dhclient is actually at fault here. It uses AF_PACKET to receive DHCP responses > but does not check auxdata to see if the packet has a valid csum. This causes > it to throw out the DHCP responses it gets from the virtio interface as there > is not a valid checksum. > > Fedora has carried a patch to fix their dhclient (it's needed for Xen too) but > this patch has not made it into a release of dhclient. AFAIK, the patch is in > the dhclient CVS but I cannot confirm since their CVS is not public. > > This patch, suggested by Rusty, looks for UDP packets (of a normal MTU) and > explicitly adds a checksum to them if they are missing one. We could further > refine the search criteria based on srcport but that's probably unnecessary. > > Won't this slow down nfs/udp? I think a srcport check would be good here. -- error compiling committee.c: too many arguments to function