All of lore.kernel.org
 help / color / mirror / Atom feed
From: Helge Deller <deller@gmx.de>
To: Rolf Eike Beer <eike@sf-mail.de>
Cc: John David Anglin <dave.anglin@bell.net>,
	Ming Lei <tom.leiming@gmail.com>,
	James Bottomley <James.Bottomley@hansenpartnership.com>,
	linux-block@vger.kernel.org,
	Linux SCSI List <linux-scsi@vger.kernel.org>,
	linux-parisc List <linux-parisc@vger.kernel.org>,
	Kent Overstreet <kent.overstreet@gmail.com>
Subject: Re: [BUG] "block: make generic_make_request handle arbitrarily sized bios" breaks boot on parisc-linux
Date: Sun, 20 Mar 2016 19:12:44 +0100	[thread overview]
Message-ID: <56EEE81C.4080302@gmx.de> (raw)
In-Reply-To: <4070811.QJ02MzEnrO@caliban.sf-tec.de>

Hi Eike,

On 10.03.2016 08:04, Rolf Eike Beer wrote:
>> Right.
>> I just succeeded in reproducing the bug with moving blk_bio_segment_split()
>> into an own file (and with "extern" instead of "static" in blk-merge.c).
>> When compiled with -O2 it still crashes. So, next step is to analyze what
>> gcc does wrong when compiling this function. It should get easier now to
>> find the reason, since we have a smaller reproducer now.
> 
> I have a ton of compilers here on my C8000 and a few even on my C3600 which 
> drive nightly CMake dashboards. Could you send the testcase so I can pass it 
> through the list and see what breaks?

Thanks for the offer!
Sadly it's not a stand-alone testcase, just the original source code extracted
which can be manually compiled and then the generated assembly can be compared
to what it should be.
That said, I think this specific testcase is not really usable in your test environment.

By the way, the bug was just fixed. Details are in bugzilla:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70188

Thanks,
Helge

      reply	other threads:[~2016-03-20 18:12 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-24  2:28 [BUG] "block: make generic_make_request handle arbitrarily sized bios" breaks boot on parisc-linux John David Anglin
2016-02-24  7:59 ` Ming Lei
2016-02-24 21:36   ` Helge Deller
2016-02-24 23:28     ` John David Anglin
2016-02-25  3:38       ` Ming Lei
2016-02-25 10:10         ` Aw: " Helge Deller
2016-03-09 12:55         ` James Bottomley
2016-03-09 14:43           ` Ming Lei
2016-03-09 15:15             ` John David Anglin
2016-03-09 15:51               ` Ming Lei
2016-03-09 21:20               ` Helge Deller
2016-03-10  0:16                 ` James Bottomley
2016-03-10  7:04                 ` Rolf Eike Beer
2016-03-20 18:12                   ` Helge Deller [this message]

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=56EEE81C.4080302@gmx.de \
    --to=deller@gmx.de \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=dave.anglin@bell.net \
    --cc=eike@sf-mail.de \
    --cc=kent.overstreet@gmail.com \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-parisc@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=tom.leiming@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.