Messages in this thread | | | Date | Thu, 14 Jul 2005 22:27:59 -0700 | From | Badari Pulavarty <> | Subject | Re: [rfc patch 2/2] direct-io: remove address alignment check |
| |
Daniel McNeil wrote:
> On Thu, 2005-07-14 at 16:16, Badari Pulavarty wrote: > >>How does your patch ensures that we meet the driver alignment >>restrictions ? Like you said, you need atleast "even" byte alignment >>for IDE etc.. >> >>And also, are there any restrictions on how much the "minimum" IO >>size has to be ? I mean, can I read "1" byte ? I guess you are >>not relaxing it (yet).. >> > > > This patch does not change the i/o size requirements -- they > must be a multiple of device block size (usually 512). > > It only relaxes the address alignment restriction. I do not > know what the driver alignment restrictions are. Without the > 1st patch, it was impossible to relax the address space > check and have direct-io generate the correct i/o's to submit. > > This 2nd patch, is just for testing and generating feedback > to find out what the address alignment issues are. Then > we can decide how to proceed. > > Did you look over the 1st patch? Comments?
Yes. I did look at the first patch and my questions were basically towards the first patch. I don't see any enforcement of alignment with your patch at all. So, we let the driver fail if it can't handle it ?
BTW, I don't think the first patch is really doing the right thing. You got little carried away while cleaning up. You are trying to relax "user buffer" alignment only. If your "offset" is in the middle of a filesystem block (say 4k), you still need to zero out the first portion to be able to write into the middle. That "evil" code is still needed. :(
Thanks, Badari
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |