memcg: Fix build warning and avoid checking for mem != null again and again
Subsystems (2)
PATCH RESEND 1/1 memcg: Fix build warning and avoid checking for mem != null again and again
Resending the patch to Andrew for inclusion in -mm tree.
Thanks
Nikanth
Fix build warning, "mem_cgroup_is_obsolete defined but not used" when CONFIG_DEBUG_VM is not set. Also avoid checking for !mem again and again.
---
mm/memcontrol.c +1 -9
@@ -314,14 +314,6 @@ static struct mem_cgroup *try_get_mem_cgroup_from_mm(struct mm_struct *mm)return mem; } -static bool mem_cgroup_is_obsolete (struct mem_cgroup *mem) -{ -if (!mem) -return true; -return css_is_removed (&mem->css); -} - -/* * Call callback function against all cgroup under hierarchy tree. */
@@ -932,7 +924,7 @@ static int __mem_cgroup_try_charge(struct mm_struct *mm,if (unlikely (!mem))return 0 ; -VM_BUG_ON (!mem ||mem_cgroup_is_obsolete (mem)); +VM_BUG_ON (css_is_removed (&mem->css));while (1 ) {int ret;
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>
Quoted 12 lines · On Thu, 28 May 2009 14:20:43 +0530 Nikanth Karthikesan <[email protected]> wrote:
Resending the patch to Andrew for inclusion in -mm tree.
Thanks
NikanthFix build warning, "
mem_cgroup_is_obsoletedefined but not used" whenCONFIG_DEBUG_VMis not set. Also avoid checking for !mem again and again.Signed-off-by Nikanth Karthikesan <[email protected]>Acked-by Pekka Enberg <[email protected]>
Quoted 33 lines
---
mm/memcontrol.c +1
-10@@ -314,14 +314,6 @@ static struct mem_cgroup *try_get_mem_cgroup_from_mm(struct mm_struct *mm)return mem; } -static bool mem_cgroup_is_obsolete (struct mem_cgroup *mem) -{ -if (!mem) -return true; -return css_is_removed (&mem->css); -} - -/* * Call callback function against all cgroup under hierarchy tree. */ @@ -932,7 +924,7 @@ static int __mem_cgroup_try_charge(struct mm_struct *mm,if (unlikely (!mem))return 0 ; -VM_BUG_ON (!mem ||mem_cgroup_is_obsolete (mem)); +VM_BUG_ON (css_is_removed (&mem->css));while (1 ) {int ret; --
Quoted 5 lines
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>