Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 29 Oct 2007 10:21:55 +0300
From:      Alexey Popov <lol@chistydom.ru>
To:        Alexander Kabaev <kabaev@gmail.com>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, Paul Saab <ps@FreeBSD.org>, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/boot/i386/libi386 Makefile pxe.c
Message-ID:  <47258A13.3060109@chistydom.ru>
In-Reply-To: <20071025081700.63a4d71b@kan.dnsalias.net>
References:  <200710121709.l9CH9iu7075715__38270.1859264218$1192209730$gmane$org@repoman.freebsd.org>	<47203DFE.2070800@chistydom.ru> <20071025081700.63a4d71b@kan.dnsalias.net>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi.

Alexander Kabaev wrote:

>> Paul Saab wrote:
>>> ps          2007-10-12 17:09:44 UTC
>>>
>>>   FreeBSD src repository
>>>
>>>   Modified files:
>>>     sys/boot/i386/libi386 Makefile pxe.c 
>>>   Log:
>>>   Do not attempt to make an NFS rpc call if using tftp
>>>   
>>>   PR:             kern/91720
>>>   Submitted by:   Ruben Kerkhof
>>>   
>>>   Revision  Changes    Path
>>>   1.43      +7 -0      src/sys/boot/i386/libi386/Makefile
>>>   1.23      +2 -0      src/sys/boot/i386/libi386/pxe.c
>> Didn't you break feature introduced in rev 1.21 of pxe.c ???
>> ===
>> Allow kernels loaded by pxeboot, which was  compiled with 
>> LOADER_TFTP_SUPPORT, to stll be able to mount NFS root as prescribed
>> by DCHP configuration. Since pxeboot is using TFTP to get to the
>> files, pxeboot can not rely on NFS to provide it a root directory
>> hande as a side effect. pxeboot has to  make RPC mount call itself.
>> ===
> 
> Paul did break it indeed.
I agree it's annoying to wait for RPC calls running PXE loader build 
with TFTP-only support. But I believe this feature was requested by 
someone just if it was committed. I think there should be more accurate 
fix, i.e. #define LOADER_TFTP_ONLY or something like this.

With best regards,
Alexey Popov



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?47258A13.3060109>