Re: [PATCH 1/2] x86: Add Isolated Memory Regions for Quark X1000
From: Bryan O'Donoghue
Date: Thu Jan 29 2015 - 10:56:27 EST
On 29/01/15 07:44, Ingo Molnar wrote:
Hi Ingo.
I'll transmit those changes in with the following exception
+ ret = imr_write(idev, reg, &imr, false);
So 'ret' here gets mixed with other potential failure modes.
If imr_write() fails here then that's a highly anomalous internal
failure - so it might be better to add a WARN_ON(ret) or so.
imr_write() already does a WARN - so no need to do it again.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/