Bug 2405

Summary:	PHY-MAC interface improvement: EndTx event should be added to UanPhyListener class
Product:	ns-3	Reporter:	Federico Guerra <fedwar82>
Component:	uan	Assignee:	Federico Guerra <fedwar82>
Status:	PATCH PENDING ---
Severity:	enhancement	CC:	hossamkhader, ns-bugs, tomh
Priority:	P5
Version:	ns-3.25
Hardware:	All
OS:	All
Attachments:	Patch candidate 00

Description Federico Guerra 2016-05-08 04:11:14 EDT

Currently
there is no mandatory tx end event from PHY to MAC.
This is awkward, since it forces the MAC to use an internal timer to mimic the
EndTx of the lower PHY.
The correct solution would be to ensure that the PHY would always notify the MAC
in case of tx completed or dropped due to energy drained.

Comment 1 Hossam Khader 2018-03-05 16:38:09 EST

Hi Federico,

Do you already have a patch for this? Let me know if you need help with this?

Comment 2 Federico Guerra 2018-03-09 09:51:46 EST

Created attachment 3085 [details]
Patch candidate 00

Hi Hossam, 
I'm finally feeling a little better.
I've attached a first patch proposal.

1) Tx End event added to UanPhyListener class
2) MacCw modified accordingly. This requires a thorough check, since I haven't test it thoroughly, the logic should be ok though.

Let me know what you think.

Federico

Comment 3 Federico Guerra 2018-03-09 09:53:33 EST

modified files have been decrustified, hence the many indentation fix/changes.

Comment 4 Tom Henderson 2018-03-09 11:59:57 EST

(In reply to Federico Guerra from comment #3)
> modified files have been decrustified, hence the many indentation
> fix/changes.

Hossam and Federico, do you want to slip this into the ns-3.28 release (is it ready, in your opinion)?

Comment 5 Federico Guerra 2018-03-27 05:35:22 EDT

Hi Hossam,
do you have any comment?
thanks

Federico