virsh_attach_detach_disk: attach tmp disk with persistent option

We want to attach the disk with --config parameter, because we have to be sure,
that the disk will be attached using all versions of libvirt. But to use this
parameter we have to at first shutdown the guest because this chagne will affect
next boot of the guest.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
virsh_attach_detach_disk: move add acpi module after attach/detach required disks

To reduce restart count and time needed to do it I've just move this code after
disks preparation. For disk preparation you need to have the guest offline and
it would be pointless to start the guest, modprobe the acpiphp and than shutdonw
the guest again to attach/detach required disks.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
virsh_attach_detach_disk: attach second temp disk for the detach_twice test

We have to attach two disks for the detach_twice test in order to run the test.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
virsh_attach_detach_disk: make the proper return status checking

It's obvious that if you are running the attach/detach config test that you will
not see the change in the xml returned by dumpxml of a live guest.

This patch also fixes whitespace issues and typo in error message.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>