On Tue, 17.09.13 14:21, Chen Hanxiao ([email protected]) wrote:

> +        { "MemorySWLimit",           bus_property_append_uint64,          
> "t",     offsetof(CGroupContext, memsw_limit)       },
>          { "MemorySoftLimit",         bus_property_append_uint64,          
> "t",     offsetof(CGroupContext, memory_soft_limit)  },

The patch looks fine in principle, but I really don't like the
abbreviated name like this. Can we name this "MemorySwapLimit=" or so?
Or "MemoryAndSwapLimit="? Or "MemoryPlusSwapLimit="? Or
"MemoryIncludingSwapLimit="? "MemorySwapSumLimit="?

I don't particularly like either of the latter really, anybody else got
suggestions?

The one that is the most descriptive fromt he above is probably
"MemoryIncludingSwapLimit=", even if it sucks hard to type...

Lennart

-- 
Lennart Poettering - Red Hat, Inc.
_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to