On Fri, Oct 30, 2020 at 11:38:03AM +0200, Parav Pandit wrote: > @@ -1140,7 +1141,10 @@ int rxe_register_device(struct rxe_dev *rxe, const char *ibdev_name) > rxe->ndev->dev_addr); > dev->dev.dma_parms = &rxe->dma_parms; > dma_set_max_seg_size(&dev->dev, UINT_MAX); > - dma_set_coherent_mask(&dev->dev, dma_get_required_mask(&dev->dev)); > + dma_mask = IS_ENABLED(CONFIG_64BIT) ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32); > + err = dma_coerce_mask_and_coherent(&dev->dev, dma_mask); Since this mask doesn't actually do anything, what is the reason to have the 64/32? Jason
