This used to be behind an #ifdef COMPAT_COMPAT, so most of userspace
wouldn't have seen the definition before. Unfortunately this header
file became visible to userspace, so the definition has instead been
moved to net/atm/svc.c (the only user).
Signed-off-by: Palmer Dabbelt
Reviewed-by: Andrew
On Thu, 17 Sep 2015 02:57:12 PDT (-0700), dhowe...@redhat.com wrote:
> Palmer Dabbelt wrote:
>
>> +#ifdef CONFIG_COMPAT
>> +/* It actually takes struct sockaddr_atmsvc, not struct atm_iobuf */
>> +#define COMPAT_ATM_ADDPARTY _IOW('a', ATMIOC_SPECIAL+4, struct
>> compat_atm_iobuf)
>> +#endif
>
> I
Palmer Dabbelt wrote:
> +#ifdef CONFIG_COMPAT
> +/* It actually takes struct sockaddr_atmsvc, not struct atm_iobuf */
> +#define COMPAT_ATM_ADDPARTY _IOW('a', ATMIOC_SPECIAL+4, struct
> compat_atm_iobuf)
> +#endif
I wonder if it would hurt to ditch the conditionals entirely. It only eats
cpp n
This used to be behind an #ifdef COMPAT_COMPAT, so most of userspace
wouldn't have seen the definition before. Unfortunately this header
file became visible to userspace, so the definition has instead been
moved to net/atm/svc.c (the only user).
Signed-off-by: Palmer Dabbelt
Reviewed-by: Andrew
4 matches
Mail list logo