The IESG discussed mld-source last week. Here are the comments.

Thomas:
>    MLD Report and Done messages MUST be sent with a valid link-local
>    address as the IPv6 source address.  If a valid link-local address is
>    not available, the message MUST be sent with the unspecified address
>    (::) as the IPv6 source address.

A bit of a nit, but the MUSTs are almost contradictory. It's a bit odd
to say: Do X. Well except if you can't, in which case you do Y. Better
to rewrite something like:

    MLD Report and Done messages are sent with a link-local address as
    the IPv6 source address, if the one has been assigned to the
    interface.  If a none exists (e.g., one hasn't been assigned to
    the interface yet), the message is sent with the unspecified
    address (::) as the IPv6 source address.

nit: RFC 2119 reference should be normative rather than informative.

Finally, the document really could include a bit more context about
why the document exists and what problem is being solved.

Based on some discussions with the author, the exact problem being
solved seems to be:

The rules and desired behavior with regards to receiving multicast
traffic prior to having an IP address need clarification. Before
assigning an LL address to an interface, a node needs to run DAD. But
DAD involves recieving multicast traffic sent to the solicited node
multicast address. But joining a multicast group involves running
MLD. The MLD spec says that MLD messages MUST be sourced from a LL
source address. This is needed even for LL multicast addresses due to
l2 bridge snooping. Thus,  clarifications/guidelines regarding the
handling of joining multicast groups  when one has no LL address are
needed.

It would be good to get words into the document that explain this
better.

Also, I think it would be good to add some explicit wording to the
document making it clear which rules in 2710 are being changed. I
particular, it would be good to indicate that receivers should not
drop packets with a source address of unspecified.

In general, you might want to say something about what problems have
been encountered in practice from the current wording, and whether
this change will result in different/better/worse behavior.


For example, should an implementation (once it has a valid ll address)
also run MLD again with its new address, to be sure that it
interoperates well with routers that drop packets with a source
address of unspecified?

---

The text about:
MLD Report and Done messages SHOULD be sent with a valid link-local
  address as the IPv6 source address.  If a valid link-local address is
  not available, the message MAY be sent with the unspecified address
  (::) as the IPv6 source address.

seems odd. Shouldn't it say "MUST be link local if a valid link local address is availble, otherwise it MUST be the unspecified address"?

Also, the references should be split (per rfc-ed policy) into normative and non-normative references.