On Thu, Nov 6, 2008 at 12:32 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Thu, Nov 06, 2008 at 09:06:36AM -0500, J.R. Mauro wrote: >> static int poch_ioctl(struct inode *inode, struct file *filp, >> unsigned int cmd, unsigned long arg) >> { >> /* ---snip---*/ >> case POCH_IOC_SYNC_GROUP_FOR_USER: >> case POCH_IOC_SYNC_GROUP_FOR_DEVICE: >> vms = find_vma(current->mm, arg); >> if (!vms) >> /* Address not mapped. */ >> return -EINVAL; >> if (vms->vm_file != filp) >> /* Address mapped from different device/file. */ >> return -EINVAL; >> >> flush_cache_range(vms, arg, arg + channel->group_size); > > This doesn't look like something a driver should ever do. Could someone > explain what it's trying to do from a high level point of view? > > CC'd driver maintainers mentioned in the README -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html