On Thu, Aug 08, 2019 at 09:12:54AM -0500, Parav Pandit wrote: > While generating interrupt, mdev_state is already available for which > interrupt is generated. > Instead of doing indirect way from state->device->uuid-> to searching > state linearly in linked list on every interrupt generation, > directly use the available state. > > Hence, simplify the code to use mdev_state and remove unused helper > function with that. > > Reviewed-by: Cornelia Huck <cohuck@xxxxxxxxxx> > Signed-off-by: Parav Pandit <parav@xxxxxxxxxxxx> Looks good, the old code is rather backwards and a bad example to copy: Reviewed-by: Christoph Hellwig <hch@xxxxxx>