On Tue, Apr 29, 2025 at 08:34:56PM +0000, Pranjal Shrivastava wrote: > On Tue, Apr 29, 2025 at 08:24:33PM +0000, Pranjal Shrivastava wrote: > > On Fri, Apr 25, 2025 at 10:58:08PM -0700, Nicolin Chen wrote: > > > + struct iommufd_mmap *immap; > > > + int rc; > > > + > > > + if (WARN_ON_ONCE(!immap_id)) > > > + return -EINVAL; > > > + if (base & ~PAGE_MASK) > > > + return -EINVAL; > > > + if (!size || size & ~PAGE_MASK) > > > + return -EINVAL; > > > + > > > + immap = kzalloc(sizeof(*immap), GFP_KERNEL); > > > + if (!immap) > > > + return -ENOMEM; > > > + immap->pfn_start = base >> PAGE_SHIFT; > > > + immap->pfn_end = immap->pfn_start + (size >> PAGE_SHIFT) - 1; > > > + > > > + rc = mtree_alloc_range(&ictx->mt_mmap, immap_id, immap, sizeof(immap), > > > > I believe this should be sizeof(*immap) ? > > Ugh, Sorry, shouldn't this be size >> PAGE_SHIFT (num_indices to alloc) ? mtree_load() returns a "struct iommufd_map *" pointer. Nicolin
