Re: [PATCH net-next] sctp: Pull the user copies out of the individual sockopt functions.

2020-05-21 Thread 'Marcelo Ricardo Leitner'
On Thu, May 21, 2020 at 04:09:15PM +, David Laight wrote: > From: 'Marcelo Ricardo Leitner' > > Sent: 21 May 2020 16:37 > > On Wed, May 20, 2020 at 03:08:13PM +, David Laight wrote: > ... > > > Only SCTP_SOCKOPT_CONNECTX3 contains an indirect pointer. > > > It is also the only getsockopt()

RE: [PATCH net-next] sctp: Pull the user copies out of the individual sockopt functions.

2020-05-21 Thread David Laight
From: 'Marcelo Ricardo Leitner' > Sent: 21 May 2020 16:37 > On Wed, May 20, 2020 at 03:08:13PM +, David Laight wrote: ... > > Only SCTP_SOCKOPT_CONNECTX3 contains an indirect pointer. > > It is also the only getsockopt() that wants to return a buffer > > and an error code. It is also definitely

RE: [PATCH net-next] sctp: Pull the user copies out of the individual sockopt functions.

2020-05-21 Thread David Laight
From: Christoph Hellwig > Sent: 21 May 2020 16:40 > On Thu, May 21, 2020 at 12:37:29PM -0300, 'Marcelo Ricardo Leitner' wrote: > > On Wed, May 20, 2020 at 03:08:13PM +, David Laight wrote: > > > > I wish we could split this patch into multiple ones. Like, one for > > each sockopt, but it doesn'

Re: [PATCH net-next] sctp: Pull the user copies out of the individual sockopt functions.

2020-05-21 Thread Christoph Hellwig
On Thu, May 21, 2020 at 12:37:29PM -0300, 'Marcelo Ricardo Leitner' wrote: > On Wed, May 20, 2020 at 03:08:13PM +, David Laight wrote: > > I wish we could split this patch into multiple ones. Like, one for > each sockopt, but it doesn't seem possible. It's tough to traverse > trough 5k lines l

Re: [PATCH net-next] sctp: Pull the user copies out of the individual sockopt functions.

2020-05-21 Thread 'Marcelo Ricardo Leitner'
On Wed, May 20, 2020 at 03:08:13PM +, David Laight wrote: I wish we could split this patch into multiple ones. Like, one for each sockopt, but it doesn't seem possible. It's tough to traverse trough 5k lines long patch. :-( > Since SCTP rather abuses getsockopt() to perform operations and use

Re: [PATCH net-next] sctp: Pull the user copies out of the individual sockopt functions.

2020-05-21 Thread 'Marcelo Ricardo Leitner'
On Thu, May 21, 2020 at 07:32:14AM +, David Laight wrote: > From: 'Marcelo Ricardo Leitner' > > Sent: 21 May 2020 01:17 > > On Wed, May 20, 2020 at 03:08:13PM +, David Laight wrote: > > ... > > > Only SCTP_SOCKOPT_CONNECTX3 contains an indirect pointer. > > > It is also the only getsockopt(

RE: [PATCH net-next] sctp: Pull the user copies out of the individual sockopt functions.

2020-05-21 Thread David Laight
From: 'Marcelo Ricardo Leitner' > Sent: 21 May 2020 01:17 > On Wed, May 20, 2020 at 03:08:13PM +, David Laight wrote: > ... > > Only SCTP_SOCKOPT_CONNECTX3 contains an indirect pointer. > > It is also the only getsockopt() that wants to return a buffer > > and an error code. It is also definite

Re: [PATCH net-next] sctp: Pull the user copies out of the individual sockopt functions.

2020-05-20 Thread 'Marcelo Ricardo Leitner'
On Wed, May 20, 2020 at 03:08:13PM +, David Laight wrote: ... > Only SCTP_SOCKOPT_CONNECTX3 contains an indirect pointer. > It is also the only getsockopt() that wants to return a buffer > and an error code. It is also definitely abusing getsockopt(). ... > @@ -1375,11 +1350,11 @@ struct compat

Re: [PATCH net-next] sctp: Pull the user copies out of the individual sockopt functions.

2020-05-20 Thread kbuild test robot
Hi David, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on net-next/master] [also build test WARNING on net/master linus/master v5.7-rc6 next-20200519] [cannot apply to ipvs/master] [if your patch is applied to the wrong git tree, please drop us a note to help im