1. 03 Sep, 2018 18 commits
  2. 31 Aug, 2018 12 commits
  3. 30 Aug, 2018 3 commits
  4. 29 Aug, 2018 2 commits
  5. 28 Aug, 2018 3 commits
  6. 27 Aug, 2018 2 commits
    • Cedric Roux's avatar
      bugfix: fix threading in the UE · a48c3658
      Cedric Roux authored
      Various problems were found with the basic simulator.
      Variables used by various threads for synchronization need to
      be initialized properly. That is before the various threads
      start using them.
      This goes for:
      - UE->is_synchronized
      - UE->proc.instance_cnt_synch
      - UE->proc.instance_cnt_rxtx
      The function "UE->rfdevice.trx_start_func" was called in
      "UE_thread_synch" but should be called in "UE_thread" because
      "UE_thread" is the one that calls "UE->rfdevice.trx_read_func"
      and there is no guaranty that the call to "UE->rfdevice.trx_start_func"
      is done before as it has to (it's in another thread).
      And finally "pthread_cond_signal(&proc->cond_rxtx)" was called twice,
      which may not be a problem but was certainly not intended. Plus
      removing one call simplifies the code by removing some "if" logic,
      which is a good thing per se.
      This commit was not tested with a real UE and may thus introduce some
      issues. Hopefully not!
    • Cedric Roux's avatar
      basic simulator: bugfix: do not put random data in DCI when not used · 164bbc65
      Cedric Roux authored
      With 50 RBs and 100 RBs the UE crashes in the basic simulator
      because of false detection of DCIs.
      Putting nothing in the unused REs in the DCI instead of random
      bits solves this issue and let the basic simulator run with 50 RBs
      and 100 RBs.
      Note that in the real UE the problem needs to be solved because
      the channel may lead to false DCI detection and the consequent
      crashes have to be solved. This commit sort of "hides" the issue
      to have the basic simulator functional.
      Note also that putting nothing in the unused REs in the DCI should
      be the normal general case. But it has not been tested with anything
      but the basic simulator so the previous code is left.