Refactor: scheduler: best practices for sort_promotable_instance()

Rename sort_promotable_instance() to cmp_promotable_instance() to indicate it's
a standard comparison function, drop an unused argument, and improve variable
names, comments, log messages, and formatting.

Drop the sort_rsc_index() internal API from libpe_status that was only used
here, since it boils down to a few lines.