1. 18 May, 2017 4 commits
    • Cedric Roux's avatar
      warnings removal in LOG_X · 9d65f4dd
      Cedric Roux authored
      those popped up thanks to previous commit
      9d65f4dd
    • Cedric Roux's avatar
      redo commit cb67d586 · 94dd8843
      Cedric Roux authored
      Seems like this commit got lost at some point.
      
      Here was the message:
      -----------------------------------------------------------------------------
      Author: Cedric Roux <cedric.roux@eurecom.fr>
      Date:   Tue Jan 10 14:21:02 2017 +0100
      
          add format indicator to get LOG_X warnings
      
          The LOG_X macros emit a lot of warnings when compiling with the T
          because they call the function logRecord (or logRecord_mt)
          which has not been "marked" as calling printf, so gcc won't
          emit typical printf warnings.
      
          With the T, they directly translate to sprintf, so those warnings
          pop up, much more verbose due to several macro expansions.
      
          Let's make them pop up all the time so it's easier to fix them.
      -----------------------------------------------------------------------------
      94dd8843
    • Cedric Roux's avatar
      add CQI to periodic logging of UE · f6693af6
      Cedric Roux authored
      f6693af6
    • Cedric Roux's avatar
      minor: change files mode (from 755 to 644) · 4eefaffb
      Cedric Roux authored
      4eefaffb
  2. 15 May, 2017 9 commits
  3. 13 May, 2017 1 commit
    • Cedric Roux's avatar
      rewrite phy_stats_exist · 9eaee53b
      Cedric Roux authored
      We have to test that all active CCs of a given UE exist in the PHY layer.
      Maybe not the end of the story...
      9eaee53b
  4. 12 May, 2017 14 commits
    • Cedric Roux's avatar
    • Cedric Roux's avatar
    • Cedric Roux's avatar
    • Cedric Roux's avatar
    • Cedric Roux's avatar
    • Tien-Thinh Nguyen's avatar
      fix the errors related to multiple carriers (for a given UE considering only... · 3d4ba1a3
      Tien-Thinh Nguyen authored
      fix the errors related to multiple carriers (for a given UE considering only the CCs which this UE belongs to)
      3d4ba1a3
    • Gabriel's avatar
      bug fixes from Fujitsu (bug 37) · ad6d0ac5
      Gabriel authored
      ```-------------------------------------------------------
      bug 37
      
      Ttile:
      Sending side is as follows.
      rar[3] = (((mcs&0x7)<<5)) | ((TPC&7)<<2) | ((ULdelay&1)<<1) | (cqireq&1);
      So, 2 bit shift looks correct.
      
      Bug Location:
      ulsch->harq_processes[harq_pid]->TPC = (rar[3]>>3)&7;//rar->TPC;
      ```
      
      -------------------------------------------------------
      ad6d0ac5
    • Gabriel's avatar
      bug fixes from Fujitsu (bug 29) · 893f36e8
      Gabriel authored
      ```-------------------------------------------------------
      bug 29
      
      Ttile:
      FDD should be TDD.
      
      Bug Location:
        switch (oai_emulation.info.frame_type[0]) {
        case FDD:
          frame_type = "FDD";
          break;
      
        case TDD:
          frame_type = "FDD";
          break;
        }
      ```
      
      -------------------------------------------------------
      893f36e8
    • Gabriel's avatar
      bug fixes from Fujitsu (bug 26) · 899d1506
      Gabriel authored
      ```-------------------------------------------------------
      bug 26
      
      Ttile:
      g_buffer has only 32 index, but he/she tries to zero clear
      1024 index.
      
      Bug Location:
      static uint8_t g_buffer[32];
      :
      memset (g_buffer, 0, 1024);
      
      Note:
      Detected by CppCheck
      ```
      
      -------------------------------------------------------
      899d1506
    • Cedric Roux's avatar
      bug fixes from Fujitsu (bug 24) · adb3b25c
      Cedric Roux authored
      ```-------------------------------------------------------
      bug 24
      
      Ttile:
      The last line "multicast_group = multicast_group" is hard
      to understand. The local variable should be different name
      than the global variable.
      
      Bug Location:
      const char *multicast_group_list[MULTICAST_LINK_NUM_GROUPS] = {
        "239.0.0.161",
        "239.0.0.162",
        "239.0.0.163",
        "239.0.0.164"
      };
      :
      :
      void multicast_link_start(void (*rx_handlerP) (unsigned int, char *),
                                unsigned char multicast_group, char *multicast_ifname)
      {
        rx_handler = rx_handlerP;
        multicast_group = multicast_group;
      
      Note:
      Detected by CppCheck
      ```
      
      -------------------------------------------------------
      adb3b25c
    • Gabriel's avatar
      bug fixes from Fujitsu (bug 22) · 34c6805f
      Gabriel authored
      Note: not sure that the code is correct,
      to be checked (Cédric Roux).
      
      ----------------------------------------------------------
      bug 22
      
      Ttile:
      (subframe!=2)||(subframe!=7) is always TRUE. This coding
      is really intended? Operator || should be && instead?
      
      Bug Location:
      ((frame_parms->frame_type==TDD)&&(frame_parms->tdd_config==1)&&((subframe!=2)||(subframe!=7)))) {
      
      Note:
      Detected by CppCheck
      ----------------------------------------------------------
      34c6805f
    • Gabriel's avatar
      bug fixes from Fujitsu (bug 21) · 98a7a16e
      Gabriel authored
      Note: this fix does not solve the issue with how
      this buffer is used in this function, but I'm not
      sure this code is used at all (Cédric Roux).
      
      ----------------------------------------------------------
      bug 21
      
      Ttile:
      PAYLOAD_MAX=1500, so this coding results in index overflow.
      Additionally, operator "!=" just compairs something, this
      coding is meaningless.
      
      Bug Location:
      buffer[PAYLOAD_MAX] != '\0';
      
      Note:
      Detected by CppCheck
      ----------------------------------------------------------
      98a7a16e
    • Gabriel's avatar
      bug fixes from Fujitsu (bugs 18, 19, 20) · bdaef795
      Gabriel authored
      ```-------------------------------------------------------
      bug 18
      
      Ttile:
      Probably he/she wants to copy the contents of arrays, but
      this coding doesn't copy anything, moreover index overflow
      occurs.
      
      Bug Location:
      u2e_t[UE_id][eNB_id]->tx_data[3]=tx[eNB_id][3];
      u2e_t[UE_id][eNB_id]->rx_data[3]=rx[NB_eNB_INST+UE_id][3];
      
      Note:
      Detected by CppCheck
      ```
      
      -------------------------------------------------------
      bug 19
      
      Title:
      Probably he/she wants to copy the contents of arrays, but
      this coding doesn't copy anything, moreover index overflow
      occurs.
      
      Bug Location:
      e2u_t[eNB_id][UE_id]->tx_data[3]=tx[NB_eNB_INST+UE_id][3];
      e2u_t[eNB_id][UE_id]->rx_data[3]=rx[eNB_id][3];
      
      Note:
      Detected by CppCheck
      ----------------------------------------------------------
      bug 20
      
      Title:
      Probably he/she wants to copy the contents of arrays, but
      this coding doesn't copy anything, moreover index overflow
      occurs.
      
      Bug Location:
      tx_data[3]=cthread->tx_data[3];
      rx_data[3]=cthread->rx_data[3];
      
      Note:
      Detected by CppCheck
      ----------------------------------------------------------
      bdaef795
    • Francois TABURET's avatar
      Fix shared lib build problem, where shared lib sources were also included in main exec's · 26135368
      Francois TABURET authored
      Discovered and fix problems in shared lib build for usrp,bladerf and lime
      26135368
  5. 03 May, 2017 6 commits
  6. 02 May, 2017 1 commit
  7. 26 Apr, 2017 1 commit
  8. 25 Apr, 2017 3 commits
  9. 24 Apr, 2017 1 commit