Skip to contentSkip to conversation

parisc: remove copat_ioctl entires for pp

PATCH Matthew Wilcox 1 participant 1 message Nov 12, 2005 1 message not in the archive

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

#1 lore raw
Quoted 8 lines · Christoph Hellwig

ppp ioctls handlers in parisc are wrong:

- they would need to do a copat_ptr on ifr->ifr_ifru.ifru_data
- SIOCGPPP* ioctls are defines as SIOCDEVPRIVATE + n, so they'd kick in for all private ioctls which is defintely wrong.

to implement proper ppp compat_ioctl handlig we'd need a compat_ioctl method in struct net_device, which we don't have yet.

While I agree that this is theoretically wrong, the problem is that if we remove them, people using ppp on 64-bit PA machines will no longer be able to do so. And yes, there are some, that's why these got added in the first place.

Perhaps I'll look into the compat_ioctl route later. In any case, I would prefer this patch not be added for the moment.
_______________________________________________
parisc-linux mailing list [email protected]
http://lists.parisc-linux.org/mailman/listinfo/parisc-linux