All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Theodore Ts'o" <tytso@mit.edu>
To: Nick Krause <xerofoify@gmail.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: Arm Compiler - Part 1 of Compiling Tests
Date: Mon, 7 Jul 2014 09:46:37 -0400	[thread overview]
Message-ID: <20140707134637.GA31174@thunk.org> (raw)
In-Reply-To: <CAPDOMVieG951+0chzvjmKTdpZf-fEB5mjUL9YK-rgqF3+O537w@mail.gmail.com>

On Mon, Jul 07, 2014 at 01:22:13AM -0400, Nick Krause wrote:
> Here are my logs of the builds attached  with warnings if they succeed
> for now failing arm configs
> according to the tests here,

fs/direct-io.c: In function ‘__blockdev_direct_IO’:
fs/direct-io.c:1011:12: warning: ‘to’ may be used uninitialized in this
function [-Wmaybe-uninitialized]
    u = (to - from) >> blkbits;

OK, do you see why this is a false positive?  And why asking the
thousands of people "in the commmunity" to all do exactly the same
evaluation is a massive waste of time?

And why people, after doing a quick evaluation to determine that the
very first warning you sent out (which was repeated multiple times in
your log; you didn't even bother to winnow out duplicate warnings)
is a false positive, might be inclined to ignore all e-mails from
you "asking for help" in the future?

Look, it's good that you're being enthusiastic.  But you need to do
more than just send screen shots of a kernel bugzilla where it's
already been explained to you that darned few people care about the
open/closed statistics, or running builds to complain about warnings.

If you want to send a patch to clean up the warning, figure out how to
do that, and then to send the to the right people.  (Hint: reading the
Documentation/SubmittingPatches and Documentation/Submitchecklist
files.)

Regards,

						- Ted

  reply	other threads:[~2014-07-07 13:46 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-07  5:22 Arm Compiler - Part 1 of Compiling Tests Nick Krause
2014-07-07 13:46 ` Theodore Ts'o [this message]
2014-07-07 17:35   ` Nick Krause
2014-07-07 18:22     ` Paul Bolle
2014-07-07 20:10       ` Nick Krause
2014-07-08 14:37     ` Jason Cooper

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140707134637.GA31174@thunk.org \
    --to=tytso@mit.edu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=xerofoify@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.