Skip to content

Commit 635bda2

Browse files
committed
PHY: LtePhyEnb: broadcast beacons by default
Make enableBeacons default to true (was: default to enableHandover), so every eNB/gNB broadcasts beacons regardless of handover -- as real base stations always transmit reference signals. This makes the beacon-based radio link monitoring / RLF detection path available out of the box, not only when handover happens to be enabled. Adds about 10s to fingerprint suite runtime (~238s -> ~248s). Fingerprint impact (re-recorded): 105 of 132 simulations drift -- exactly the handover-disabled ones, where beacons now flow. In every one of them ONLY tplx and tilx move: the all-events fingerprints pick up the added beaconStarter self-messages and beacon airframe send/receive events. ~tNl (network-level data traffic) and sz (all scalar/statistic results) are byte-identical everywhere, confirming beacons perturb neither the data plane nor the RNG stream -- the UE beacon RX path early-outs before getSINR when handover is disabled, and beacon TX draws no random numbers. The 27 handover-enabled simulations are unchanged (beacons already flowed there). Both new error paths verified to fire.
1 parent 92053a1 commit 635bda2

2 files changed

Lines changed: 110 additions & 113 deletions

File tree

src/simu5g/stack/phy/LtePhyEnb.ned

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,18 +25,15 @@ simple LtePhyEnb extends LtePhyBase
2525
@class("LtePhyEnb");
2626

2727
// Marks this eNB as a handover target: its beacons are used by UEs for handover
28-
// feasibility assessment. Requires enableBeacons=true (the default when handover is
29-
// on); the module errors out at initialization if beacons are disabled while
30-
// handover is enabled.
28+
// feasibility assessment. Requires enableBeacons=true (the default).
3129
bool enableHandover = default(false);
3230

3331
// Enables broadcasting periodic beacons (at cellInfo.beaconInterval). Beacons are the
3432
// measurement signal UEs use both for handover feasibility assessment and for radio
35-
// link monitoring (RLF detection), decoupled from enableHandover so radio link
36-
// monitoring can run without handover. Defaults to enableHandover (beacons flow
37-
// exactly when handover is enabled). cellInfo.beaconInterval must be positive: a
38-
// zero/negative interval is an error -- switch beacons off with enableBeacons=false.
39-
bool enableBeacons = default(enableHandover);
33+
// link failure (RLF) detection. On by default so radio link monitoring works out
34+
// of the box; set to false to suppress beacons entirely. Must stay true when
35+
// enableHandover=true. Requires cellInfo.beaconInterval > 0.
36+
bool enableBeacons = default(true);
4037

4138
double targetBler = default(0.001);
4239
}

0 commit comments

Comments
 (0)