On 01/07/2013 02:41 PM, Seth Jennings wrote: > On 01/07/2013 02:32 PM, Greg Kroah-Hartman wrote: >> On Mon, Jan 07, 2013 at 02:24:36PM -0600, Seth Jennings wrote: >>> debugfs currently lack the ability to create attributes >>> that set/get atomic_t values. >> >> I hate to ask, but why would you ever want to do such a thing? > > There are a few atomic_t statistics in zswap that are valuable to have > in the debugfs attributes. Rather than have non-atomic mirrors of all > of them, as is done in zcache right now (see > drivers/staging/ramster/zcache-main.c:131), I thought this to be a > cleaner solution. > > Granted, I personally have no use for the setting part; only the > getting part. I only included the setting operations to keep the > balance and conform with the rest of the debugfs implementation.
Greg, I never did get your ack or rejection here. Are you ok with this patch? Thanks, Seth -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
