From 4006d229e50204c93c1aa04c58385ce2e66d597e Mon Sep 17 00:00:00 2001 From: Alexey Zaytsev Date: Sat, 1 Dec 2007 07:07:01 +0300 Subject: Introduce the new timer subsystem. Timer subsystem initialization code in core/timer.c Split the BIOS and RTDSC timer drivers from i386_timer.c Split arch/i386/firmware/pcbios/bios.c into the RTSDC timer driver and arch/i386/core/nap.c Split the headers properly: include/unistd.h - delay functions to be used by the gPXE core and drivers. include/gpxe/timer.h - the fimer subsystem interface to be used by the timer drivers and currticks() to be used by the code gPXE subsystems. include/latch.h - removed include/timer.h - scheduled for removal. Some driver are using currticks, which is only for core subsystems. Signed-off-by: Alexey Zaytsev --- src/core/config.c | 11 ++++++ src/core/misc.c | 12 ------ src/core/timer.c | 107 ++++++++++++++++++++++++++++++++++++++++++++++-------- 3 files changed, 103 insertions(+), 27 deletions(-) (limited to 'src/core') diff --git a/src/core/config.c b/src/core/config.c index d5accecfe..f3e17b62e 100644 --- a/src/core/config.c +++ b/src/core/config.c @@ -74,6 +74,17 @@ REQUIRE_OBJECT ( pc_kbd ); REQUIRE_OBJECT ( syslog ); #endif +/* + * Timers + */ + +#ifdef TIMER_BIOS +REQUIRE_OBJECT ( timer_bios ); +#endif + +#ifdef TIMER_RTDSC +REQUIRE_OBJECT ( timer_rtdsc ); +#endif /* * Drag in all requested protocols * diff --git a/src/core/misc.c b/src/core/misc.c index 4219a36c5..fcf0aeab3 100644 --- a/src/core/misc.c +++ b/src/core/misc.c @@ -7,18 +7,6 @@ MISC Support Routines #include #include -/************************************************************************** -SLEEP -**************************************************************************/ -unsigned int sleep(unsigned int secs) -{ - unsigned long tmo; - - for (tmo = currticks()+secs*TICKS_PER_SEC; currticks() < tmo; ) { - } - return 0; -} - /************************************************************************** INET_ATON - Convert an ascii x.x.x.x to binary form **************************************************************************/ diff --git a/src/core/timer.c b/src/core/timer.c index c56e53109..da53e0539 100644 --- a/src/core/timer.c +++ b/src/core/timer.c @@ -1,27 +1,104 @@ -/* A couple of routines to implement a low-overhead timer for drivers */ - - /* +/* + * core/timer.c + * + * Copyright (C) 2007 Alexey Zaytsev + * * This program is free software; you can redistribute it and/or * modify it under the terms of the GNU General Public License as - * published by the Free Software Foundation; either version 2, or (at - * your option) any later version. + * published by the Free Software Foundation; either version 2 of the + * License, or any later version. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ -#include "timer.h" +#include +#include +#include +#include +#include -/* Machine Independant timer helper functions */ +static struct timer ts_table[0] + __table_start ( struct timer, timers ); +static struct timer ts_table_end[0] + __table_end ( struct timer, timers ); -void mdelay(unsigned int msecs) + +static struct timer *used_ts = NULL; + +/* + * This function may be used in custom timer driver. + * + * This udelay implementation works well if you've got a + * fast currticks(). + */ +void generic_currticks_udelay(unsigned int usecs) { - unsigned int i; - for(i = 0; i < msecs; i++) { - udelay(1000); - } + tick_t t; + + t = currticks(); + while (t + usecs > currticks()) + ; /* xxx: Relax the cpu some way. */ } -void waiton_timer2(unsigned int ticks) + +static void timer_init(void) { - load_timer2(ticks); - while(timer2_running()) { + struct timer *ts; + + for (ts = ts_table; ts < ts_table_end; ts++) { + if (ts->init && !ts->init()) { + used_ts = ts; + break; + } + } + + if (!used_ts) { + printf("No timer available. This should never happen. Expect gPXE to die soon.\n"); + /* Panic */ } + } + +struct init_fn ts_init_fn __init_fn ( INIT_NORMAL ) = { + .initialise = timer_init, +}; + +/* Functions for public use. */ + +tick_t currticks(void) +{ + tick_t ct; + assert(used_ts); + + ct = used_ts->currticks(); + DBG("currticks: %ld seconds and %06ld microseconds\n", ct/USECS_IN_SEC, ct%USECS_IN_SEC); + + return ct; +} + +void udelay(unsigned int usecs) +{ + used_ts->udelay(usecs); +} + +void mdelay(unsigned int msecs) +{ + while(msecs--) + used_ts->udelay(USECS_IN_MSEC); +} + +unsigned int sleep(unsigned int secs) +{ + while (secs--) + mdelay(MSECS_IN_SEC); + + return 0; +} + -- cgit v1.2.3-55-g7522 From a1572e0ab0f5fab9a1f0185ded593bec811aa175 Mon Sep 17 00:00:00 2001 From: Alexey Zaytsev Date: Sun, 2 Mar 2008 03:41:10 +0300 Subject: Modify gPXE core and drivers to work with the new timer subsystem Signed-off-by: Alexey Zaytsev --- src/arch/i386/Makefile | 1 + src/arch/i386/firmware/pcbios/gateA20.c | 7 +++---- src/arch/i386/include/latch.h | 12 ------------ src/core/getkey.c | 2 +- src/core/misc.c | 2 +- src/core/random.c | 2 +- src/core/serial.c | 2 +- src/drivers/bitbash/bitbash.c | 1 - src/drivers/bitbash/i2c_bit.c | 2 +- src/drivers/bitbash/spi_bit.c | 2 +- src/drivers/bus/eisa.c | 2 +- src/drivers/bus/isapnp.c | 2 +- src/drivers/bus/mca.c | 1 - src/drivers/infiniband/arbel.c | 1 - src/drivers/infiniband/hermon.c | 1 - src/drivers/net/3c509.c | 2 +- src/drivers/net/3c515.c | 1 - src/drivers/net/3c595.c | 1 - src/drivers/net/3c5x9.c | 1 - src/drivers/net/3c90x.c | 7 ++++--- src/drivers/net/amd8111e.c | 1 - src/drivers/net/bnx2.c | 1 - src/drivers/net/dmfe.c | 1 - src/drivers/net/e1000/e1000.h | 1 - src/drivers/net/e1000/e1000_osdep.h | 2 +- src/drivers/net/eepro.c | 1 - src/drivers/net/eepro100.c | 13 ++++++++----- src/drivers/net/epic100.c | 16 +++++++++------- src/drivers/net/etherfabric.c | 1 - src/drivers/net/forcedeth.c | 1 - src/drivers/net/ipoib.c | 1 - src/drivers/net/mlx_ipoib/mt23108_imp.c | 1 - src/drivers/net/mlx_ipoib/mt25218_imp.c | 1 - src/drivers/net/mtnic.c | 2 +- src/drivers/net/natsemi.c | 2 +- src/drivers/net/pcnet32.c | 1 - src/drivers/net/prism2.c | 1 - src/drivers/net/r8169.c | 1 - src/drivers/net/rtl8139.c | 2 +- src/drivers/net/sis900.c | 1 - src/drivers/net/smc9000.c | 1 - src/drivers/net/sundance.c | 1 - src/drivers/net/tg3.c | 1 - src/drivers/net/tlan.c | 1 - src/drivers/net/via-rhine.c | 21 ++++++++++++--------- src/drivers/net/via-velocity.h | 2 -- src/drivers/net/w89c840.c | 9 ++++----- src/drivers/nvs/spi.c | 2 +- src/drivers/nvs/threewire.c | 2 +- src/hci/mucurses/kb.c | 2 +- src/hci/shell_banner.c | 2 +- src/include/etherboot.h | 2 +- src/include/gpxe/dhcp.h | 1 - src/include/gpxe/tcp.h | 1 - src/include/timer.h | 27 --------------------------- src/net/retry.c | 2 +- src/net/tcp.c | 2 +- src/net/udp/dhcp.c | 1 + 58 files changed, 62 insertions(+), 121 deletions(-) delete mode 100644 src/arch/i386/include/latch.h delete mode 100644 src/include/timer.h (limited to 'src/core') diff --git a/src/arch/i386/Makefile b/src/arch/i386/Makefile index 5e7416cbb..da7976df7 100644 --- a/src/arch/i386/Makefile +++ b/src/arch/i386/Makefile @@ -7,6 +7,7 @@ ISOLINUX_BIN = /usr/lib/syslinux/isolinux.bin SRCDIRS += arch/i386/core arch/i386/transitions arch/i386/prefix SRCDIRS += arch/i386/firmware/pcbios SRCDIRS += arch/i386/image +SRCDIRS += arch/i386/drivers SRCDIRS += arch/i386/drivers/bus SRCDIRS += arch/i386/drivers/net SRCDIRS += arch/i386/drivers/disk diff --git a/src/arch/i386/firmware/pcbios/gateA20.c b/src/arch/i386/firmware/pcbios/gateA20.c index 66b4da18a..2caac8941 100644 --- a/src/arch/i386/firmware/pcbios/gateA20.c +++ b/src/arch/i386/firmware/pcbios/gateA20.c @@ -1,8 +1,7 @@ #include -#include "realmode.h" -#include "timer.h" -#include "latch.h" -#include "bios.h" +#include +#include +#include #define K_RDWR 0x60 /* keyboard data & cmds (read/write) */ #define K_STATUS 0x64 /* keyboard status */ diff --git a/src/arch/i386/include/latch.h b/src/arch/i386/include/latch.h deleted file mode 100644 index 5000d5826..000000000 --- a/src/arch/i386/include/latch.h +++ /dev/null @@ -1,12 +0,0 @@ -#ifndef LATCH_H -#define LATCH_H - -#define TICKS_PER_SEC 18 - -/* For different calibrators of the TSC move the declaration of - * sleep_latch and the definitions of it's length here... - */ - -extern unsigned long currticks ( void ); - -#endif /* LATCH_H */ diff --git a/src/core/getkey.c b/src/core/getkey.c index 71ec6cc1b..1551cf37c 100644 --- a/src/core/getkey.c +++ b/src/core/getkey.c @@ -17,9 +17,9 @@ */ #include -#include #include #include +#include /** @file * diff --git a/src/core/misc.c b/src/core/misc.c index fcf0aeab3..a54f5a108 100644 --- a/src/core/misc.c +++ b/src/core/misc.c @@ -4,8 +4,8 @@ MISC Support Routines #include #include -#include #include +#include /************************************************************************** INET_ATON - Convert an ascii x.x.x.x to binary form diff --git a/src/core/random.c b/src/core/random.c index e4a70d434..d34e763a6 100644 --- a/src/core/random.c +++ b/src/core/random.c @@ -5,7 +5,7 @@ */ #include -#include +#include static int32_t rnd_seed = 0; diff --git a/src/core/serial.c b/src/core/serial.c index f325bc45b..f6d0ecbb9 100644 --- a/src/core/serial.c +++ b/src/core/serial.c @@ -15,7 +15,7 @@ #include "console.h" #include #include "io.h" -#include "timer.h" +#include #include "config/serial.h" /* Set default values if none specified */ diff --git a/src/drivers/bitbash/bitbash.c b/src/drivers/bitbash/bitbash.c index ec94feeea..c6f935202 100644 --- a/src/drivers/bitbash/bitbash.c +++ b/src/drivers/bitbash/bitbash.c @@ -16,7 +16,6 @@ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ -#include #include /** @file diff --git a/src/drivers/bitbash/i2c_bit.c b/src/drivers/bitbash/i2c_bit.c index d8fbacc0e..a3af610b0 100644 --- a/src/drivers/bitbash/i2c_bit.c +++ b/src/drivers/bitbash/i2c_bit.c @@ -20,7 +20,7 @@ #include #include #include -#include +#include #include #include diff --git a/src/drivers/bitbash/spi_bit.c b/src/drivers/bitbash/spi_bit.c index a4e7136f8..ef87b5a24 100644 --- a/src/drivers/bitbash/spi_bit.c +++ b/src/drivers/bitbash/spi_bit.c @@ -22,7 +22,7 @@ #include #include #include -#include +#include #include #include diff --git a/src/drivers/bus/eisa.c b/src/drivers/bus/eisa.c index d9d02d727..ee03df3af 100644 --- a/src/drivers/bus/eisa.c +++ b/src/drivers/bus/eisa.c @@ -4,7 +4,7 @@ #include #include #include -#include +#include #include static struct eisa_driver eisa_drivers[0] diff --git a/src/drivers/bus/isapnp.c b/src/drivers/bus/isapnp.c index 7903208d0..f4968eb13 100644 --- a/src/drivers/bus/isapnp.c +++ b/src/drivers/bus/isapnp.c @@ -61,7 +61,7 @@ #include #include #include -#include +#include #include /** diff --git a/src/drivers/bus/mca.c b/src/drivers/bus/mca.c index 375a68344..eb7b7e395 100644 --- a/src/drivers/bus/mca.c +++ b/src/drivers/bus/mca.c @@ -11,7 +11,6 @@ #include #include #include -#include #include static struct mca_driver mca_drivers[0] diff --git a/src/drivers/infiniband/arbel.c b/src/drivers/infiniband/arbel.c index 147cf8b90..462638eae 100644 --- a/src/drivers/infiniband/arbel.c +++ b/src/drivers/infiniband/arbel.c @@ -26,7 +26,6 @@ #include #include #include -#include #include #include #include diff --git a/src/drivers/infiniband/hermon.c b/src/drivers/infiniband/hermon.c index fecf39882..c10559f98 100644 --- a/src/drivers/infiniband/hermon.c +++ b/src/drivers/infiniband/hermon.c @@ -24,7 +24,6 @@ #include #include #include -#include #include #include #include diff --git a/src/drivers/net/3c509.c b/src/drivers/net/3c509.c index e704cfca1..8a15aff29 100644 --- a/src/drivers/net/3c509.c +++ b/src/drivers/net/3c509.c @@ -9,7 +9,7 @@ #include #include #include -#include +#include #include #include #include "3c509.h" diff --git a/src/drivers/net/3c515.c b/src/drivers/net/3c515.c index d0bf3189e..02f03dc78 100644 --- a/src/drivers/net/3c515.c +++ b/src/drivers/net/3c515.c @@ -50,7 +50,6 @@ #include "nic.h" #include #include /* for ISA_ROM */ -#include "timer.h" #include static void t3c515_wait(unsigned int nticks) diff --git a/src/drivers/net/3c595.c b/src/drivers/net/3c595.c index 572a99e0b..7138f936e 100644 --- a/src/drivers/net/3c595.c +++ b/src/drivers/net/3c595.c @@ -30,7 +30,6 @@ #include #include #include "3c595.h" -#include "timer.h" static struct nic_operations t595_operations; diff --git a/src/drivers/net/3c5x9.c b/src/drivers/net/3c5x9.c index 1ade6f47d..565044a11 100644 --- a/src/drivers/net/3c5x9.c +++ b/src/drivers/net/3c5x9.c @@ -28,7 +28,6 @@ $Id$ #include "etherboot.h" #include "nic.h" #include -#include "timer.h" #include "3c509.h" static enum { none, bnc, utp } connector = none; /* for 3C509 */ diff --git a/src/drivers/net/3c90x.c b/src/drivers/net/3c90x.c index 8e1f160ea..8158239d6 100644 --- a/src/drivers/net/3c90x.c +++ b/src/drivers/net/3c90x.c @@ -43,7 +43,6 @@ #include "nic.h" #include #include -#include "timer.h" static struct nic_operations a3c90x_operations; @@ -498,6 +497,7 @@ a3c90x_transmit(struct nic *nic __unused, const char *d, unsigned int t, unsigned char status; unsigned i, retries; + tick_t ct; for (retries=0; retries < XMIT_RETRIES ; retries++) { @@ -540,9 +540,10 @@ a3c90x_transmit(struct nic *nic __unused, const char *d, unsigned int t, ; /** Wait for NIC Transmit to Complete **/ - load_timer2(10*TICKS_PER_MS); /* Give it 10 ms */ + ct = currticks(); + while (!(inw(INF_3C90X.IOAddr + regCommandIntStatus_w)&0x0004) && - timer2_running()) + ct + 10*USECS_IN_MSEC < currticks()); ; if (!(inw(INF_3C90X.IOAddr + regCommandIntStatus_w)&0x0004)) diff --git a/src/drivers/net/amd8111e.c b/src/drivers/net/amd8111e.c index c26c07cf0..1c41add12 100644 --- a/src/drivers/net/amd8111e.c +++ b/src/drivers/net/amd8111e.c @@ -33,7 +33,6 @@ #include "mii.h" #include #include -#include "timer.h" #include "string.h" #include "stdint.h" #include "amd8111e.h" diff --git a/src/drivers/net/bnx2.c b/src/drivers/net/bnx2.c index fdd6655bb..9c989c4c1 100644 --- a/src/drivers/net/bnx2.c +++ b/src/drivers/net/bnx2.c @@ -18,7 +18,6 @@ #include #include #include -#include "timer.h" #include "string.h" #include "bnx2.h" #include "bnx2_fw.h" diff --git a/src/drivers/net/dmfe.c b/src/drivers/net/dmfe.c index 34e211d4b..9cf50418d 100644 --- a/src/drivers/net/dmfe.c +++ b/src/drivers/net/dmfe.c @@ -43,7 +43,6 @@ /* to get the PCI support functions, if this is a PCI NIC */ #include #include -#include "timer.h" /* #define EDEBUG 1 */ #ifdef EDEBUG diff --git a/src/drivers/net/e1000/e1000.h b/src/drivers/net/e1000/e1000.h index b4bc1d874..4ae414513 100644 --- a/src/drivers/net/e1000/e1000.h +++ b/src/drivers/net/e1000/e1000.h @@ -36,7 +36,6 @@ #include #include #include -#include #include #include #include diff --git a/src/drivers/net/e1000/e1000_osdep.h b/src/drivers/net/e1000/e1000_osdep.h index 04e148927..7df9b5e9d 100644 --- a/src/drivers/net/e1000/e1000_osdep.h +++ b/src/drivers/net/e1000/e1000_osdep.h @@ -39,7 +39,7 @@ #include #include #include -#include +#include #include #include #include diff --git a/src/drivers/net/eepro.c b/src/drivers/net/eepro.c index 0a9c95e65..2a163d1b3 100644 --- a/src/drivers/net/eepro.c +++ b/src/drivers/net/eepro.c @@ -35,7 +35,6 @@ has 34 pins, the top row of 2 are not used. #include #include "nic.h" #include -#include "timer.h" #include /* Different 82595 chips */ diff --git a/src/drivers/net/eepro100.c b/src/drivers/net/eepro100.c index bb46d163c..f746976ad 100644 --- a/src/drivers/net/eepro100.c +++ b/src/drivers/net/eepro100.c @@ -107,7 +107,6 @@ #include "nic.h" #include #include -#include "timer.h" static int ioaddr; @@ -408,6 +407,7 @@ static void eepro100_transmit(struct nic *nic, const char *d, unsigned int t, un } hdr; unsigned short status; int s1, s2; + tick_t ct; status = inw(ioaddr + SCBStatus); /* Acknowledge all of the current interrupt sources ASAP. */ @@ -445,8 +445,10 @@ static void eepro100_transmit(struct nic *nic, const char *d, unsigned int t, un wait_for_cmd_done(ioaddr + SCBCmd); s1 = inw (ioaddr + SCBStatus); - load_timer2(10*TICKS_PER_MS); /* timeout 10 ms for transmit */ - while (!txfd.status && timer2_running()) + + ct = currticks(); + /* timeout 10 ms for transmit */ + while (!txfd.status && ct + 10*USECS_IN_MSEC) /* Wait */; s2 = inw (ioaddr + SCBStatus); @@ -606,6 +608,7 @@ static int eepro100_probe ( struct nic *nic, struct pci_device *pci ) { int read_cmd, ee_size; int options; int rx_mode; + tick_t ct; /* we cache only the first few words of the EEPROM data be careful not to access beyond this array */ @@ -749,8 +752,8 @@ static int eepro100_probe ( struct nic *nic, struct pci_device *pci ) { whereami ("started TX thingy (config, iasetup)."); - load_timer2(10*TICKS_PER_MS); - while (!txfd.status && timer2_running()) + ct = currticks(); + while (!txfd.status && ct + 10*USECS_IN_MSEC < currticks()) /* Wait */; /* Read the status register once to disgard stale data */ diff --git a/src/drivers/net/epic100.c b/src/drivers/net/epic100.c index e79da2b9b..67b4f0fb4 100644 --- a/src/drivers/net/epic100.c +++ b/src/drivers/net/epic100.c @@ -8,7 +8,6 @@ #include #include #include "nic.h" -#include "timer.h" #include "console.h" #include "epic100.h" @@ -310,6 +309,7 @@ epic100_transmit(struct nic *nic, const char *destaddr, unsigned int type, unsigned short nstype; unsigned char *txp; int entry; + tick_t ct; /* Calculate the next Tx descriptor entry. */ entry = cur_tx % TX_RING_SIZE; @@ -339,18 +339,20 @@ epic100_transmit(struct nic *nic, const char *destaddr, unsigned int type, * set the base address with the "ownership" * bits last. */ - + tx_ring[entry].buflength |= cpu_to_le32(len); - tx_ring[entry].status = cpu_to_le32(len << 16) | + tx_ring[entry].status = cpu_to_le32(len << 16) | cpu_to_le32(TRING_OWN); /* Pass ownership to the chip. */ cur_tx++; /* Trigger an immediate transmit demand. */ - outl(CR_QUEUE_TX, command); - - load_timer2(10*TICKS_PER_MS); /* timeout 10 ms for transmit */ - while ((le32_to_cpu(tx_ring[entry].status) & (TRING_OWN)) && timer2_running()) + outl(CR_QUEUE_TX, command); + + ct = currticks(); + /* timeout 10 ms for transmit */ + while ((le32_to_cpu(tx_ring[entry].status) & (TRING_OWN)) && + ct + 10*USECS_IN_MSEC < currticks()) /* Wait */; if ((le32_to_cpu(tx_ring[entry].status) & TRING_OWN) != 0) diff --git a/src/drivers/net/etherfabric.c b/src/drivers/net/etherfabric.c index 6b7634516..90cc73f4b 100644 --- a/src/drivers/net/etherfabric.c +++ b/src/drivers/net/etherfabric.c @@ -24,7 +24,6 @@ #include #include #include -#include "timer.h" #define dma_addr_t unsigned long #include "etherfabric.h" diff --git a/src/drivers/net/forcedeth.c b/src/drivers/net/forcedeth.c index 9ef1dd793..f61958852 100644 --- a/src/drivers/net/forcedeth.c +++ b/src/drivers/net/forcedeth.c @@ -51,7 +51,6 @@ #include /* Include timer support functions */ #include -#include "timer.h" #include "mii.h" #define drv_version "v1.2" diff --git a/src/drivers/net/ipoib.c b/src/drivers/net/ipoib.c index 905051bfa..d457b258a 100644 --- a/src/drivers/net/ipoib.c +++ b/src/drivers/net/ipoib.c @@ -22,7 +22,6 @@ #include #include #include -#include "timer.h" #include #include #include diff --git a/src/drivers/net/mlx_ipoib/mt23108_imp.c b/src/drivers/net/mlx_ipoib/mt23108_imp.c index 4e601668c..bb2383c5f 100644 --- a/src/drivers/net/mlx_ipoib/mt23108_imp.c +++ b/src/drivers/net/mlx_ipoib/mt23108_imp.c @@ -37,7 +37,6 @@ static void be_to_cpu_buf(void *buf, int size) } } -#include "timer.h" #include "cmdif_mt23108.c" #include "cmdif_comm.c" #include "ib_mt23108.c" diff --git a/src/drivers/net/mlx_ipoib/mt25218_imp.c b/src/drivers/net/mlx_ipoib/mt25218_imp.c index fe407041e..007ee653c 100644 --- a/src/drivers/net/mlx_ipoib/mt25218_imp.c +++ b/src/drivers/net/mlx_ipoib/mt25218_imp.c @@ -37,7 +37,6 @@ static void be_to_cpu_buf(void *buf, int size) } } -#include "timer.h" #include "cmdif_mt25218.c" #include "cmdif_comm.c" #include "ib_mt25218.c" diff --git a/src/drivers/net/mtnic.c b/src/drivers/net/mtnic.c index a568b2f7e..536fcb8d7 100755 --- a/src/drivers/net/mtnic.c +++ b/src/drivers/net/mtnic.c @@ -37,7 +37,7 @@ #include #include #include -#include +#include #include #include #include diff --git a/src/drivers/net/natsemi.c b/src/drivers/net/natsemi.c index 720476109..98a5ff848 100644 --- a/src/drivers/net/natsemi.c +++ b/src/drivers/net/natsemi.c @@ -63,8 +63,8 @@ #include #include #include -#include #include +#include #include #include #include diff --git a/src/drivers/net/pcnet32.c b/src/drivers/net/pcnet32.c index e4318429a..d8529da17 100644 --- a/src/drivers/net/pcnet32.c +++ b/src/drivers/net/pcnet32.c @@ -43,7 +43,6 @@ #include "nic.h" #include #include -#include "timer.h" #include "mii.h" /* void hex_dump(const char *data, const unsigned int len); */ diff --git a/src/drivers/net/prism2.c b/src/drivers/net/prism2.c index 3c5125192..aaf4c968f 100644 --- a/src/drivers/net/prism2.c +++ b/src/drivers/net/prism2.c @@ -62,7 +62,6 @@ static const char hardcoded_ssid[] = ""; * quicker to convert code from the Linux Prism2 driver. */ #include -#include "timer.h" #define __le16_to_cpu(x) (x) #define __le32_to_cpu(x) (x) #define __cpu_to_le16(x) (x) diff --git a/src/drivers/net/r8169.c b/src/drivers/net/r8169.c index 52cb0d0e4..ab9a30f88 100644 --- a/src/drivers/net/r8169.c +++ b/src/drivers/net/r8169.c @@ -49,7 +49,6 @@ #include #include #include -#include "timer.h" #define drv_version "v1.6" #define drv_date "03-27-2004" diff --git a/src/drivers/net/rtl8139.c b/src/drivers/net/rtl8139.c index b4c99fcbd..83fac554a 100644 --- a/src/drivers/net/rtl8139.c +++ b/src/drivers/net/rtl8139.c @@ -71,7 +71,7 @@ #include #include #include -#include +#include #include #include #include diff --git a/src/drivers/net/sis900.c b/src/drivers/net/sis900.c index cf9b2ce6a..585265063 100644 --- a/src/drivers/net/sis900.c +++ b/src/drivers/net/sis900.c @@ -46,7 +46,6 @@ #include "etherboot.h" #include #include "nic.h" -#include "timer.h" #include "sis900.h" diff --git a/src/drivers/net/smc9000.c b/src/drivers/net/smc9000.c index c6219ee74..31a1e1b11 100644 --- a/src/drivers/net/smc9000.c +++ b/src/drivers/net/smc9000.c @@ -40,7 +40,6 @@ #include "etherboot.h" #include "nic.h" #include -#include "timer.h" #include "smc9000.h" # define _outb outb diff --git a/src/drivers/net/sundance.c b/src/drivers/net/sundance.c index 3bbfbfbbd..3b6f5a88b 100644 --- a/src/drivers/net/sundance.c +++ b/src/drivers/net/sundance.c @@ -46,7 +46,6 @@ #include "nic.h" /* to get the PCI support functions, if this is a PCI NIC */ #include -#include "timer.h" #include "mii.h" #define drv_version "v1.12" diff --git a/src/drivers/net/tg3.c b/src/drivers/net/tg3.c index 2188def42..57b9df1e0 100644 --- a/src/drivers/net/tg3.c +++ b/src/drivers/net/tg3.c @@ -17,7 +17,6 @@ #include #include #include -#include "timer.h" #include "string.h" #include "tg3.h" diff --git a/src/drivers/net/tlan.c b/src/drivers/net/tlan.c index 60ca182e6..4fae01703 100644 --- a/src/drivers/net/tlan.c +++ b/src/drivers/net/tlan.c @@ -42,7 +42,6 @@ #include "nic.h" #include #include -#include "timer.h" #include "tlan.h" #define drv_version "v1.4" diff --git a/src/drivers/net/via-rhine.c b/src/drivers/net/via-rhine.c index 0c955fab3..81350091b 100644 --- a/src/drivers/net/via-rhine.c +++ b/src/drivers/net/via-rhine.c @@ -50,7 +50,6 @@ static const char *version = "rhine.c v1.0.2 2004-10-29\n"; #include "nic.h" #include #include -#include "timer.h" /* define all ioaddr */ @@ -785,6 +784,7 @@ ReadMII (int byMIIIndex, int ioaddr) char byMIIAdrbak; char byMIICRbak; char byMIItemp; + tick_t ct; byMIIAdrbak = inb (byMIIAD); byMIICRbak = inb (byMIICR); @@ -799,8 +799,8 @@ ReadMII (int byMIIIndex, int ioaddr) byMIItemp = inb (byMIICR); byMIItemp = byMIItemp & 0x40; - load_timer2(2*TICKS_PER_MS); - while (byMIItemp != 0 && timer2_running()) + ct = currticks(); + while (byMIItemp != 0 && ct + 2*USECS_IN_MSEC < currticks()) { byMIItemp = inb (byMIICR); byMIItemp = byMIItemp & 0x40; @@ -825,6 +825,7 @@ WriteMII (char byMIISetByte, char byMIISetBit, char byMIIOP, int ioaddr) char byMIIAdrbak; char byMIICRbak; char byMIItemp; + tick_t ct; byMIIAdrbak = inb (byMIIAD); @@ -840,8 +841,8 @@ WriteMII (char byMIISetByte, char byMIISetBit, char byMIIOP, int ioaddr) byMIItemp = inb (byMIICR); byMIItemp = byMIItemp & 0x40; - load_timer2(2*TICKS_PER_MS); - while (byMIItemp != 0 && timer2_running()) + ct = currticks(); + while (byMIItemp != 0 && ct + 2*USECS_IN_MSEC < currticks()) { byMIItemp = inb (byMIICR); byMIItemp = byMIItemp & 0x40; @@ -870,8 +871,8 @@ WriteMII (char byMIISetByte, char byMIISetBit, char byMIIOP, int ioaddr) byMIItemp = inb (byMIICR); byMIItemp = byMIItemp & 0x20; - load_timer2(2*TICKS_PER_MS); - while (byMIItemp != 0 && timer2_running()) + ct = currticks(); + while (byMIItemp != 0 && ct + 2*USECS_IN_MSEC < currticks()) { byMIItemp = inb (byMIICR); byMIItemp = byMIItemp & 0x20; @@ -1345,6 +1346,7 @@ rhine_transmit (struct nic *nic, unsigned char CR1bak; unsigned char CR0bak; unsigned int nstype; + tick_t ct; /*printf ("rhine_transmit\n"); */ @@ -1385,9 +1387,10 @@ rhine_transmit (struct nic *nic, outb (CR1bak, byCR1); do { - load_timer2(10*TICKS_PER_MS); + ct = currticks(); /* Wait until transmit is finished or timeout*/ - while((tp->tx_ring[entry].tx_status.bits.own_bit !=0) && timer2_running()) + while((tp->tx_ring[entry].tx_status.bits.own_bit !=0) && + ct + 10*USECS_IN_MSEC < currticks()) ; if(tp->tx_ring[entry].tx_status.bits.terr == 0) diff --git a/src/drivers/net/via-velocity.h b/src/drivers/net/via-velocity.h index c296d2899..a43028bd9 100644 --- a/src/drivers/net/via-velocity.h +++ b/src/drivers/net/via-velocity.h @@ -24,8 +24,6 @@ * Copyright (c) 2006 by Timothy Legge */ -#include "timer.h" - #ifndef VELOCITY_H #define VELOCITY_H diff --git a/src/drivers/net/w89c840.c b/src/drivers/net/w89c840.c index 2f141aa98..144976404 100644 --- a/src/drivers/net/w89c840.c +++ b/src/drivers/net/w89c840.c @@ -81,7 +81,6 @@ #include "nic.h" #include #include -#include "timer.h" static const char *w89c840_version = "driver Version 0.94 - December 12, 2003"; @@ -113,7 +112,7 @@ static const char *w89c840_version = "driver Version 0.94 - December 12, 2003"; /* Operational parameters that usually are not changed. */ /* Time in jiffies before concluding the transmitter is hung. */ -#define TX_TIMEOUT (10*TICKS_PER_MS) +#define TX_TIMEOUT (10*USECS_IN_MSEC) #define PKT_BUF_SZ 1536 /* Size of each temporary Rx buffer.*/ @@ -487,6 +486,7 @@ static void w89c840_transmit( /* send the packet to destination */ unsigned entry; int transmit_status; + tick_t ct; /* Caution: the write order is important here, set the field with the "ownership" bits last. */ @@ -535,8 +535,7 @@ static void w89c840_transmit( /* Now wait for TX to complete. */ transmit_status = w840private.tx_ring[entry].status; - load_timer2(TX_TIMEOUT); - + ct = currticks(); { #if defined W89C840_DEBUG u32 intr_stat = 0; @@ -547,7 +546,7 @@ static void w89c840_transmit( decode_interrupt(intr_stat); #endif - while ( (transmit_status & DescOwn) && timer2_running()) { + while ( (transmit_status & DescOwn) && ct + TX_TIMEOUT < currticks()) { transmit_status = w840private.tx_ring[entry].status; } diff --git a/src/drivers/nvs/spi.c b/src/drivers/nvs/spi.c index be6b5904f..dbdc32f61 100644 --- a/src/drivers/nvs/spi.c +++ b/src/drivers/nvs/spi.c @@ -18,7 +18,7 @@ #include #include -#include +#include #include /** @file diff --git a/src/drivers/nvs/threewire.c b/src/drivers/nvs/threewire.c index 622e85857..dbecf9099 100644 --- a/src/drivers/nvs/threewire.c +++ b/src/drivers/nvs/threewire.c @@ -18,7 +18,7 @@ #include #include -#include +#include #include /** @file diff --git a/src/hci/mucurses/kb.c b/src/hci/mucurses/kb.c index 291ee6ac8..cada72917 100644 --- a/src/hci/mucurses/kb.c +++ b/src/hci/mucurses/kb.c @@ -1,6 +1,6 @@ #include #include -#include +#include #include "mucurses.h" /** @file diff --git a/src/hci/shell_banner.c b/src/hci/shell_banner.c index d313d4dd0..62da487c7 100644 --- a/src/hci/shell_banner.c +++ b/src/hci/shell_banner.c @@ -18,8 +18,8 @@ #include #include -#include #include +#include #include /** @file diff --git a/src/include/etherboot.h b/src/include/etherboot.h index 8d658194f..2a465954f 100644 --- a/src/include/etherboot.h +++ b/src/include/etherboot.h @@ -12,8 +12,8 @@ #include #include #include -#include #include +#include #include #include diff --git a/src/include/gpxe/dhcp.h b/src/include/gpxe/dhcp.h index 9a9ba7484..6db0e026c 100644 --- a/src/include/gpxe/dhcp.h +++ b/src/include/gpxe/dhcp.h @@ -12,7 +12,6 @@ #include #include #include -#include struct net_device; struct job_interface; diff --git a/src/include/gpxe/tcp.h b/src/include/gpxe/tcp.h index e2753120c..264ec29b4 100644 --- a/src/include/gpxe/tcp.h +++ b/src/include/gpxe/tcp.h @@ -9,7 +9,6 @@ * */ -#include "latch.h" #include /** diff --git a/src/include/timer.h b/src/include/timer.h deleted file mode 100644 index cd1b2821a..000000000 --- a/src/include/timer.h +++ /dev/null @@ -1,27 +0,0 @@ -#ifndef TIMER_H -#define TIMER_H - -/* - * This file should be removed as soon as there are no - * currticks() abusers. - */ - -#include -/* -#warning Please fix me. I'm abusing the deprecated include/timer.h -*/ -#include - -/* Duplicates include/gpxe/timer.h */ -typedef uint32_t tick_t; - -#define MSECS_IN_SEC (1000) -#define USECS_IN_SEC (1000*1000) -#define USECS_IN_MSEC (1000) - -#define TICKS_PER_SEC USECS_IN_SEC - -tick_t currticks(void); - -#endif - diff --git a/src/net/retry.c b/src/net/retry.c index 0f711e6d1..90b897112 100644 --- a/src/net/retry.c +++ b/src/net/retry.c @@ -17,7 +17,7 @@ */ #include -#include +#include #include #include #include diff --git a/src/net/tcp.c b/src/net/tcp.c index d6b44b02b..da8e87b4b 100644 --- a/src/net/tcp.c +++ b/src/net/tcp.c @@ -4,7 +4,7 @@ #include #include #include -#include +#include #include #include #include diff --git a/src/net/udp/dhcp.c b/src/net/udp/dhcp.c index 081fa841a..2cb258e43 100644 --- a/src/net/udp/dhcp.c +++ b/src/net/udp/dhcp.c @@ -33,6 +33,7 @@ #include #include #include +#include /** @file * -- cgit v1.2.3-55-g7522 From 379c37bafba794b47afcb1e9970b6207fb0eb9f4 Mon Sep 17 00:00:00 2001 From: Alexey Zaytsev Date: Sun, 2 Mar 2008 04:36:50 +0300 Subject: Cleanups Replace a printf with a DBG in timer_rtdsc.c Replace a printf in timer.c with assert Return proper error codes from timer drivers Signed-off-by: Alexey Zaytsev --- src/arch/i386/drivers/timer_rtdsc.c | 5 +++-- src/arch/i386/include/bits/errfile.h | 3 +++ src/core/timer.c | 8 ++------ 3 files changed, 8 insertions(+), 8 deletions(-) (limited to 'src/core') diff --git a/src/arch/i386/drivers/timer_rtdsc.c b/src/arch/i386/drivers/timer_rtdsc.c index 1cd2abead..336e3e6fc 100644 --- a/src/arch/i386/drivers/timer_rtdsc.c +++ b/src/arch/i386/drivers/timer_rtdsc.c @@ -1,6 +1,7 @@ #include #include +#include #include #include #include @@ -78,8 +79,8 @@ static int rtdsc_ts_init(void) } } - printf("RTDSC timer not available on this machine.\n"); - return 1; + DBG("RTDSC timer not available on this machine.\n"); + return -ENODEV; } struct timer rtdsc_ts __timer (01) = { diff --git a/src/arch/i386/include/bits/errfile.h b/src/arch/i386/include/bits/errfile.h index a6f878254..ce58eefd0 100644 --- a/src/arch/i386/include/bits/errfile.h +++ b/src/arch/i386/include/bits/errfile.h @@ -26,6 +26,9 @@ #define ERRFILE_undionly ( ERRFILE_ARCH | ERRFILE_NET | 0x00030000 ) #define ERRFILE_undirom ( ERRFILE_ARCH | ERRFILE_NET | 0x00040000 ) +#define ERRFILE_timer_rtdsc ( ERRFILE_ARCH | ERRFILE_DRIVER | 0x00000000 ) +#define ERRFILE_timer_bios ( ERRFILE_ARCH | ERRFILE_DRIVER | 0x00010000 ) + /** @} */ #endif /* _BITS_ERRFILE_H */ diff --git a/src/core/timer.c b/src/core/timer.c index da53e0539..e736f5285 100644 --- a/src/core/timer.c +++ b/src/core/timer.c @@ -53,17 +53,13 @@ static void timer_init(void) struct timer *ts; for (ts = ts_table; ts < ts_table_end; ts++) { - if (ts->init && !ts->init()) { + if (ts->init && ts->init() >= 0) { used_ts = ts; break; } } - if (!used_ts) { - printf("No timer available. This should never happen. Expect gPXE to die soon.\n"); - /* Panic */ - } - + assert(used_ts); } struct init_fn ts_init_fn __init_fn ( INIT_NORMAL ) = { -- cgit v1.2.3-55-g7522 From 1935439f863a43986ce386c830f4824d10c4cc18 Mon Sep 17 00:00:00 2001 From: Alexey Zaytsev Date: Sun, 2 Mar 2008 05:19:29 +0300 Subject: fix the rdtsc namimg --- src/arch/i386/drivers/timer_rdtsc.c | 91 ++++++++++++++++++++++++++++++++++++ src/arch/i386/drivers/timer_rtdsc.c | 91 ------------------------------------ src/arch/i386/include/bits/errfile.h | 2 +- src/config.h | 2 +- src/core/config.c | 4 +- 5 files changed, 95 insertions(+), 95 deletions(-) create mode 100644 src/arch/i386/drivers/timer_rdtsc.c delete mode 100644 src/arch/i386/drivers/timer_rtdsc.c (limited to 'src/core') diff --git a/src/arch/i386/drivers/timer_rdtsc.c b/src/arch/i386/drivers/timer_rdtsc.c new file mode 100644 index 000000000..57b8826c7 --- /dev/null +++ b/src/arch/i386/drivers/timer_rdtsc.c @@ -0,0 +1,91 @@ + +#include +#include +#include +#include +#include +#include +#include + + +#define rdtsc(low,high) \ + __asm__ __volatile__("rdtsc" : "=a" (low), "=d" (high)) + +#define rdtscll(val) \ + __asm__ __volatile__ ("rdtsc" : "=A" (val)) + +static unsigned long long calibrate_tsc(void) +{ + uint32_t startlow, starthigh; + uint32_t endlow, endhigh; + + rdtsc(startlow,starthigh); + i386_timer2_udelay(USECS_IN_MSEC/2); + rdtsc(endlow,endhigh); + + /* 64-bit subtract - gcc just messes up with long longs */ + /* XXX ORLY? Check it. */ + __asm__("subl %2,%0\n\t" + "sbbl %3,%1" + :"=a" (endlow), "=d" (endhigh) + :"g" (startlow), "g" (starthigh), + "0" (endlow), "1" (endhigh)); + + /* Error: ECPUTOOFAST */ + if (endhigh) + goto bad_ctc; + + endlow *= MSECS_IN_SEC*2; + return endlow; + + /* + * The CTC wasn't reliable: we got a hit on the very first read, + * or the CPU was so fast/slow that the quotient wouldn't fit in + * 32 bits.. + */ +bad_ctc: + return 0; +} +static uint32_t clocks_per_second = 0; + +static tick_t rdtsc_currticks(void) +{ + uint32_t clocks_high, clocks_low; + uint32_t currticks; + + /* Read the Time Stamp Counter */ + rdtsc(clocks_low, clocks_high); + + /* currticks = clocks / clocks_per_tick; */ + __asm__("divl %1" + :"=a" (currticks) + :"r" (clocks_per_second/USECS_IN_SEC), "0" (clocks_low), "d" (clocks_high)); + + return currticks; +} + +static int rdtsc_ts_init(void) +{ + + struct cpuinfo_x86 cpu_info; + + get_cpuinfo(&cpu_info); + if (cpu_info.features & X86_FEATURE_TSC) { + clocks_per_second = calibrate_tsc(); + if (clocks_per_second) { + DBG("RDTSC ticksource installed. CPU running at %ld Mhz\n", + clocks_per_second/(1000*1000)); + return 0; + } + } + + DBG("RDTSC ticksource not available on this machine.\n"); + return -ENODEV; +} + +struct timer rdtsc_ts __timer (01) = { + .init = rdtsc_ts_init, + .udelay = generic_currticks_udelay, + .currticks = rdtsc_currticks, +}; + diff --git a/src/arch/i386/drivers/timer_rtdsc.c b/src/arch/i386/drivers/timer_rtdsc.c deleted file mode 100644 index 336e3e6fc..000000000 --- a/src/arch/i386/drivers/timer_rtdsc.c +++ /dev/null @@ -1,91 +0,0 @@ - -#include -#include -#include -#include -#include -#include -#include - - -#define rdtsc(low,high) \ - __asm__ __volatile__("rdtsc" : "=a" (low), "=d" (high)) - -#define rdtscll(val) \ - __asm__ __volatile__ ("rdtsc" : "=A" (val)) - -static unsigned long long calibrate_tsc(void) -{ - uint32_t startlow, starthigh; - uint32_t endlow, endhigh; - - rdtsc(startlow,starthigh); - i386_timer2_udelay(USECS_IN_MSEC/2); - rdtsc(endlow,endhigh); - - /* 64-bit subtract - gcc just messes up with long longs */ - /* XXX ORLY? Check it. */ - __asm__("subl %2,%0\n\t" - "sbbl %3,%1" - :"=a" (endlow), "=d" (endhigh) - :"g" (startlow), "g" (starthigh), - "0" (endlow), "1" (endhigh)); - - /* Error: ECPUTOOFAST */ - if (endhigh) - goto bad_ctc; - - endlow *= MSECS_IN_SEC*2; - return endlow; - - /* - * The CTC wasn't reliable: we got a hit on the very first read, - * or the CPU was so fast/slow that the quotient wouldn't fit in - * 32 bits.. - */ -bad_ctc: - return 0; -} -static uint32_t clocks_per_second = 0; - -static tick_t rtdsc_currticks(void) -{ - uint32_t clocks_high, clocks_low; - uint32_t currticks; - - /* Read the Time Stamp Counter */ - rdtsc(clocks_low, clocks_high); - - /* currticks = clocks / clocks_per_tick; */ - __asm__("divl %1" - :"=a" (currticks) - :"r" (clocks_per_second/USECS_IN_SEC), "0" (clocks_low), "d" (clocks_high)); - - return currticks; -} - -static int rtdsc_ts_init(void) -{ - - struct cpuinfo_x86 cpu_info; - - get_cpuinfo(&cpu_info); - if (cpu_info.features & X86_FEATURE_TSC) { - clocks_per_second = calibrate_tsc(); - if (clocks_per_second) { - DBG("RTDSC Ticksource installed. CPU running at %ld Mhz\n", - clocks_per_second/(1000*1000)); - return 0; - } - } - - DBG("RTDSC timer not available on this machine.\n"); - return -ENODEV; -} - -struct timer rtdsc_ts __timer (01) = { - .init = rtdsc_ts_init, - .udelay = generic_currticks_udelay, - .currticks = rtdsc_currticks, -}; - diff --git a/src/arch/i386/include/bits/errfile.h b/src/arch/i386/include/bits/errfile.h index ce58eefd0..0f1402148 100644 --- a/src/arch/i386/include/bits/errfile.h +++ b/src/arch/i386/include/bits/errfile.h @@ -26,7 +26,7 @@ #define ERRFILE_undionly ( ERRFILE_ARCH | ERRFILE_NET | 0x00030000 ) #define ERRFILE_undirom ( ERRFILE_ARCH | ERRFILE_NET | 0x00040000 ) -#define ERRFILE_timer_rtdsc ( ERRFILE_ARCH | ERRFILE_DRIVER | 0x00000000 ) +#define ERRFILE_timer_rdtsc ( ERRFILE_ARCH | ERRFILE_DRIVER | 0x00000000 ) #define ERRFILE_timer_bios ( ERRFILE_ARCH | ERRFILE_DRIVER | 0x00010000 ) /** @} */ diff --git a/src/config.h b/src/config.h index af0df1b69..4aac654a2 100644 --- a/src/config.h +++ b/src/config.h @@ -24,7 +24,7 @@ #undef CONSOLE_PC_KBD /* Direct access to PC keyboard */ #define TIMER_BIOS -#define TIMER_RTDSC +#define TIMER_RDTSC /* @END general.h */ diff --git a/src/core/config.c b/src/core/config.c index f3e17b62e..94fd0b82c 100644 --- a/src/core/config.c +++ b/src/core/config.c @@ -82,8 +82,8 @@ REQUIRE_OBJECT ( syslog ); REQUIRE_OBJECT ( timer_bios ); #endif -#ifdef TIMER_RTDSC -REQUIRE_OBJECT ( timer_rtdsc ); +#ifdef TIMER_RDTSC +REQUIRE_OBJECT ( timer_rdtsc ); #endif /* * Drag in all requested protocols -- cgit v1.2.3-55-g7522