Re: Genart last call review of draft-ietf-teas-sr-rsvp-coexistence-rec-02

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Joel,

Thanks for the review comments. Comments are inline with [HS].

On 4/19/18, 3:15 AM, "Joel Halpern" <jmh@xxxxxxxxxxxxxxx> wrote:
   
    Summary:
    
    Major issues:
        The focus of the draft seems to be the recommendation in section 3.5 that
        the maximum reservable bandwidth on a link be adjusted to reflect the SR
        traffic consumption.  There appear to be two issues that need to be
        discussed, both related to the difference between what the SR controller
        wants to reserve and what the router observes. First, an SR controller may
        be performing calculations without requiring that bandwidth be committed to
        the traffic.  The recommendation here assumes that all traffic using SR is
        high priority.  It may suffice to note that QoS markings in the labels
        (corresponding to diffserv markings in the underlying packet may hel with
        this.  Given the range of allowed behaviors in when RSVP-TE and SR are
        separate, it may also be necessary to restrict what the SR controllers do
        in these interworking cases. 

[HS] In the first paragraph of section 3.5, there is text referring to SR having highest preemption priority but the SR traffic could have different QoS markings i.e. within SR there could be different classes of traffic which is accordingly handled by the forwarding plane (e.g. defined operator policy).

        Second, and more importantly, this solution
        assumes that short term traffic measurements are a good proxy for intended
        reservation.  Even assuming edge policing so that usage is less than or
        equal to the reservation, this will frequently underestimate the traffic
        reservation.  Such underestimates would seem to be able to cause
        significant problems.

[HS] Even with RSVP-TE LSPs where explicit admission-control is done to reserve bandwidth, the bandwidth that is reserved on the RSVP-TE LSP may differ from how much traffic actually arrives on the LSP (assuming no edge policing). As the traffic changes, auto-bandwidth implementation procedures might be there to adjust the LSP reservation at periodic intervals. This relies on short term LSP traffic measurement to achieve change in reservation. 

[HS] SR traffic can preempt RSVP LSPs to make room for itself based on short term SR traffic measurement. The frequency at which SR statistics is collected for a TE interface and how often the Maximum-Reservable-Bandwidth is adjusted so that path computation engines for RSVP-TE LSPs get the updated TED information is implementation and deployment dependent (it could be aggressive to reflect SR traffic utilization in the TED often or done less frequently due to other deployment parameters). In the penultimate paragraph of section 3.5, there is text referring to the implementation choice.
    
    Minor issues:
        Section 3.5 assumes that the router can measure the traffic using SR.  This
        seems to rely on the unstated premise that the measurement is conditioned
        by the recognition of which labels are being used for SR.  This is
        reasonable.  It should be stated.

[HS] Fair point. However, the measured traffic is not just from SR labels (transit traffic) but also any traffic entering the SR domain over that outgoing interface. An implementation should be capable of measuring all the SR traffic going out of an interface. The text generically refers to needing ability to measure SR traffic statistics across the TE interface.
    
    Nits/editorial comments:
        The second paragraph of the introduction seems to have the opening text
        repeated twice.

[HS] Thanks - noted also as part of OPsDir review comments. The 2nd repeated sentence will be removed.
    
        The third paragraph of section 3.1 seems to be a repetition of the end of
        the second paragraph using slightly different words.

[HS] Ok, the aim was to offer a summary of the behavior separately so that its clear. If its ok, we can retain it.

--
Harish
    





[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Mhonarc]     [Fedora Users]

  Powered by Linux