dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
overview.adoc: update network diagram
The network diagram in the manual lacks the recently added co located MGW.
Change-Id: Ica1782a407f6b944aa26748b54055168f5d250c3 Related: OS#5152 --- M doc/manuals/chapters/overview.adoc 1 file changed, 14 insertions(+), 10 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/96/26796/1
diff --git a/doc/manuals/chapters/overview.adoc b/doc/manuals/chapters/overview.adoc index 2368b4f..fa8dda0 100644 --- a/doc/manuals/chapters/overview.adoc +++ b/doc/manuals/chapters/overview.adoc @@ -15,16 +15,20 @@ [[fig-3g]] .Typical 3G network architecture used with OsmoHNBGW ---- - +------------+ +--------+ +----------+ +---------+ - UE <-->| hNodeB |<--Iuh---->| HNB-GW |<--IuCS-->| OsmoMSC |<--GSUP-->| OsmoHLR | - UE <-->| femto cell | ...-->| | ...-->| | | | - | | | | +----------+ +---------| - +------------+<--GTP-U | | - \ | | +------+ +------+ - | | |<--IuPS-->| SGSN |<--GTP-C-->| GGSN | - | +--------+ ...-->| | GTP-U-->| | - | +------+ / +------+ - _______________________________/ + +------------+ +--------+ +---------+ +---------+ + UE <-->| hNodeB |<----Iuh----->| HNB-GW |<--IuCS-->| OsmoMSC |<--GSUP-->| OsmoHLR | + UE <-->| femto cell | | | | | | | + | | | | +---------+ +---------+ + | |<---IuPF--\ | | + | |<--GTP-U | | | +------+ +------+ + +------------+ \ | | |<--IuPS-->| SGSN |<--GTP-C-->| GGSN | + | | +--------+ | | GTP-U-->| | + | | +------+ / +------+ + | | +-----+ +-----+ | + | -->| MGW |<----IuPF--->| MGW | | + | | | | | | + | +-----+ +-----+ | + ___________________________________/ ----
The HNB-GW performs a translation interface between the IuCS/IuPS interfaces on the one hand
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/1/doc/manuals/chapters/overv... File doc/manuals/chapters/overview.adoc:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/1/doc/manuals/chapters/overv... PS1, Line 29: | | | | | | Maybe use "MGW (HNB-GW)" and "MGW (MSC)" in there, as suggested in older version of the patch.
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/1/doc/manuals/chapters/overv... PS1, Line 31: ___________________________________/ You can probably move the "GTP-U" label here instead of having it twice.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 6: Code-Review-1
(2 comments)
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/6/doc/manuals/chapters/overv... File doc/manuals/chapters/overview.adoc:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/6/doc/manuals/chapters/overv... PS6, Line 22: IuPF IuUP?
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/6/doc/manuals/chapters/overv... PS6, Line 28: IuPF IuUP?
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 6:
(1 comment)
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/6/doc/manuals/chapters/overv... File doc/manuals/chapters/overview.adoc:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/6/doc/manuals/chapters/overv... PS6, Line 22: IuPF
IuUP?
Ah yes, it's IUFP (3GPP TS 29.414, 29.415) or IuUP.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 8:
(2 comments)
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/6/doc/manuals/chapters/overv... File doc/manuals/chapters/overview.adoc:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/6/doc/manuals/chapters/overv... PS6, Line 22: IuPF
Ah yes, it's IUFP (3GPP TS 29.414, 29.415) or IuUP.
Done
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/6/doc/manuals/chapters/overv... PS6, Line 28: IuPF
IuUP?
Done
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796
to look at the new patch set (#8).
Change subject: overview.adoc: update network diagram ......................................................................
overview.adoc: update network diagram
The network diagram in the manual lacks the recently added co located MGW.
Change-Id: Ica1782a407f6b944aa26748b54055168f5d250c3 Related: OS#5152 --- M doc/manuals/chapters/overview.adoc 1 file changed, 14 insertions(+), 10 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/96/26796/8
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 8:
(1 comment)
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/8/doc/manuals/chapters/overv... File doc/manuals/chapters/overview.adoc:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/8/doc/manuals/chapters/overv... PS8, Line 28: | -->| MGW |<----IuUP--->| MGW | | I wrote several times that it would be great to have "MGW (BSC)" and "MGW (MSC)" in here, specally since the later is not by the side of MGW, but my comments were ignored all the time.
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796
to look at the new patch set (#10).
Change subject: overview.adoc: update network diagram ......................................................................
overview.adoc: update network diagram
The network diagram in the manual lacks the recently added co located MGW.
Change-Id: Ica1782a407f6b944aa26748b54055168f5d250c3 Related: OS#5152 --- M doc/manuals/chapters/overview.adoc 1 file changed, 14 insertions(+), 10 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/96/26796/10
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 10:
(3 comments)
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/1/doc/manuals/chapters/overv... File doc/manuals/chapters/overview.adoc:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/1/doc/manuals/chapters/overv... PS1, Line 29: | | | | | |
Maybe use "MGW (HNB-GW)" and "MGW (MSC)" in there, as suggested in older version of the patch.
Done
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/1/doc/manuals/chapters/overv... PS1, Line 31: ___________________________________/
You can probably move the "GTP-U" label here instead of having it twice.
Done
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/8/doc/manuals/chapters/overv... File doc/manuals/chapters/overview.adoc:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796/8/doc/manuals/chapters/overv... PS8, Line 28: | -->| MGW |<----IuUP--->| MGW | |
I wrote several times that it would be great to have "MGW (BSC)" and "MGW (MSC)" in here, specally s […]
I did not see the comment. I have updated it now. I hope its better now.
Attention is currently required from: dexter. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 13: Code-Review+1
Attention is currently required from: dexter. Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796
to look at the new patch set (#19).
Change subject: overview.adoc: update network diagram ......................................................................
overview.adoc: update network diagram
The network diagram in the manual lacks the recently added co located MGW.
Change-Id: Ica1782a407f6b944aa26748b54055168f5d250c3 Related: OS#5152 --- M doc/manuals/chapters/overview.adoc 1 file changed, 14 insertions(+), 10 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/96/26796/19
Attention is currently required from: dexter. Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796
to look at the new patch set (#20).
Change subject: overview.adoc: update network diagram ......................................................................
overview.adoc: update network diagram
The network diagram in the manual lacks the recently added co located MGW.
Change-Id: Ica1782a407f6b944aa26748b54055168f5d250c3 Related: OS#5152 --- M doc/manuals/chapters/overview.adoc 1 file changed, 14 insertions(+), 10 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/96/26796/20
Attention is currently required from: dexter. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 20: Code-Review+2
Attention is currently required from: dexter. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 22: Code-Review+2
Attention is currently required from: dexter. Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796
to look at the new patch set (#24).
Change subject: overview.adoc: update network diagram ......................................................................
overview.adoc: update network diagram
The network diagram in the manual lacks the recently added co located MGW.
Change-Id: Ica1782a407f6b944aa26748b54055168f5d250c3 Related: OS#5152 --- M doc/manuals/chapters/overview.adoc 1 file changed, 14 insertions(+), 10 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/96/26796/24
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
Patch Set 24: Code-Review+2
dexter has submitted this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/26796 )
Change subject: overview.adoc: update network diagram ......................................................................
overview.adoc: update network diagram
The network diagram in the manual lacks the recently added co located MGW.
Change-Id: Ica1782a407f6b944aa26748b54055168f5d250c3 Related: OS#5152 --- M doc/manuals/chapters/overview.adoc 1 file changed, 14 insertions(+), 10 deletions(-)
Approvals: dexter: Looks good to me, approved Jenkins Builder: Verified
diff --git a/doc/manuals/chapters/overview.adoc b/doc/manuals/chapters/overview.adoc index 2368b4f..aecb21a 100644 --- a/doc/manuals/chapters/overview.adoc +++ b/doc/manuals/chapters/overview.adoc @@ -15,16 +15,20 @@ [[fig-3g]] .Typical 3G network architecture used with OsmoHNBGW ---- - +------------+ +--------+ +----------+ +---------+ - UE <-->| hNodeB |<--Iuh---->| HNB-GW |<--IuCS-->| OsmoMSC |<--GSUP-->| OsmoHLR | - UE <-->| femto cell | ...-->| | ...-->| | | | - | | | | +----------+ +---------| - +------------+<--GTP-U | | - \ | | +------+ +------+ - | | |<--IuPS-->| SGSN |<--GTP-C-->| GGSN | - | +--------+ ...-->| | GTP-U-->| | - | +------+ / +------+ - _______________________________/ + +------------+ +----------+ +---------+ +---------+ + UE <-->| hNodeB |<----Iuh----->| HNB-GW |<--IuCS-->| OsmoMSC |<--GSUP-->| OsmoHLR | + UE <-->| femto cell | | | | | | | + | | | | +---------+ +---------+ + | |<---IuUP--\ | | + | |<-------. | | | +---------+ +------+ + +------------+ | | | |<--IuPS-->| SGSN |<--GTP-C-->| GGSN | + | | +----------+ | | .------>| | + | | +---------+ | +------+ + | | +----------+ +---------+ | + | -->| MGW |<--IuUP-->| MGW | | + | | (HNB-GW) | | (MSC) | | + | +----------+ +---------+ | + `--------------------GTP-U----------------' ----
The HNB-GW performs a translation interface between the IuCS/IuPS interfaces on the one hand