On 19.11.18 20:47, Heiko Carstens wrote:
On Mon, Nov 19, 2018 at 06:25:30PM +0100, Michael Mueller wrote:
The function atomically returns the address of the first
GISA of this list and stores a zero address in the origin
list field of the GIB instead.
The GIB alert list contains all GISAs that have pending
adapter interruptions.
Signed-off-by: Michael Mueller <mimu@xxxxxxxxxxxxx>
Reviewed-by: Pierre Morel <pmorel@xxxxxxxxxxxxx>
...
+static struct kvm_s390_gisa __maybe_unused *unlink_gib_alert_list(void)
+{
+ u32 gisa;
+
+ if (!gib)
+ return NULL;
+
...
+ do {
+ gisa = READ_ONCE(gib->alert_list_origin);
+ } while (cmpxchg(&gib->alert_list_origin, gisa, 0) != gisa);
+
+ return (void *)(u64) gisa;
+}
Please make this shorter:
return (void *)(u64) xchg(&gib->alert_list_origin, gisa, 0);
Unbelievable... just one macro further down. Thanks for pointing me to.
