public inbox for linux-kernel@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox