Everything else looks good, just that the 2x 10Gb links to the MX will be in active/passive mode with one being blocked at the switch end by spanning tree. Don't set the links to the MX up as a port channel on the switches, just do two matching ports.
... View more