Re: [PATCH V2 1/7] staging: vchiq_arm: Add compat ioctl for create service

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 23, 2017 at 07:43:09AM -0800, Michael Zoran wrote:
> On Mon, 2017-01-23 at 14:38 +0300, Dan Carpenter wrote:
> > I'm attaching how I would basically like to see patch 1/7 done.  It's
> > not perfect, I did it as quickly as possible and haven't tested
> > anything.  I would normally be more careful if I were sending this
> > for
> > inclusion so there are no sign offs etc.
> 
> Thanks, Dan. I appreciate the clarify on what you are looking for.
> 
> Sounds like if I go that route it's going to take some time.  I'm
> thinking that if I do this I should deal with one ioctl cleanup per
> series instead of aiming for the goal of getting compat to work.
> 

It doesn't take that long really...  I kind of figured things out as I
went along.  Implementing the second ioctl would maybe have been
quicker.  But it also might be good to just go through and break the
ioctl up into tons of little functions first before implementing the
COMPAT stuff.  Do the cleanup first then the rest.

What I like about my code is that it removes abstractions.  The
do_ioctl_create_service() takes a VCHIQ_CREATE_SERVICE_T and writes an
int at the end.  It's not generic at all.

> What's your opinion of building the compat_ioctl function on top of the
> existing code?  Using the internal compat functions to build a 64 bit
> version of the data on the user mode stack? With that approach none of
> the existing code would need to be touched although the end result
> wouldn't be as nice or maintainable.

My instinct is that would be more subtle and complicated.

regards,
dan carpenter

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux