Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1468001

Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size()

Show all headers | View raw

Hi Willy,

On 08/22/2016 09:35 AM, Willy Tarreau wrote:
> Hi Michael,
> 
> On Mon, Aug 22, 2016 at 09:15:35AM +1200, Michael Kerrisk (man-pages) wrote:
>> Hi Willy,
>>
>> Might you have a chance to further review of this patch series?
>> It would be great if you could, since much of it touches changes
>> made by you earlier.
> 
> Well, all I did there was implementing a suggestion from Linus, but I'm
> not a specialist at all there. However I've read all your series and at
> least with my limited knowledge, all I've read seems to make sense at
> the code matches the descriptions. I don't remember any particular trap
> in this place so I'm not worried.

Okay.

> I remember that I noticed this inaccuracy in the accounting but I
> estimated it was not important since the goal was to *limit* resource
> usage with a simple patch that we could easily backport. Your approach
> looks clean and possibly backportable if needed. That's all I can say
> I'm afraid :-/

No problem. Thanks for the reply!

Cheers,

Michael



-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 4/8] pipe: fix limit checking in pipe_set_size() "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2016-08-19 07:50 +0200
  Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() Willy Tarreau <w@1wt.eu> - 2016-08-19 07:50 +0200
    Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2016-08-19 23:00 +0200
    Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2016-08-21 23:20 +0200
      Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() Willy Tarreau <w@1wt.eu> - 2016-08-21 23:40 +0200
        Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2016-08-22 21:40 +0200
  Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() Vegard Nossum <vegard.nossum@oracle.com> - 2016-08-19 10:40 +0200
    Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2016-08-19 23:00 +0200
      Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2016-08-20 01:20 +0200
        Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() Vegard Nossum <vegard.nossum@oracle.com> - 2016-08-21 12:40 +0200
          Re: [PATCH 4/8] pipe: fix limit checking in pipe_set_size() "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2016-08-21 23:20 +0200

csiph-web