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

Groups > linux.kernel > #1178707 > unrolled thread

Re: [PATCH RFC 0/5] kdbus: minor quota code improvements

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH RFC 0/5] kdbus: minor quota code improvements Sergei Zviagintsev <sergei@s15v.net> - 2015-07-07 18:30 +0200
    Re: [PATCH RFC 0/5] kdbus: minor quota code improvements Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2015-07-08 00:30 +0200

#1178707 — Re: [PATCH RFC 0/5] kdbus: minor quota code improvements

Hi David,

On Mon, Jul 06, 2015 at 08:48:26PM +0200, David Herrmann wrote:
> Hi
> 
> On Sun, Jun 28, 2015 at 3:17 PM, Sergei Zviagintsev <sergei@s15v.net> wrote:
> > Hi,
> >
> > Main points here are to use conventional types, rewrite tests in obvious
> > way, add couple of new WARN_ONs and do a bit more work in the others.
> >
> > Keeping in mind the number of mistakes I made while preparing this
> > patchset, I decided to mark it as RFC :) I'd appreciate any feedback on
> > these.
> >
> >
> > Sergei Zviagintsev (5):
> >   kdbus: fix typos in kdbus_conn_quota_inc()
> >   kdbus: use standard kernel types in struct kdbus_quota
> >   kdbus: do explicit overflow check in kdbus_conn_quota_inc()
> >   kdbus: handle WARN_ON cases properly when decrementing quota
> >   kdbus: improve tests on incrementing quota
> 
> I applied #1 and #2 now. If you resend, please drop those from the set.

As I see, neither Greg's tree nor out-of-tree kdbus repo
(i.e. https://github.com/systemd/kdbus) contains them. Am I missing
something?

> 
> Thanks!
> David
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1179125

On Tue, Jul 07, 2015 at 07:29:41PM +0300, Sergei Zviagintsev wrote:
> Hi David,
> 
> On Mon, Jul 06, 2015 at 08:48:26PM +0200, David Herrmann wrote:
> > Hi
> > 
> > On Sun, Jun 28, 2015 at 3:17 PM, Sergei Zviagintsev <sergei@s15v.net> wrote:
> > > Hi,
> > >
> > > Main points here are to use conventional types, rewrite tests in obvious
> > > way, add couple of new WARN_ONs and do a bit more work in the others.
> > >
> > > Keeping in mind the number of mistakes I made while preparing this
> > > patchset, I decided to mark it as RFC :) I'd appreciate any feedback on
> > > these.
> > >
> > >
> > > Sergei Zviagintsev (5):
> > >   kdbus: fix typos in kdbus_conn_quota_inc()
> > >   kdbus: use standard kernel types in struct kdbus_quota
> > >   kdbus: do explicit overflow check in kdbus_conn_quota_inc()
> > >   kdbus: handle WARN_ON cases properly when decrementing quota
> > >   kdbus: improve tests on incrementing quota
> > 
> > I applied #1 and #2 now. If you resend, please drop those from the set.
> 
> As I see, neither Greg's tree nor out-of-tree kdbus repo
> (i.e. https://github.com/systemd/kdbus) contains them. Am I missing
> something?

David has sent me a pull request for these, give me a few days to catch
up on my patches before I pull it in.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web