Hi. On Tuesday 27 June 2006 15:36, Jens Axboe wrote: > On Tue, Jun 27 2006, Nigel Cunningham wrote: > > Hi. > > > > On Tuesday 27 June 2006 07:20, Rafael J. Wysocki wrote: > > > On Monday 26 June 2006 18:54, Nigel Cunningham wrote: > > > > Add Suspend2 extent support. Extents are used for storing the lists > > > > of blocks to which the image will be written, and are stored in the > > > > image header for use at resume time. > > > > > > Could you please put all of the changes in kernel/power/extents.c into > > > one patch? It's quite difficult to review them now, at least for me. > > > > I spent a long time splitting them up because I was asked in previous > > iterations to break them into manageable chunks. How about if I were to > > email you the individual files off line, so as to not send the same > > amount again? > > Managable chunks means logical changes go together, one function per > diff is really extreme and unreviewable. Support for extents is one > logical change, so it's one patch. Unless of course you have to do some > preparatory patches, then you'd do those separately. > > I must admit I thought you were kidding when I read through this extents > patch series, having a single patch just adding includes! Sorry for fluffing it up. I'm pretty inexperienced, but I'm trying to follow CodingStyle and all the other advice. If I'd known I'd misunderstood what was wanted, I probably could have submitted this months ago. Oh well. Live and learn. What would you have me do at this point? Regards, Nigel -- See http://www.suspend2.net for Howtos, FAQs, mailing lists, wiki and bugzilla info.
Attachment:
pgpgMxqiZ2BNb.pgp
Description: PGP signature
- Follow-Ups:
- Re: [Suspend2][ 0/9] Extents support.
- From: Greg KH <[email protected]>
- Re: [Suspend2][ 0/9] Extents support.
- From: Jens Axboe <[email protected]>
- Re: [Suspend2][ 0/9] Extents support.
- References:
- [Suspend2][ 0/9] Extents support.
- From: Nigel Cunningham <[email protected]>
- Re: [Suspend2][ 0/9] Extents support.
- From: Nigel Cunningham <[email protected]>
- Re: [Suspend2][ 0/9] Extents support.
- From: Jens Axboe <[email protected]>
- [Suspend2][ 0/9] Extents support.
- Prev by Date: Re: [git pull] Input update for 2.6.17
- Next by Date: Re: [PATCH 1/1] Fix boot on efi 32 bit Machines [try #4]
- Previous by thread: Re: [Suspend2][ 0/9] Extents support.
- Next by thread: Re: [Suspend2][ 0/9] Extents support.
- Index(es):
 |