From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Martin K. Petersen" Subject: Re: ATA 4 KiB sector issues. Date: Wed, 10 Mar 2010 00:06:25 -0500 Message-ID: References: <4B947393.2050002@kernel.org> <170fa0d21003081134g491034e5v4aad4d43853e48ec@mail.gmail.com> <4B95F071.3070400@msgid.tls.msk.ru> <20100309100153.GD18077@nb.net.home> <4B961FE1.5040105@msgid.tls.msk.ru> <20100309121858.GA2020@nb.net.home> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: In-Reply-To: <20100309121858.GA2020@nb.net.home> (Karel Zak's message of "Tue, 9 Mar 2010 13:18:59 +0100") Sender: linux-kernel-owner@vger.kernel.org To: Karel Zak Cc: Michael Tokarev , Mike Snitzer , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-ide@vger.kernel.org >>>>> "Karel" == Karel Zak writes: [Cleaned up the CC: list from hell] Karel> # cat /sys/block/md8/queue/{minimum,optimal}_io_size Karel> 65536 65536 This one had me puzzled. We set min_io and opt_io correctly in raid5.c depending on number of non-parity disks. And yet it turns into something nonsensical after. Turns out we overrun unsigned int calculating the lowest common multiple in the stacking function. That's why we ended up with the wrong value. I never noticed this because my userland topology regression test tool uses unsigned long. I'll get a patch off to Jens right away. -- Martin K. Petersen Oracle Linux Engineering