Re: [gPXE-devel] [PULL] ipxe-sync

2010-07-07 Thread Marty Connor
On 7/7/10 6:01 PM, Michael Brown wrote: > On Wednesday 07 Jul 2010 20:25:27 Stefan Hajnoczi wrote: >> Applied ipxe-sync! >> >> http://git.etherboot.org/?p=gpxe.git;a=shortlog;h=refs/heads/master > > You missed one: > > http://git.ipxe.org/ipxe.git/commit/8406115 > > [This is only partly tongue

[gPXE-devel] [PATCH][v2] Run com32 programs with a valid IDT

2010-07-07 Thread Geoff Lywood
Here is an updated patch. Changes since the first version: - Instructions "retf" and "movzxb" have been replaced with "lret" and "movzbl" because that's the correct AT&T syntax - The three calls to prep_segment were merged into a single call $ util/diffsize.pl origin/master HEAD com32 +1

Re: [gPXE-devel] [PULL] ipxe-sync

2010-07-07 Thread Michael Brown
On Wednesday 07 Jul 2010 20:25:27 Stefan Hajnoczi wrote: > Applied ipxe-sync! > > http://git.etherboot.org/?p=gpxe.git;a=shortlog;h=refs/heads/master You missed one: http://git.ipxe.org/ipxe.git/commit/8406115 [This is only partly tongue-in-cheek; over 60% of the gPXE changes since the time

Re: [gPXE-devel] [PATCH] Fix autoboot for multiple NIC's on different networks

2010-07-07 Thread Michael Brown
On Wednesday 07 Jul 2010 17:21:21 Joshua Oreman wrote: > I'm not convinced that this patch is correct. I think you want to > clear the netX/filename and netX/root-path settings after a boot from > netX fails, but still allow global filename and root-path settings to > override them (which is the cu

Re: [gPXE-devel] [PATCH] Run COM32 programs with a valid IDT

2010-07-07 Thread Stefan Hajnoczi
On Wed, Jul 7, 2010 at 8:49 PM, Geoff Lywood wrote: > This patch adds a fair amount of code size... ~300 bytes I think. Is there > some kind of script I can run that will tell me exactly how much it has > changed? $ size bin/com32.o or $ make bin/gpxe.hd.sizes or util/diffsize.pl Take your

Re: [gPXE-devel] [PATCH] Run COM32 programs with a valid IDT

2010-07-07 Thread Geoff Lywood
This patch adds a fair amount of code size... ~300 bytes I think. Is there some kind of script I can run that will tell me exactly how much it has changed? I can see some obvious optimizations I can do to eliminate some of this added code, and I'll send out a new patch soon. Thanks, Geoff > -

Re: [gPXE-devel] [PATCH] Run COM32 programs with a valid IDT

2010-07-07 Thread Stefan Hajnoczi
On Tue, Jun 29, 2010 at 2:18 AM, Geoff Lywood wrote: > COM32 binaries generally expect to run with interrupts enabled. Syslinux does > so, and COM32 programs will execute cli/sti pairs when running a critical > section, to provide mutual exclusion against BIOS interrupt handlers. > Previously, und

Re: [gPXE-devel] [PULL] ipxe-sync

2010-07-07 Thread Stefan Hajnoczi
Applied ipxe-sync! http://git.etherboot.org/?p=gpxe.git;a=shortlog;h=refs/heads/master Stefan ___ gPXE-devel mailing list gPXE-devel@etherboot.org http://etherboot.org/mailman/listinfo/gpxe-devel

Re: [gPXE-devel] [PATCH] Fix autoboot for multiple NIC's on different networks

2010-07-07 Thread Gianni Tedesco
On Wed, 2010-07-07 at 17:21 +0100, Joshua Oreman wrote: > On Wed, Jul 7, 2010 at 8:35 AM, Gianni Tedesco > wrote: > > Hi, > > > > Without this patch retrieving boot image from the second "next-server" > > and "filename" actually ends up using the results obtained from DHCP on > > the first NIC aft

Re: [gPXE-devel] [PATCH] Fix autoboot for multiple NIC's on different networks

2010-07-07 Thread Joshua Oreman
On Wed, Jul 7, 2010 at 8:35 AM, Gianni Tedesco wrote: > Hi, > > Without this patch retrieving boot image from the second "next-server" > and "filename" actually ends up using the results obtained from DHCP on > the first NIC after boot either fails or continues. I'm not convinced that this patch

[gPXE-devel] [PATCH] Fix autoboot for multiple NIC's on different networks

2010-07-07 Thread Gianni Tedesco
Hi, Without this patch retrieving boot image from the second "next-server" and "filename" actually ends up using the results obtained from DHCP on the first NIC after boot either fails or continues. Signed-off-by: Gianni Tedesco src/usr/autoboot.c |6 -- 1 files changed, 4 insertions(+

Re: [gPXE-devel] [PATCH][pcnet32] Fix pcnet32_wio_reset() bug

2010-07-07 Thread Stefan Hajnoczi
On Wed, Jul 7, 2010 at 1:43 PM, Andrei Faur wrote: > This bug caused .probe to fail because the NIC did not reset properly. Looks good. Stefan ___ gPXE-devel mailing list gPXE-devel@etherboot.org http://etherboot.org/mailman/listinfo/gpxe-devel

[gPXE-devel] [PATCH][pcnet32] Fix pcnet32_wio_reset() bug

2010-07-07 Thread Andrei Faur
This bug caused .probe to fail because the NIC did not reset properly. Signed-off-by: Andrei Faur --- src/drivers/net/pcnet32.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/drivers/net/pcnet32.c b/src/drivers/net/pcnet32.c index eeb7b72..a997f3c 100644 --- a/src/

Re: [gPXE-devel] [PULL] ipxe-sync

2010-07-07 Thread Piotr Jaroszyński
2010/7/7 Michael Brown : > On Tuesday 06 Jul 2010 22:09:01 Piotr Jaroszyński wrote: >> 2010/7/6 Piotr Jaroszyński : >> > While we are looking at tcp, there is also an access after free, which >> > is maybe fixed in [1]. Maybe because I am lazy and didn't look at the >> > RFC yet, so not sure whethe

Re: [gPXE-devel] [PULL] ipxe-sync

2010-07-07 Thread Michael Brown
On Tuesday 06 Jul 2010 22:09:01 Piotr Jaroszyński wrote: > 2010/7/6 Piotr Jaroszyński : > > While we are looking at tcp, there is also an access after free, which > > is maybe fixed in [1]. Maybe because I am lazy and didn't look at the > > RFC yet, so not sure whether moving the timestamp update i