lkml.org 
[lkml]   [2015]   [Nov]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 3/4] nvdimm: Add IOCTL pass thru
On Tue, Nov 10, 2015 at 11:24:47AM -0500, Jeff Moyer wrote:
> Jerry Hoemann <jerry.hoemann@hpe.com> writes:
>
> > @@ -633,10 +718,11 @@ static int match_dimm(struct device *dev, void *data)
> >
> > static long nvdimm_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
> > {
> > - int rc = -ENXIO, read_only;
> > + int rc = -ENXIO, ro;
> > struct nvdimm_bus *nvdimm_bus;
> > + unsigned int type = _IOC_TYPE(cmd);
> >
> > - read_only = (O_RDWR != (file->f_flags & O_ACCMODE));
> > + ro = (O_RDWR != (file->f_flags & O_ACCMODE));
>
> I'm still reviewing the rest of this, but this is bugging me. The
> existing check for read_only looks pretty fishy to me. O_WRONLY is a
> thing (even though it's probably not a supportable mode for this ioctl).
> Why not just check for O_RDONLY?


Good question. I'll look into changing for version 2.
I suspect you would like something more like:

ro = ((file->f_flags & O_ACCMODE) == O_RDONLY);

>
> Cheers,
> Jeff

--

-----------------------------------------------------------------------------
Jerry Hoemann Software Engineer Hewlett-Packard Enterprise

3404 E Harmony Rd. MS 36 phone: (970) 898-1022
Ft. Collins, CO 80528 FAX: (970) 898-0707
email: jerry.hoemann@hpe.com
-----------------------------------------------------------------------------


\
 
 \ /
  Last update: 2015-11-10 23:01    [W:0.053 / U:0.284 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site