Re: [PATCH 005 of 9] md: Replace magic numbers in sb_dirty with well defined bit flags

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

 



Ingo Oeser wrote:

Hi Neil,

I think the names in this patch don't match the description at all.
May I suggest different ones?

On Monday, 31. July 2006 09:32, NeilBrown wrote:
Instead of magic numbers (0,1,2,3) in sb_dirty, we have
some flags instead:
MD_CHANGE_DEVS
  Some device state has changed requiring superblock update
  on all devices.

MD_SB_STALE or MD_SB_NEED_UPDATE
I think STALE is better, it is unambigous.

MD_CHANGE_CLEAN
  The array has transitions from 'clean' to 'dirty' or back,
  requiring a superblock update on active devices, but possibly
  not on spares

Maybe split this into MD_SB_DIRTY and MD_SB_CLEAN ?
I don't think the split is beneficial, but I don't care for the name much. Some name like SB_UPDATE_NEEDED or the like might be better.

MD_CHANGE_PENDING
A superblock update is underway.

MD_SB_PENDING_UPDATE

I would have said UPDATE_PENDING, but either is more descriptive than the original.

Neil - the logic in this code is pretty complex, all the help you can give the occasional reader, by using very descriptive names for things, is helpful to the reader and reduces your "question due to misunderstanding" load.

--
bill davidsen <[email protected]>
 CTO TMR Associates, Inc
 Doing interesting things with small computers since 1979

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux