* [PATCH] use check_block_and_slaves in for_each_host_dev_and_slaves
@ 2012-08-28 2:05 Dave Young
[not found] ` <20120828020549.GA6961-je1gSBvt1Td3da3rpXeqgR/sF2h8X+2i0E9HWUfgJXw@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Dave Young @ 2012-08-28 2:05 UTC (permalink / raw)
To: harald-H+wXaHxf7aLQT0dZR+AlfA, chaowang-H+wXaHxf7aLQT0dZR+AlfA,
vgoyal-H+wXaHxf7aLQT0dZR+AlfA, initramfs-u79uwXL29TY76Z2rM5mHXA
for_each_host_dev_and_slaves currently is used in some module check()
functions and it's not necessary iterate all slaves. So use
check_block_and_slaves instead of check_block_and_slaves_all is fine.
Signed-off-by: Dave Young <dyoung-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
---
dracut-functions.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- dracut.orig/dracut-functions.sh
+++ dracut/dracut-functions.sh
@@ -468,7 +468,7 @@ for_each_host_dev_and_slaves()
local _dev
for _dev in ${host_devs[@]}; do
[[ -b "$_dev" ]] || continue
- check_block_and_slaves_all $_func $(get_maj_min $_dev) && return 0
+ check_block_and_slaves $_func $(get_maj_min $_dev) && return 0
done
return 1
}
^ permalink raw reply [flat|nested] 2+ messages in thread[parent not found: <20120828020549.GA6961-je1gSBvt1Td3da3rpXeqgR/sF2h8X+2i0E9HWUfgJXw@public.gmane.org>]
* Re: [PATCH] use check_block_and_slaves in for_each_host_dev_and_slaves [not found] ` <20120828020549.GA6961-je1gSBvt1Td3da3rpXeqgR/sF2h8X+2i0E9HWUfgJXw@public.gmane.org> @ 2012-08-28 2:08 ` Dave Young 0 siblings, 0 replies; 2+ messages in thread From: Dave Young @ 2012-08-28 2:08 UTC (permalink / raw) To: harald-H+wXaHxf7aLQT0dZR+AlfA, chaowang-H+wXaHxf7aLQT0dZR+AlfA, vgoyal-H+wXaHxf7aLQT0dZR+AlfA, initramfs-u79uwXL29TY76Z2rM5mHXA On 08/28/2012 10:05 AM, Dave Young wrote: > > for_each_host_dev_and_slaves currently is used in some module check() > functions and it's not necessary iterate all slaves. So use > check_block_and_slaves instead of check_block_and_slaves_all is fine. Hi, harald Thanks for cleanup the redundant "echo" lines in my last patch series Could you apply this cleanup as well? > > Signed-off-by: Dave Young <dyoung-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> > --- > dracut-functions.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > --- dracut.orig/dracut-functions.sh > +++ dracut/dracut-functions.sh > @@ -468,7 +468,7 @@ for_each_host_dev_and_slaves() > local _dev > for _dev in ${host_devs[@]}; do > [[ -b "$_dev" ]] || continue > - check_block_and_slaves_all $_func $(get_maj_min $_dev) && return 0 > + check_block_and_slaves $_func $(get_maj_min $_dev) && return 0 > done > return 1 > } > -- > To unsubscribe from this list: send the line "unsubscribe initramfs" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Thanks Dave ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-08-28 2:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-28 2:05 [PATCH] use check_block_and_slaves in for_each_host_dev_and_slaves Dave Young
[not found] ` <20120828020549.GA6961-je1gSBvt1Td3da3rpXeqgR/sF2h8X+2i0E9HWUfgJXw@public.gmane.org>
2012-08-28 2:08 ` Dave Young
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.