On Sun, May 30, 2021 at 11:06:28AM -0400, Tianyu Lan wrote: > + for (i = 0; i < request->hvpg_count; i++) > + dma_unmap_page(&device->device, > + request->dma_range[i].dma, > + request->dma_range[i].mapping_size, > + request->vstor_packet.vm_srb.data_in > + == READ_TYPE ? > + DMA_FROM_DEVICE : DMA_TO_DEVICE); > + kfree(request->dma_range); Unreadably long lines. You probably want to factor the quoted code into a little readable helper and do the same for the map side.
