Vadim Yanitskiy has posted comments on this change. ( 
https://gerrit.osmocom.org/11423 )

Change subject: VTY: add extended (11-bit) RACH detection toggle
......................................................................


Patch Set 1:

Hi Harald,

> [...] when RACH Detection is enabled on TCH during handover,
> we never need TS1/TS2 and hence can optimize that case.

Thanks, good point! I was thinking about that.

> did you actually do some benchmarking to see
> if my suspicion about performance impact was correct?

Not yet, and not sure will I be able to do in short term.
Anyway, which tool(s) would you recommend for benchmarking?

> it would be great if we could somehow do this automatically,
> [...] Maybe some addition to the TRXCTRL protocol? [...]

This is also a good idea! But it would require more efforts,
and could be done in a separate change I think.


--
To view, visit https://gerrit.osmocom.org/11423
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1aee998d83b06692d76a83f79748f9129a2547e8
Gerrit-Change-Number: 11423
Gerrit-PatchSet: 1
Gerrit-Owner: Vadim Yanitskiy <axilira...@gmail.com>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Vadim Yanitskiy <axilira...@gmail.com>
Gerrit-CC: Harald Welte <lafo...@gnumonks.org>
Gerrit-Comment-Date: Mon, 22 Oct 2018 15:33:20 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No

Reply via email to