Re: [MMUSIC] Open Issues: draft-ietf-mmusic-sdp-simulcast-03

Paul Kyzivat <pkyzivat@alum.mit.edu> Fri, 23 October 2015 16:49 UTC

Return-Path: <pkyzivat@alum.mit.edu>
X-Original-To: mmusic@ietfa.amsl.com
Delivered-To: mmusic@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id ED1F81A8713 for <mmusic@ietfa.amsl.com>; Fri, 23 Oct 2015 09:49:40 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.235
X-Spam-Level:
X-Spam-Status: No, score=-1.235 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, SPF_SOFTFAIL=0.665] autolearn=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id CeqZ00XyIXgq for <mmusic@ietfa.amsl.com>; Fri, 23 Oct 2015 09:49:40 -0700 (PDT)
Received: from resqmta-ch2-05v.sys.comcast.net (resqmta-ch2-05v.sys.comcast.net [IPv6:2001:558:fe21:29:69:252:207:37]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id DCC151A8711 for <mmusic@ietf.org>; Fri, 23 Oct 2015 09:49:39 -0700 (PDT)
Received: from resomta-ch2-08v.sys.comcast.net ([69.252.207.104]) by resqmta-ch2-05v.sys.comcast.net with comcast id Ygp21r0092Fh1PH01gpfeh; Fri, 23 Oct 2015 16:49:39 +0000
Received: from Paul-Kyzivats-MacBook-Pro.local ([50.138.229.151]) by resomta-ch2-08v.sys.comcast.net with comcast id Ygpe1r00V3Ge9ey01gpe7q; Fri, 23 Oct 2015 16:49:39 +0000
To: mmusic@ietf.org
References: <56294C8A.1030205@nostrum.com>
From: Paul Kyzivat <pkyzivat@alum.mit.edu>
Message-ID: <562A6522.60501@alum.mit.edu>
Date: Fri, 23 Oct 2015 12:49:38 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Thunderbird/38.3.0
MIME-Version: 1.0
In-Reply-To: <56294C8A.1030205@nostrum.com>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=comcast.net; s=q20140121; t=1445618979; bh=SHPlZKlQtq44Zjont/hXYVRz+1OO9TrnKIPri7n/P4c=; h=Received:Received:Subject:To:From:Message-ID:Date:MIME-Version: Content-Type; b=rA+VoU4PazKBQVD0bkogC0gqNh4zx5YTCzc3Dy9LuBI2a3tRhOhSEQC832bhjMSeZ G82swUO2tUOnK6B+gzUof88TDpL3+b5i4AdoHb/CpJ/TfEWFG66fJPG9534iaaUNeU dlR3HXjiMcJJlhzi9y73yHRMpl80QqqyPawRVwWRBKYZx99W5tSlllNCcdpDvlbBRb kdOQJIfuD+T3tQoGaC4ZyXqgPcdISPzWqyi9qK17ASwIQUOeP71CKiCJq48Sw+CgHS +0Q46TeHH4vuamHvAj5Z5/Iq5w3/rlaIYUzgtJlzeBhZ9itBLQFvDce4HrncTPoCQT lCerN/x3hjz2A==
Archived-At: <http://mailarchive.ietf.org/arch/msg/mmusic/7bUkyhJAN2IRjm51DyvrPhS_yPI>
Subject: Re: [MMUSIC] Open Issues: draft-ietf-mmusic-sdp-simulcast-03
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Multiparty Multimedia Session Control Working Group <mmusic.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mmusic>, <mailto:mmusic-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mmusic/>
List-Post: <mailto:mmusic@ietf.org>
List-Help: <mailto:mmusic-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mmusic>, <mailto:mmusic-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 23 Oct 2015 16:49:41 -0000

On 10/22/15 4:52 PM, Adam Roach wrote:

> BIKESHED TIME! I think Martin is correct that the "space-colon"
> delimiter is unaesthetic; I'll go further and point out that it appears
> to be unprecedented (I could be missing something), which will make it
> unnecessary parsing exception as compared to other attributes. Mo's
> point that we want a colon is well-taken, but that doesn't justify the
> space. It seems that the current formulation was selected to make the
> ABNF easy to write. To that end, I've put together a formulation for the
> "just a colon" format.
>
> sc-attr     = "a=simulcast:" sc-str-list [WSP sc-str-list] [WSP sc-pause-list]
>
>
> Is there any reason we can't make the format consistent with all the
> other "a=foo:bar" syntaxes out there, as described by this rule?

Yes please. And please follow the format for attribute definition 
established in rfc4566bis.

Also, I have some other issues:
- syntax allows two sends, or two recvs, but those are semantically 
forbidden
- the syntax allows paused with PTs, but that is semantically forbidden.
- syntax allows pt values that aren't numeric
- use of WSP (SP or HTAB). Contrary to typical SDP usage.

Below is a proposal for revised syntax that follows the spirit, but 
aligns better with other attributes and 4566bis. It doesn't have some of 
the extensibility of the prior syntax because I couldn't see how to do 
it in a meaningful way. I don't think it is a great loss here.

This may perhaps need further tweaking based on the ongoing discussion 
of pt vs rid.

The following would replace the beginning of section 6.1, or maybe be a 
new section/subsection. It also ought to be cross referenced from IANA 
considerations as a request to add it to the iana sdp attribute registry.

	Thanks,
	Paul

6.1 Simulcast attribute

    Name: simulcast

    Value: simulcast-value

    Usage Level: media

    Charset Dependent: no

    Syntax:

       simulcast-value = sc-send-opt [SP sc-recv-opt]
                       / sc-recv-opt
       sc-send-opt = "send" SP sc-sendrecv-args
       sc-recv-opt = "recv" SP sc-sendrecv-args

       sc-sendrecv-args = sc-pt-args / sc-rid-args
       sc-pt-args = "pt=" sc-pt-list *(";" sc-pt-list)
       sc-rid-args = "rid=" sc-rid-list *(";" sc-rid-list)
                     [SP sc-pause-opt]

       sc-pt-list = payload-type *("," payload-type)
       sc-rid-list = rid-identifier *("," rid-identifier)
       sc-pause-opt = "paused=" sc-rid-list

       ; SP defined in [RFC5234]
       ; payload-type, defined in [draft-ietf-mmusic-rfc4566bis-15]
       ; rid-identifier defined in [I-D.pthatcher-mmusic-rid]

    Examples:

       a=simulcast: send pt=97;98 recv pt=97
       a=simulcast: send rid=1,2,3 recv rid=4 paused=3