illumos gate

in ql_get_serdes_regs, we write the contents of our registers into a struct ql_mpi_coredump. Unfortunately in two cases we mis-index arrays in the dump structure, and write to array+<physical offset of register>, way out of bounds.

GCC4 notices.

This code is only called in the face of (systemic?) device errors signalled from the board, and is likely very hard to reproduce, even while having hardware, unless a call to ql_8xxx_binary_core_dump is artificially inserted.