Jeff Morriss
changed
bug 9158
| What |
Removed |
Added |
| Status |
INCOMPLETE
|
CONFIRMED
|
Comment # 5
on bug 9158
from Jeff Morriss
(In reply to comment #4)
> Jeff, I am not sure I understand your suggestion but my natural idea would
> be to display both bitmasks in the subtree of DSAP/SSAP value, the SAP and
> the I/G or C/R bit, while not adding any new fields to the top-level tree,
> as that was exactly what I was trying to achieve with my original patch. My
> reasoning here is that mostly, you are interested into the entire DSAP/SSAP
> value, not in its individual bits, so the top-level tree under the LLC
> should just display the DSAP and SSAP value as it does now with my patch. If
> you want to see it dissected into bits including their meaning, you enter
> the subtrees of DSAP/SSAP. This is similar to the display of MAC addresses -
> normally, you're not interested in seeing the U/L and I/G bits, but should
> you want to go that deep, you enter the subtrees of destination/source MAC.
> Would you find that more acceptable?
Yes, that seems reasonable too. Maybe someone else might have a different
opinion on the display in the current patch but to me it does "feel" kind of
incomplete or funny. But it may just be me.
You are receiving this mail because:
- You are watching all bug changes.