On Fri, Mar 24, 2017 at 1:57 PM, NeilBrown <ne...@suse.com> wrote:
> On Fri, Mar 17 2017, Ming Lei wrote:
>
>> Both raid1 and raid10 share common resync
>> block size and page count, so move them into md.h.
>
> I don't think this is necessary.
> These are just "magic" numbers. They don't have any real
> meaning and so don't belong in md.h, or and .h file.
The thing is that RESYNC_PAGES is needed in the following patch 3:
[PATCH v3 03/14] md: prepare for managing resync I/O pages in clean way
so how about moving the macros into raid1-10.h? Cause you suggest
to create that header for holding the introduced helpers in patch3.
Thanks,
Ming
