| To: | Lachlan McIlroy <lachlan@xxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] Fix off by one error in page_region_mask() |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Sun, 15 Feb 2009 14:16:14 -0500 |
| Cc: | Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs-oss <xfs@xxxxxxxxxxx> |
| In-reply-to: | <20090122222539.GA9643@xxxxxxxxxxxxx> |
| References: | <49378B60.1060603@xxxxxxx> <20081222085354.GC24795@xxxxxxxxxxxxx> <49502F91.4050406@xxxxxxx> <49742371.2000806@xxxxxxx> <20090122222539.GA9643@xxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.18 (2008-05-17) |
On Thu, Jan 22, 2009 at 05:25:39PM -0500, Christoph Hellwig wrote: > On Mon, Jan 19, 2009 at 05:53:37PM +1100, Lachlan McIlroy wrote: > > Lachlan McIlroy wrote: > > > Christoph Hellwig wrote: > > >> Did you guys come to a conclusion in your discussions? > > >> > > > No conclusion, well nothing that resulted in an ack. > > > > Was anyone okay with this fix? > > I think it's okay to go. It fixes the one off so it's go. > > The code is still a very hard to understand mess, but that's not what > the patch was supposed to fix.. Any reason this isn't in yet? |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: reproducible xfs/vmap oops, Christoph Hellwig |
|---|---|
| Next by Date: | Re: Oops during umount of r/o file system, Christoph Hellwig |
| Previous by Thread: | xfs: prevent kernel crash due to corrupted inode log format, Christoph Hellwig |
| Next by Thread: | Re: [PATCH] Fix off by one error in page_region_mask(), Felix Blyakher |
| Indexes: | [Date] [Thread] [Top] [All Lists] |