NeilBrown <[email protected]> wrote:
>
> + /* Got them all.
> + * Return the new ones and free the old ones.
> + * At this point, we are holding all the stripes so the array
> + * is completely stalled, so now is a good time to resize
> + * conf->disks.
> + */
> + ndisks = kzalloc(newsize * sizeof(struct disk_info), GFP_NOIO);
> + if (ndisks) {
> + for (i=0; i<conf->raid_disks; i++)
> + ndisks[i] = conf->disks[i];
> + kfree(conf->disks);
> + conf->disks = ndisks;
> + } else
> + err = -ENOMEM;
> + while(!list_empty(&newstripes)) {
> + nsh = list_entry(newstripes.next, struct stripe_head, lru);
> + list_del_init(&nsh->lru);
> + for (i=conf->raid_disks; i < newsize; i++)
> + if (nsh->dev[i].page == NULL) {
> + struct page *p = alloc_page(GFP_NOIO);
> + nsh->dev[i].page = p;
> + if (!p)
> + err = -ENOMEM;
> + }
> + release_stripe(nsh);
> + }
> + while(!list_empty(&oldstripes)) {
> + osh = list_entry(oldstripes.next, struct stripe_head, lru);
> + list_del(&osh->lru);
> + kmem_cache_free(conf->slab_cache, osh);
> + }
> + kmem_cache_destroy(conf->slab_cache);
> + conf->slab_cache = sc;
> + conf->active_name = 1-conf->active_name;
> + conf->pool_size = newsize;
> + return err;
> +}
Are you sure the -ENOMEM handling here is solid? It looks.... strange.
There are a few more GFP_NOIOs in this function, which can possibly become
GFP_KERNEL.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
