On Thu, May 22, 2014 at 01:11:40PM +0200, Borislav Petkov wrote:
> Date: Thu, 22 May 2014 13:11:40 +0200
> From: Borislav Petkov <bp@xxxxxxxxx>
> To: "Chen, Gong" <gong.chen@xxxxxxxxxxxxxxx>
> Cc: tony.luck@xxxxxxxxx, m.chehab@xxxxxxxxxxx, linux-acpi@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 6/7 v3] trace, eMCA: Add a knob to adjust where to save
> event log
> User-Agent: Mutt/1.5.23 (2014-03-12)
>
> On Wed, May 21, 2014 at 07:46:26PM -0400, Chen, Gong wrote:
> > On Wed, May 21, 2014 at 01:06:31PM +0200, Borislav Petkov wrote:
> > > > + if (ras_userspace_consumers() == 0) {
> > >
> > > if (!ras_userspace_consumers())
> > >
> > No, it is not a pointer so I don't think it is very
> > meaningful just to save some bytes.
>
> Btw, this is exactly why your patches take too long to review - you like
> to debate more instead of listening to the maintainers. Next time you
> want to speed up the process, just think about that.
>
Sorry, I'm afraid I can't agree with you. Why should I just follow you
blindly without any my own voice? Even if some times I act like an
idiot. If I can't say anything for so small thing, do you expect I
should shout aloud for something like kernel infrastructure?
OK, I will change soon, as you wish.
Attachment:
signature.asc
Description: Digital signature
