Bug 1057 - LTE Bearers issue
LTE Bearers issue
Product: ns-3
Classification: Unclassified
Component: wimax
All All
: P5 major
Assigned To: Flavio Kubota
Depends on:
  Show dependency treegraph
Reported: 2011-02-18 14:44 EST by Iffat
Modified: 2012-05-19 11:08 EDT (History)
4 users (show)

See Also:


Note You need to log in before you can comment on or make changes to this bug.
Description Iffat 2011-02-18 14:44:09 EST
There is a problem with Bearers differentiation. In the scheduler, all the bearers (either GBR or NGBR) go to Default Bearer section. However, in the scheduler they must be scheduled on their respective sections (defined for loops).

Comment 1 Tom Henderson 2011-08-16 16:04:08 EDT
fix pulled from Giuseppe's repository, pushed as changeset: 8dc11dc385fe
Comment 2 Tom Henderson 2011-08-18 01:35:35 EDT
I'm holding out this portion of the changeset related to wimax IpcsClassifierRecord:

Discussion of this change to wimax is here in item 2)

The reason that I did not commit it is that it fails the wimax-tlv test.

FAIL: Test Suite "wimax-tlv" (0.000)
FAIL: Test Suite "Test the CS parameters tlv implementation." (0.000)
      Message:   The classifier address did not match.
      Condition: !classifier.CheckMatch (Ipv4Address (''), Ipv4Address (''), 1050, 3050, 17) (actual) == false (limit)
      Actual:    1
      Limit:     0
      File:      ../src/wimax/test/wimax-tlv-test.cc
      Line:      118

After patching ipcs-classifier-record.cc, if you run this test as follows:

NS_LOG="*" ./waf --run "test-runner --suite=wimax-tlv"

You will see:
IpcsClassifierRecord:CheckMatchProtocol(): proto check match: pkt=17 cls=17
IpcsClassifierRecord:CheckMatchDstPort(): dst port check match: pkt=3050 cls= [3000 TO 3100]
IpcsClassifierRecord:CheckMatchSrcPort(): src port check match: pkt=1050 cls= [1000 TO 1100]
IpcsClassifierRecord:CheckMatchDstAddr(): dst addr check match: pkt= cls=
IpcsClassifierRecord:CheckMatchDstAddr(): dst addr check match: pkt= cls=
IpcsClassifierRecord:CheckMatchDstAddr(): NOT OK!

Note that the patch forces a comparison between and, whereas the current ns-3-dev code will mask with and compare to  I have to assume that the original wimax-tlv-test writer checked that this was the correct behavior (to mask and then compare) so I am reluctant to change this until we can get clarification of this.
Comment 3 Nicola Baldo 2012-05-19 11:08:05 EDT
With the LENA code merge the scheduler API and implementations have completely changes, hence the bug does not affect any more the lte module. 

I am reassigning this to the wimax module for the maintainer to check whether the wimax code is still affected by the bug.