(Illustration by Gaich Muramatsu)
On Fri, Apr 22, 2016 at 07:40:00PM +0200, u-myfx_at_aetey.se wrote: > On Fri, Apr 22, 2016 at 11:55:46AM -0400, Jan Harkes wrote: > > I can't believe this was the only place that had an alignment problem > > like this, those ioctls are badly packed all over the place. > > May be we are especially lucky, but this was the only visible problem. This particular VIOC_GETVOLSTAT pioctl call is used in 4 different places in cfs and 2 places in repair, your fix addressed one place. Also things like the pioctl in/out structures seem to be defined at the spot of the implementation which makes it very easy to make a change somewhere that is not picked up elsewhere. Considering how many places need to change I'm no longer sure I want to reshuffle and am starting to think that having common pioctl pack/unpack functions in a shared place might be a better approach. Going to have to ponder this a bit. JanReceived on 2016-04-22 14:04:59