[PATCH v2 1/8] x86/MCE: avoid effectively open-coding xmalloc_array()

[Jan Beulich <jbeulich@xxxxxxxx>]

There is a difference in generated code: xmalloc_bytes() forces
SMP_CACHE_BYTES alignment. But if code really cared about such higher
than default alignment, it should request so explicitly rather than
using a type-unsafe interface. And if e.g. cache line sharing was a
concern, the allocator itself should arrange to avoid such.

Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

--- a/xen/arch/x86/cpu/mcheck/mctelem.c
+++ b/xen/arch/x86/cpu/mcheck/mctelem.c
@@ -345,7 +345,7 @@ void __init mctelem_init(unsigned int da
 
        if ((mctctl.mctc_elems = xmalloc_array(struct mctelem_ent,
            MC_NENT)) == NULL ||
-           (datarr = xmalloc_bytes(MC_NENT * datasz)) == NULL) {
+           (datarr = xmalloc_array(char, MC_NENT * datasz)) == NULL) {
                xfree(mctctl.mctc_elems);
                printk("Allocations for MCA telemetry failed\n");
                return;