All of lore.kernel.org
 help / color / mirror / Atom feed
* [dave@hiauly1.hia.nrc.ca: Re: [parisc-linux] Why gas kills the hppa-linux kernel and what you can]
@ 2005-07-01 17:53 Carlos O'Donell
  2005-07-01 18:07 ` James Bottomley
  0 siblings, 1 reply; 7+ messages in thread
From: Carlos O'Donell @ 2005-07-01 17:53 UTC (permalink / raw)
  To: parisc-linux; +Cc: James Bottomley, Randolph Chung


Dave,

Forwarding your valuable comments to the list for analysis :)

c.


----- Forwarded message from John David Anglin <dave@hiauly1.hia.nrc.ca> -----

> > Backtrace:   
> >  [<101c10a4>] journal_add_journal_head+0xdc/0x13c

I think this is a bug in journal_add_journal_head:

repeat:
        if (!buffer_jbd(bh)) {
		new_jh = journal_alloc_journal_head();
		memset(new_jh, 0, sizeof(*new_jh));
	}

journal_alloc_journal_head() can return a null pointer causing
the kernel to die in memset.  I think the fix is to skip calling
memset when new_jh is null.  The rest of the code looks ok except
for possibly

                if (!new_jh) {
			jbd_unlock_bh_journal_head(bh);
			goto repeat;
		}

This might loop forever.  So, somebody who knows how this is
supposed to work should look at it.

Dave
-- 
J. David Anglin                                  dave.anglin@nrc-cnrc.gc.ca
National Research Council of Canada              (613) 990-0752 (FAX: 952-6602)

----- End forwarded message -----
_______________________________________________
parisc-linux mailing list
parisc-linux@lists.parisc-linux.org
http://lists.parisc-linux.org/mailman/listinfo/parisc-linux

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2005-07-01 19:47 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-07-01 17:53 [dave@hiauly1.hia.nrc.ca: Re: [parisc-linux] Why gas kills the hppa-linux kernel and what you can] Carlos O'Donell
2005-07-01 18:07 ` James Bottomley
2005-07-01 18:38   ` [dave@hiauly1.hia.nrc.ca: Re: [parisc-linux] Why gas kills the John David Anglin
2005-07-01 19:08     ` Carlos O'Donell
2005-07-01 19:12     ` Carlos O'Donell
2005-07-01 19:47       ` Carlos O'Donell
2005-07-01 19:05   ` [dave@hiauly1.hia.nrc.ca: Re: [parisc-linux] Why gas kills the hppa-linux kernel and what you can] Carlos O'Donell

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.