Re: [PATCH 001 of 5] md: Split disks array out of raid5 conf structure so it is easier to grow.

>>>>> "NeilBrown" == NeilBrown  <[email protected]> writes:

NeilBrown> Previously the array of disk information was included in
NeilBrown> the raid5 'conf' structure which was allocated to an
NeilBrown> appropriate size.  This makes it awkward to change the size
NeilBrown> of that array.  So we split it off into a separate
NeilBrown> kmalloced array which will require a little extra indexing,
NeilBrown> but is much easier to grow.

Neil,

Instead of setting mddev->private = NULL, should you be doing a kfree
on it as well when you are in an abort state?

John
-
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/

Follow-Ups:
- Re: [PATCH 001 of 5] md: Split disks array out of raid5 conf structure so it is easier to grow.
  - From: Neil Brown <[email protected]>

References:
- [PATCH 000 of 5] md: Introduction
  - From: NeilBrown <[email protected]>
- [PATCH 001 of 5] md: Split disks array out of raid5 conf structure so it is easier to grow.
  - From: NeilBrown <[email protected]>

Prev by Date: make "struct d_cookie" dependable on CONFIG_PROFILING?
Next by Date: RFC [patch 20/34] PID Virtualization Use pid_to_vpid conversion functions
Previous by thread: [PATCH 001 of 5] md: Split disks array out of raid5 conf structure so it is easier to grow.
Next by thread: Re: [PATCH 001 of 5] md: Split disks array out of raid5 conf structure so it is easier to grow.
Index(es):
- Date
- Thread

[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]