Bug 467093

Summary:	kernel warning when booting XO
Product:	[Fedora] Fedora	Reporter:	Bryan Kearney <bkearney>
Component:	kernel	Assignee:	Kernel Maintainer List <kernel-maint>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	cebbert, dcantrell, jlaska, jpechane, kernel-maint, lmacken, tcallawa
Target Milestone:	---	Keywords:	Reopened
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:	NEEDSRETESTING
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2008-11-13 16:10:29 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	461806, 466414

Description Bryan Kearney 2008-10-15 17:45:52 UTC

Description of problem:

The following kernel.oops shows up with regularity when booting an XO with Snap1.

NET: Registered protocol family 1

checking if image is initramfs... it is

------------[ cut here ]------------

WARNING: at arch/x86/mm/pageattr.c:782 change_page_attr_set_clr+0x5e/0x198() (Not tainted)

Modules linked in:

Pid: 1, comm: swapper Not tainted 2.6.27-1.fc10.i686 #1

 [<c06d8fc3>] ? printk+0x14/0x19

 [<c042d224>] warn_on_slowpath+0x4c/0x7a

 [<c04059cb>] ? mcount_call+0x5/0xa

 [<c04bcd8a>] ? inotify_dentry_parent_queue_event+0xe/0x88

 [<c04059cb>] ? mcount_call+0x5/0xa

 [<c047657d>] ? __free_one_page+0x17f/0x1c4

 [<c041c9c1>] change_page_attr_set_clr+0x5e/0x198

 [<c046cc28>] ? ftrace_record_ip+0x1ac/0x1c5

 [<c046cc28>] ? ftrace_record_ip+0x1ac/0x1c5

 [<c041cbae>] set_memory_rw+0x16/0x18

 [<c041b7d5>] free_init_pages+0x26/0x96

 [<c041b85b>] free_initrd_mem+0x16/0x18

 [<c08507c0>] free_initrd+0x8a/0xa4

 [<c085085a>] populate_rootfs+0x80/0xd2

 [<c040112a>] _stext+0x42/0x11a

 [<c08507da>] ? populate_rootfs+0x0/0xd2

 [<c0467b45>] ? register_irq_proc+0x97/0xb3

 [<c04059cb>] ? mcount_call+0x5/0xa

 [<c0467abc>] ? register_irq_proc+0xe/0xb3

 [<c0467ba3>] ? init_irq_proc+0x42/0x4f

 [<c084b3a8>] kernel_init+0x17c/0x1ca

 [<c084b22c>] ? kernel_init+0x0/0x1ca

 [<c04059b3>] kernel_thread_helper+0x7/0x10

 =======================

---[ end trace a7919e7f17c0a725 ]---

Freeing initrd memory: 10033k freed

Comment 1 Chuck Ebbert 2008-11-02 00:57:40 UTC

arch/x86/mm/pageattr.c:
        /* Ensure we are PAGE_SIZE aligned */
        if (addr & ~PAGE_MASK) {
                addr &= PAGE_MASK;
                /* 
                 * People should not be passing in unaligned addresses: 
                 */
                WARN_ON_ONCE(1);
        }

Comment 2 Jeremy Katz 2008-11-03 16:06:03 UTC

Assuming this is on freeing the ramdisk (whichit looks like), it looks like the ramdisk address is page aligned to me.

Comment 3 Alexander Todorov 2008-11-06 08:35:49 UTC

*** Bug 470182 has been marked as a duplicate of this bug. ***

Comment 4 Jeremy Katz 2008-11-06 20:12:48 UTC

*** Bug 470347 has been marked as a duplicate of this bug. ***

Comment 5 Jeremy Katz 2008-11-07 22:02:33 UTC

Aha, no it wasn't.

I've updated http://katzj.fedorapeople.org/olpc/livecd-iso-to-disk.sh -- if you try it now, can you confirm that you don't see this warning anymore?  It seems to do the trick for me.

Comment 6 Jesse Keating 2008-11-12 16:23:32 UTC

Can somebody please re-test this so that we can confirm the fix?

Comment 7 James Laska 2008-11-12 17:43:53 UTC

After several reboots ... not seeing this as of a livecd-creator built image from today.

Comment 8 Jesse Keating 2008-11-12 19:03:41 UTC

James, that was using jeremy's livecd-iso-to-disk right?  If so, this needs to be re-opened until a package with that change is tagged.

Comment 9 James Laska 2008-11-12 19:13:02 UTC

yeah, I grabbed a livecd-iso-to-disk.sh this morning and built a new Live image this morning.

# md5sum /tmp/livecd-iso-to-disk.sh 
6dc19fe6ed50ea70b7568ba193cdff5a  /tmp/livecd-iso-to-disk.sh

Apologies ... it slipped my mind that the script (http://katzj.fedorapeople.org/olpc/livecd-iso-to-disk.sh) wasn't yet added to livecd-tools.  

Should we re-assign this to livecd-tools?

Comment 10 Jeremy Katz 2008-11-12 19:32:06 UTC

Thanks for the verification jlaska.  Pushed changes to the livecd-tools repo and I'm intending to get a release and build of it done tonight (changes are almost entirely livecd-iso-to-disk changes)

Comment 11 Jeremy Katz 2008-11-13 16:10:29 UTC

And livecd-tools built