Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-07-17 Thread Real Name
On Mon, Jun 16, 2014 at 06:30:36PM +0800, Real Name wrote: > On Mon, Jun 16, 2014 at 11:15:23AM +0200, Geert Uytterhoeven wrote: > > On Mon, Jun 16, 2014 at 10:28 AM, Real Name wrote: > > >> > >> Can you please include in the changelog the commit sha1 which made > > >> > >> the old init_maps()

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-07-17 Thread Real Name
On Mon, Jun 16, 2014 at 06:30:36PM +0800, Real Name wrote: On Mon, Jun 16, 2014 at 11:15:23AM +0200, Geert Uytterhoeven wrote: On Mon, Jun 16, 2014 at 10:28 AM, Real Name enjoymind...@gmail.com wrote: Can you please include in the changelog the commit sha1 which made the old

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Real Name
On Mon, Jun 16, 2014 at 11:15:23AM +0200, Geert Uytterhoeven wrote: > On Mon, Jun 16, 2014 at 10:28 AM, Real Name wrote: > >> > >> Can you please include in the changelog the commit sha1 which made > >> > >> the old init_maps() obsolete? hi, The commit sha1 had been included as required.

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Geert Uytterhoeven
On Mon, Jun 16, 2014 at 10:28 AM, Real Name wrote: >> > >> Can you please include in the changelog the commit sha1 which made the >> > >> old init_maps() obsolete? > > I think we need find out which commit deleted the line "mem_map = map;" in > init_maps function.

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Real Name
On Mon, Jun 16, 2014 at 04:12:38PM +0800, Real Name wrote: > On Mon, Jun 16, 2014 at 08:50:33AM +0200, Richard Weinberger wrote: > > > > > > Am 16.06.2014 05:28, schrieb Real Name: > > > On Sat, Jun 14, 2014 at 11:44:04AM +0200, Richard Weinberger wrote: > > >> Hi! > > >> > > >> Am 03.06.2014

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Real Name
On Mon, Jun 16, 2014 at 08:50:33AM +0200, Richard Weinberger wrote: > > > Am 16.06.2014 05:28, schrieb Real Name: > > On Sat, Jun 14, 2014 at 11:44:04AM +0200, Richard Weinberger wrote: > >> Hi! > >> > >> Am 03.06.2014 07:30, schrieb Real Name: > >>> From: Honggang Li > >>> > >>> The patch

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Richard Weinberger
Am 16.06.2014 05:28, schrieb Real Name: > On Sat, Jun 14, 2014 at 11:44:04AM +0200, Richard Weinberger wrote: >> Hi! >> >> Am 03.06.2014 07:30, schrieb Real Name: >>> From: Honggang Li >>> >>> The patch based on linux-next-2014-06-02. >>> >>> The old init_maps function does two things: >>> 1)

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Richard Weinberger
Am 16.06.2014 05:28, schrieb Real Name: On Sat, Jun 14, 2014 at 11:44:04AM +0200, Richard Weinberger wrote: Hi! Am 03.06.2014 07:30, schrieb Real Name: From: Honggang Li enjoymind...@gmail.com The patch based on linux-next-2014-06-02. The old init_maps function does two things: 1)

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Real Name
On Mon, Jun 16, 2014 at 08:50:33AM +0200, Richard Weinberger wrote: Am 16.06.2014 05:28, schrieb Real Name: On Sat, Jun 14, 2014 at 11:44:04AM +0200, Richard Weinberger wrote: Hi! Am 03.06.2014 07:30, schrieb Real Name: From: Honggang Li enjoymind...@gmail.com The patch based on

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Real Name
On Mon, Jun 16, 2014 at 04:12:38PM +0800, Real Name wrote: On Mon, Jun 16, 2014 at 08:50:33AM +0200, Richard Weinberger wrote: Am 16.06.2014 05:28, schrieb Real Name: On Sat, Jun 14, 2014 at 11:44:04AM +0200, Richard Weinberger wrote: Hi! Am 03.06.2014 07:30, schrieb Real

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Geert Uytterhoeven
On Mon, Jun 16, 2014 at 10:28 AM, Real Name enjoymind...@gmail.com wrote: Can you please include in the changelog the commit sha1 which made the old init_maps() obsolete? I think we need find out which commit deleted the line mem_map = map; in init_maps function.

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-16 Thread Real Name
On Mon, Jun 16, 2014 at 11:15:23AM +0200, Geert Uytterhoeven wrote: On Mon, Jun 16, 2014 at 10:28 AM, Real Name enjoymind...@gmail.com wrote: Can you please include in the changelog the commit sha1 which made the old init_maps() obsolete? hi, The commit sha1 had been included as

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-15 Thread Real Name
On Sat, Jun 14, 2014 at 11:44:04AM +0200, Richard Weinberger wrote: > Hi! > > Am 03.06.2014 07:30, schrieb Real Name: > > From: Honggang Li > > > > The patch based on linux-next-2014-06-02. > > > > The old init_maps function does two things: > > 1) allocates and initializes one struct page

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-15 Thread Real Name
On Sat, Jun 14, 2014 at 11:44:04AM +0200, Richard Weinberger wrote: Hi! Am 03.06.2014 07:30, schrieb Real Name: From: Honggang Li enjoymind...@gmail.com The patch based on linux-next-2014-06-02. The old init_maps function does two things: 1) allocates and initializes one struct

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-14 Thread Richard Weinberger
Hi! Am 03.06.2014 07:30, schrieb Real Name: > From: Honggang Li > > The patch based on linux-next-2014-06-02. > > The old init_maps function does two things: > 1) allocates and initializes one struct page array for bootmem > 2) count the number of total pages > > After removed the source code

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-14 Thread Richard Weinberger
Hi! Am 03.06.2014 07:30, schrieb Real Name: From: Honggang Li enjoymind...@gmail.com The patch based on linux-next-2014-06-02. The old init_maps function does two things: 1) allocates and initializes one struct page array for bootmem 2) count the number of total pages After removed

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-08 Thread Real Name
Hi, Richard Any comment about this patch? thanks On Tue, Jun 03, 2014 at 01:30:44PM +0800, Real Name wrote: > From: Honggang Li > > The patch based on linux-next-2014-06-02. > > The old init_maps function does two things: > 1) allocates and initializes one struct page array for bootmem > 2)

Re: [PATCH v2] delete unnecessary bootmem struct page array

2014-06-08 Thread Real Name
Hi, Richard Any comment about this patch? thanks On Tue, Jun 03, 2014 at 01:30:44PM +0800, Real Name wrote: From: Honggang Li enjoymind...@gmail.com The patch based on linux-next-2014-06-02. The old init_maps function does two things: 1) allocates and initializes one struct page array