while beendet obwohl nicht "0"

[Michael D. (Gast)]

Nabend.

Ich bin dabei mit eine Motorsteuerung zu bauen, welche über 
Hallsensoren, welche über 6Magnete angesteuert werden zählen kann in 
welcher Position die Welle sich befindet.

Leider habe ich beim Zählen (450Schritte Linkslauf) das Problem, das er 
teilweise (jedes 5-7te mal) nach geschätzt der Hälfe stehen bleibt.

Darauf hin habe ich das Signal der Hallsensoren auf dem Oszilloskop 
angeschaut und gesehen das vereinzelnt Spikes vorhanden sind, weswegen 
ich also eine Schleife eingebaut habe, welche nicht sofort zählt.


Nachdem ich nun 1Woche dabei bin habe ich mal Spaßeshalber aus:

while(CountPulses) {
}

den Befehl:

while(CountPulses) {
}
while(CountPulses) {
}

gemacht. Also einfach die Abfrage doppelt genommen und siehe da: Es 
funktioniert (50Durchläufe).

Kann mir jemand sagen woran das liegt?
Count Pulses ist eine volatile 16Bit Variable.

Pollen mache ich über Timer0 mit einem Teiler von 8:

ISR(TIMER0_OVF_vect) {
     uint8_t temp=PHall1;
     if(!((temp&Hall1)==((temp&Hall2)>>1))) {
          if(((Status&LetzteHall1)==(temp&Hall1))) {
               switch (Status&0xC0) {

                    case 0x80:
                         break;

                    case 0x40:
                         Status^= LetzteHall1;
                         Status+= 0x40;

                    default:
                         Status&= 0xCF;
                         Status+= 0x40;
                         break;

               }
          } else {
               switch (Status&0x30) {

                    case 0x20:
                         break;

                    case 0x10:
                         if((Status&Drehrichtung)) {
                              CountPulses++;
                         } else {
                              CountPulses--;
                         }
                         Status+= 0x10;
                         break;

                    default:
                         Status&= 0x3F;
                         Status+= 0x10;
                         break;

               }
          }
     } else {
          Status&= 0x3F;
     }
}

[Jörg W. (dl8dtl) (Moderator)]

Meine neu gekaufte Glaskugel meint:

FAQ #1: "volatile" vergessen (bei "CountPulses").

[Stefan E. (sternst)]

Michael D. schrieb:
> Leider habe ich beim Zählen (450Schritte Linkslauf) das Problem, das er
> teilweise (jedes 5-7te mal) nach geschätzt der Hälfe stehen bleibt.

Michael D. schrieb:
> Count Pulses ist eine volatile 16Bit Variable.

Das liegt daran, dass das Lesen in der Schleife nicht atomar ist. Daher 
kann beim Übergang von 256 nach 255 vorzeitig eine 0 erkannt werden.

[Michael D. (Gast)]

Stefan Ernst schrieb:
> Das liegt daran, dass das Lesen in der Schleife nicht atomar ist. Daher
> kann beim Übergang von 256 nach 255 vorzeitig eine 0 erkannt werden.

Weiß zwar nicht wie das "atomar" gemeint ist, denke aber, das während 
des Lesevorganges die ISR aufgerufen wird und das Byte geändert.

Würde jetzt als Laie in der while() die Variable zwischen einem cli() 
und sei() kopieren sodass eine Kopie überprüft wird.

Oder gibt es da eine fertige Lösung?

[Stefan E. (sternst)]

Michael D. schrieb:
> Weiß zwar nicht wie das "atomar" gemeint ist, denke aber, das während
> des Lesevorganges die ISR aufgerufen wird und das Byte geändert.

Es ist ja nicht nur ein Byte, sondern zwei. Und um festzustellen, ob die 
Variable 0 ist, müssen beide Bytes getestet werden. Und wenn nun genau 
zwischen diesen beiden Tests der Interrupt die Variable von 256 auf 255 
ändert, wird fälschlicherweise eine 0 erkannt.

Michael D. schrieb:
> Würde jetzt als Laie in der while() die Variable zwischen einem cli()
> und sei() kopieren sodass eine Kopie überprüft wird.

Ich würde es so machen:

#include <util/atomic.h>
...

    uint16_t tmp = 1;
    while (tmp) {
        ATOMIC_BLOCK(ATOMIC_FORCEON) {
            tmp = CountPulses;
        }    
    }

[Michael D. (Gast)]

Vielen Dank für die Info.

Habe mir das mal im .lss angeschaut, werde beim cli()/sei() bleiben, da 
es genau gleich über setzt wird und für mich einfacher ist.