On 07/27/2015 08:09 PM, Roland Dreier wrote: > On Mon, Jul 27, 2015 at 1:09 AM, Hannes Reinecke <hare@xxxxxxx> wrote: >> Hmm? What happened to the original FIXME? >> Is it not required anymore? > > Not sure I follow the question. The original FIXME was "/* FIXME: > Re-enable Global event handling.. */" and this patch indeed re-enables > global event handling. I don't think it's a separate patch, it's all > part of the same work. > > Unless I'm misunderstanding... > No, not really. If the patch indeed enables global event handling everything's fine. Wasn't just not quite obvious from the description nor the patch itself. So: Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@xxxxxxx +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html