NetBSD Problem Report #33513

From yamt@mwd.biglobe.ne.jp  Fri May 19 13:06:01 2006
Return-Path: <yamt@mwd.biglobe.ne.jp>
Received: from yamt.dyndns.org (FLA1Ace136.kng.mesh.ad.jp [203.136.242.136])
	by narn.NetBSD.org (Postfix) with ESMTP id 8B6E163BA67
	for <gnats-bugs@gnats.NetBSD.org>; Fri, 19 May 2006 13:06:00 +0000 (UTC)
Message-Id: <1148043944.712247.2124.nullmailer@yamt.dyndns.org>
Date: Fri, 19 May 2006 22:05:44 +0900
From: yamt@mwd.biglobe.ne.jp
Reply-To: yamt@mwd.biglobe.ne.jp
To: gnats-bugs@NetBSD.org
Subject: vnode dirtiness problem
X-Send-Pr-Version: 3.95

>Number:         33513
>Category:       kern
>Synopsis:       vnode dirtiness problem
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    kern-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri May 19 13:10:00 +0000 2006
>Last-Modified:  Sat Oct 14 09:25:05 +0000 2006
>Originator:     YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
>Release:        NetBSD 3.99.17
>Organization:

>Environment:


System: NetBSD kaeru 3.99.17
Architecture: i386
Machine: i386
>Description:
	there are some code which explicitly mark pages dirty by clearing
	PG_CLEAN.  if the pages belong to a vnode, the vnode should be
	marked dirty as well.

	i guess what Joel CARNAT reported on port-xen@ is a symptom of
	this bug.
	http://mail-index.NetBSD.org/port-xen/2006/05/12/0000.html
	http://mail-index.NetBSD.org/port-xen/2006/05/18/0000.html
>How-To-Repeat:
	code inspection.
>Fix:
	when making a page dirty,
	- mark the vnode dirty.
	- if the filesystem uses genfs, increment g_dirtygen somehow.

>Audit-Trail:
From: YAMAMOTO Takashi <yamt@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: PR/33513 CVS commit: src/sys/uvm
Date: Sat, 14 Oct 2006 09:23:56 +0000 (UTC)

 Module Name:	src
 Committed By:	yamt
 Date:		Sat Oct 14 09:23:56 UTC 2006

 Modified Files:
 	src/sys/uvm: uvm_loan.c

 Log Message:
 uvm_loanbreak: transfer dirtiness of the old page to the new page,
 rather than blindly mark it dirty.  fix a part of PR/33513.


 To generate a diff of this commit:
 cvs rdiff -r1.60 -r1.61 src/sys/uvm/uvm_loan.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

>Unformatted:

Home	PR Database Search