| commit | 35b8117b0ad066d5964a85e5fc0fdbe1bc735a13 | |
| author | Benjamin Romer <benjamin.romer@unisys.com> | |
| Mon, 22 Sep 2014 17:45:03 +0000 (22 13:45 -0400) | ||
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | |
| Tue, 23 Sep 2014 20:35:03 +0000 (23 13:35 -0700) | ||
| tree | bbb3533a8af6afbb34c35a60aefed32d9a039768 | treesnapshot (tar.gz zip) |
| parent | 24fc2d01dbd569554373818b72e6d26c86290763 | commitdiff |
staging: unisys: fix IO memory copy in channel.h
The function ULTRA_check_channel_client() was recently patched to use
ioread8_rep(). Unfortunately the parameters were incorrectly ordered, and this
causes the function to fail and the guest to not boot.
This patch switches the parameters around and uses memcpy_fromio() instead,
which is clearer since a structure is being copied, not an array, and it
fixes the error message printout later in the function that was using the UUID
value in the channel, rather than the value copied from the channel, which
causes a misleading error to be reported.
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
The function ULTRA_check_channel_client() was recently patched to use
ioread8_rep(). Unfortunately the parameters were incorrectly ordered, and this
causes the function to fail and the guest to not boot.
This patch switches the parameters around and uses memcpy_fromio() instead,
which is clearer since a structure is being copied, not an array, and it
fixes the error message printout later in the function that was using the UUID
value in the channel, rather than the value copied from the channel, which
causes a misleading error to be reported.
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| drivers/staging/unisys/common-spar/include/channels/channel.h | diffblobblamehistory |