Re: [PATCH] Btrfs-progs: update btrfs-debug-tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Chris Mason wrote:
> On Thu, 2009-02-19 at 19:39 -0500, Jeff Mahoney wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> Shen Feng wrote:
>>> output objectid in btrfs_disk_key with human readable strings.
>>> Other updates are included for more readable output.
>> This gets messy fast. I'd like to see some sort of generic extensible
>> printk mechanism in the kernel. 
> 
> This is a btrfs-progs patch ;)

Heh, oops. That's an important bit of information.

>> Maybe I've been spoiled by the reiserfs
>> type printing but it seems a lot nicer to do eprintk(&ops, "key: %k\n");
>> than to split them out into different printks. Since the core behavior
>> ends up in vsnprintf, you get it for things like panic for free too.
>> Ideally, the tprintk(&ops, ...) would be hidden behind a btrfs_warning()
>> or something.
>>
>> I have a patch set that implements an extensible printk so that every
>> single pointer type doesn't need to be added to lib/vsprintf.c's pointer().
>>
> 
> That would be interesting, I do miss the old reiserfs %k

Let me clean them up a bit and send to LKML. I can probably do that
later today. I decided to rename it to eprintk from a pretty ugly
printk_custom.

- -Jeff

- --
Jeff Mahoney
SUSE Labs
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (GNU/Linux)
Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org

iEYEARECAAYFAkme2SMACgkQLPWxlyuTD7ITSwCeO3XaNGY29kA/5oBcdDL7DqpH
B5wAnRv7RYjzX5h4pX+FbjTGTBGAFy7S
=a5iu
-----END PGP SIGNATURE-----
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux