xfs
[Top] [All Lists]

Re: [PATCH] buffer flag NONE is actually DONE

To: Timothy Shimmin <tes@xxxxxxx>
Subject: Re: [PATCH] buffer flag NONE is actually DONE
From: Lachlan McIlroy <lachlan@xxxxxxx>
Date: Wed, 03 Sep 2008 14:04:07 +1000
Cc: xfs-dev <xfs-dev@xxxxxxx>, xfs-oss <xfs@xxxxxxxxxxx>
In-reply-to: <48BCE564.5070408@sgi.com>
References: <48BCC8C2.7040103@sgi.com> <48BCE564.5070408@sgi.com>
Reply-to: lachlan@xxxxxxx
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Thunderbird 2.0.0.16 (X11/20080707)
How's this then?  There's a few flags that don't exist anymore.

--- a/fs/xfs/xfsidbg.c  2008-09-03 13:53:12.000000000 +1000
+++ b/fs/xfs/xfsidbg.c  2008-09-03 13:59:47.000000000 +1000
@@ -1925,11 +1925,11 @@ static int      kdbm_iptraceaddr(


static char *bp_flag_vals[] = { -/* 0 */ "READ", "WRITE", "MAPPED", "PARTIAL", "ASYNC", -/* 5 */ "NONE", "DELWRI", "STALE", "FS_MANAGED", "FS_DATAIOD", -/* 10 */ "FORCEIO", "FLUSH", "READ_AHEAD", "DIRECTIO", "LOCK", -/* 15 */ "TRYLOCK", "DONT_BLOCK", "PAGE_CACHE", "KMEM_ALLOC", "RUN_QUEUES", -/* 20 */ "PRIVATE_BH", "DELWRI_Q", +/* 0 */ "READ", "WRITE", "MAPPED", "<unknown(3)>", "ASYNC", +/* 5 */ "DONE", "DELWRI", "STALE", "FS_MANAGED", "<unknown(9)>", +/* 10 */ "<unknown(10)>", "ORDERED", "READ_AHEAD", "<unknown(13)>", "LOCK", +/* 15 */ "TRYLOCK", "DONT_BLOCK", "PAGE_CACHE", "PAGES", "RUN_QUEUES", +/* 20 */ "<unknown(20)>", "DELWRI_Q", "PAGE_LOCKED", NULL };

static char     *iomap_flag_vals[] = {


Timothy Shimmin wrote:
Lachlan McIlroy wrote:
Amazing but true.

--- a/fs/xfs/xfsidbg.c    2008-09-02 14:52:11.000000000 +1000
+++ b/fs/xfs/xfsidbg.c    2008-09-02 14:51:04.000000000 +1000
@@ -1926,7 +1926,7 @@ static int    kdbm_iptraceaddr(

static char    *bp_flag_vals[] = {
/*  0 */ "READ", "WRITE", "MAPPED", "PARTIAL", "ASYNC",
-/*  5 */ "NONE", "DELWRI",  "STALE", "FS_MANAGED", "FS_DATAIOD",
+/*  5 */ "DONE", "DELWRI",  "STALE", "FS_MANAGED", "FS_DATAIOD",
/* 10 */ "FORCEIO", "FLUSH", "READ_AHEAD", "DIRECTIO", "LOCK",
/* 15 */ "TRYLOCK", "DONT_BLOCK", "PAGE_CACHE", "KMEM_ALLOC", "RUN_QUEUES",
/* 20 */ "PRIVATE_BH", "DELWRI_Q",

Looks good.

Quick look at possible mismatching names:

11 "FLUSH" vs "ORDERED"
18 "KMEM_ALLOC" vs "PAGES"
22 NULL vs "PAGE_LOCKED"

--Tim



<Prev in Thread] Current Thread [Next in Thread>