[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <3aaafc130811060936u371e8b9eyccd0c52693f4c433@mail.gmail.com>
Date: Thu, 6 Nov 2008 12:36:11 -0500
From: "J.R. Mauro" <jrm8005@...il.com>
To: "Christoph Hellwig" <hch@...radead.org>
Cc: "Paul Mackerras" <paulus@...ba.org>, "Greg KH" <greg@...ah.com>,
"Stephen Rothwell" <sfr@...b.auug.org.au>,
LKML <linux-kernel@...r.kernel.org>,
"David S. Miller" <davem@...emloft.net>,
"William L. Irwin" <wli@...omorphy.com>, vijaykumar@...vegnu.org,
jayakumar.lkml@...il.com, sparclinux@...r.kernel.org
Subject: Re: sparc/staging compile error
On Thu, Nov 6, 2008 at 12:32 PM, Christoph Hellwig <hch@...radead.org> 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 linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists