On Fri, Apr 29, 2011 at 07:05:16AM +0900, Simon Horman wrote: > On Thu, Apr 28, 2011 at 09:15:39AM -0400, Vivek Goyal wrote: > > On Thu, Apr 28, 2011 at 03:57:48PM +0800, Cong Wang wrote: > > > ? 2011?04?27? 23:01, Vivek Goyal ??: > > > >Amerigo, > > > > > > > >Now who will do following initialization for ppc? > > > > > > > >info->backup_src_start = BACKUP_SRC_START; > > > >info->backup_sz = BACKUP_SRC_END - BACKUP_SRC_START + 1; > > > > > > > >I think I have concern for mips too. > > > > > > Right, we need to initialize them in load_crashdump_segments(). > > > > > > How about the attached patch? It is untested. > > > > > > > > Signed-off-by: WANG Cong <amwang at redhat.com> > > > > Ok, this one looks better. > > Vivek, can I take that as an Acked-by? Simon, yes please. Acked-by: Vivek Goyal <vgoyal at redhat.com> Vivek