Помогите с отладкой

[K_E_S_T]

Спаял себе индикатор загрузки ЦП. 7 светодиодов, подключенных к разным линиям LPT порта. Написал "сервис" который управляет самодельным девайсом.
Запускаю с правами рута, всё работает. только через некоторое (рандомное) время вылетает с ошибкой "Floating point exception". Помогите победить этот злой баг. Советы по общей оптимизации кода тоже принимаются

(Нажмите, чтобы показать/скрыть)

#include <iostream>
#include "/usr/include/sys/io.h"

#define BASEPORT 0x378 /* lp1 */


int main(int argc, char** argv)
{
  int load;
  int cpu;
 
  if (ioperm(BASEPORT, 3, 1)) {perror("ioperm"); return(1);}
 
  FILE* stat;
  long int u0 =1;
  long int n0=1;
  long int s0=1;
  long int i0=1;
  long int u1=1;
  long int n1=1;
  long int s1=1;
  long int i1=1;
  long int ud=1;
  long int nd=1;
  long int sd=1;
  long int id=1;
  long int tot=1;
  printf("CPU Loading on LPT Service");
re:stat = fopen("/proc/stat", "r");
  fscanf(stat, "cpu %lu %lu %lu %lu",&u0, &n0, &s0, &i0);
  fclose(stat);
  usleep(200000);
  stat = fopen("/proc/stat", "r");
  fscanf(stat, "cpu %lu %lu %lu %lu",&u1, &n1, &s1, &i1);
  fclose(stat);
  ud = u1 - u0;
  nd = n1 - n0;
  sd = s1 - s0;
  id = i1 - i0;
  tot = ud+nd+sd+id;
  cpu =100* (ud + nd + sd)/tot;
  if (cpu<10) {load=0;}
  if (cpu>=10) {load=1;}
  if (cpu>=25) {load=3;}
  if (cpu>=35) {load=7;}
  if (cpu>=50) {load=15;}
  if (cpu>=65) {load=31;}
  if (cpu>=75) {load=63;}
  if (cpu>=90) {load=127;}
  if (cpu<10) {load=0;}
  outb(load, BASEPORT);
  goto re;
  return 0;
}

[Yurror]

деление на ноль в строчке
cpu =100* (ud + nd + sd)/tot;

[alexander.pronin]

проверить перед делением
if(!tot) return 1;

[K_E_S_T]

спасибо. Даже предположить не мог что распределение процессорного времени может не изменяться. решил проблему проще - немного изменил формулу расчёта:
cpu =100* (ud + nd + sd)/(tot+1);
точность от этого, я думаю, не особо пострадает, а деления на ноль быть не сможет.

[alexander.pronin]

Теоретически, может.
tot+1=0
tot=-1
 

[K_E_S_T]

тот никогда не будет -1   иначе загрузка была бы отрицательным числом
Пользователь решил продолжить мысль 31 Августа 2009, 15:24:37:хотя теоретически, и вправду может. при использовании функций энергосбережения, например. когда количество тактов не постоянно...

[maximum2000]

Сейчас работает?

[K_E_S_T]

прекрасно работает

[wl]

А что за странная конструкция

(Нажмите, чтобы показать/скрыть)

#include "/usr/include/sys/io.h"

Почему не <sys/io.h> ?

[K_E_S_T]

копипаст    да и как то спокойней так )))