From 164bbc65eba44b37d15207a5c39e631a06023145 Mon Sep 17 00:00:00 2001 From: Cedric Roux <cedric.roux@eurecom.fr> Date: Mon, 27 Aug 2018 17:38:47 +0200 Subject: [PATCH] basic simulator: bugfix: do not put random data in DCI when not used 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. --- openair1/PHY/LTE_TRANSPORT/dci.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/openair1/PHY/LTE_TRANSPORT/dci.c b/openair1/PHY/LTE_TRANSPORT/dci.c index 1c015b1c84..520488664d 100755 --- a/openair1/PHY/LTE_TRANSPORT/dci.c +++ b/openair1/PHY/LTE_TRANSPORT/dci.c @@ -346,6 +346,13 @@ uint8_t generate_dci_top(uint8_t num_pdcch_symbols, y[0] = &yseq0[0]; y[1] = &yseq1[0]; +#if BASIC_SIMULATOR + /* this should be the normal case + * but it has to be validated for all the various cases + * so let's just do it for the basic simulator + */ + memset(e, 2, DCI_BITS_MAX); +#else #if 1 // reset all bits to <NIL>, here we set <NIL> elements as 2 // memset(e, 2, DCI_BITS_MAX); @@ -358,6 +365,7 @@ uint8_t generate_dci_top(uint8_t num_pdcch_symbols, * (not sure about this, to be checked somehow) */ //memset(e, 0, DCI_BITS_MAX); +#endif /* BASIC_SIMULATOR */ e_ptr = e; -- 2.26.2