<p dir="ltr"><br>
在 2013-5-10 PM1:41，"MORITA Kazutaka" <<a href="mailto:morita.kazutaka@lab.ntt.co.jp">morita.kazutaka@lab.ntt.co.jp</a>>写道：<br>
><br>
> At Fri, 10 May 2013 13:33:00 +0800,<br>
> Liu Yuan wrote:<br>
> ><br>
> > @@ -608,13 +609,12 @@ static int do_plug_unplug(char *disks, bool plug)<br>
> >       old_nr = md_nr_disks;<br>
> >       path = strtok(disks, ",");<br>
> >       do {<br>
> > -             if (purge_directory(path) < 0)<br>
> > -                     goto out;<br>
> > -<br>
> > -             if (plug)<br>
> > -                     md_add_disk(path);<br>
> > -             else<br>
> > +             if (plug) {<br>
> > +                     if (md_add_disk(path) && purge_directory(path) < 0)<br>
> > +                                     goto out;<br>
><br>
> Wrong indentation.<br>
><br>
> What happens if md_add_disk() succeeds but purge_directory() fails?<br>
> Should we call md_del_disk() against the added disk?<br>
></p>
<p dir="ltr">no, we dont plug any disks in that case<br>
> Thanks,<br>
><br>
> Kazutaka<br>
</p>