* [PATCH] memblock: remove stale #else and the code it protects @ 2018-09-27 17:03 Mike Rapoport 2018-10-19 8:17 ` Wei Yang 0 siblings, 1 reply; 4+ messages in thread From: Mike Rapoport @ 2018-09-27 17:03 UTC (permalink / raw) To: Andrew Morton Cc: Alexander Duyck, Michal Hocko, linux-mm, linux-kernel, Mike Rapoport During removal of HAVE_MEMBLOCK definition, the #else clause of the #ifdef CONFIG_HAVE_MEMBLOCK ... #else ... #endif conditional was not removed. Remove it now. Signed-off-by: Mike Rapoport <rppt@linux.vnet.ibm.com> Reported-by: Alexander Duyck <alexander.duyck@gmail.com> Cc: Michal Hocko <mhocko@suse.com> --- include/linux/memblock.h | 5 ----- 1 file changed, 5 deletions(-) diff --git a/include/linux/memblock.h b/include/linux/memblock.h index d3bc270..d4d0e01 100644 --- a/include/linux/memblock.h +++ b/include/linux/memblock.h @@ -597,11 +597,6 @@ static inline void early_memtest(phys_addr_t start, phys_addr_t end) { } #endif -#else -static inline phys_addr_t memblock_alloc(phys_addr_t size, phys_addr_t align) -{ - return 0; -} #endif /* __KERNEL__ */ -- 2.7.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] memblock: remove stale #else and the code it protects 2018-09-27 17:03 [PATCH] memblock: remove stale #else and the code it protects Mike Rapoport @ 2018-10-19 8:17 ` Wei Yang 2018-10-21 7:30 ` Mike Rapoport 0 siblings, 1 reply; 4+ messages in thread From: Wei Yang @ 2018-10-19 8:17 UTC (permalink / raw) To: Mike Rapoport Cc: Andrew Morton, Alexander Duyck, Michal Hocko, linux-mm, linux-kernel Which tree it applies? On Thu, Sep 27, 2018 at 08:03:45PM +0300, Mike Rapoport wrote: >During removal of HAVE_MEMBLOCK definition, the #else clause of the > > #ifdef CONFIG_HAVE_MEMBLOCK > ... > #else > ... > #endif > >conditional was not removed. > >Remove it now. > >Signed-off-by: Mike Rapoport <rppt@linux.vnet.ibm.com> >Reported-by: Alexander Duyck <alexander.duyck@gmail.com> >Cc: Michal Hocko <mhocko@suse.com> >--- > include/linux/memblock.h | 5 ----- > 1 file changed, 5 deletions(-) > >diff --git a/include/linux/memblock.h b/include/linux/memblock.h >index d3bc270..d4d0e01 100644 >--- a/include/linux/memblock.h >+++ b/include/linux/memblock.h >@@ -597,11 +597,6 @@ static inline void early_memtest(phys_addr_t start, phys_addr_t end) > { > } > #endif >-#else >-static inline phys_addr_t memblock_alloc(phys_addr_t size, phys_addr_t align) >-{ >- return 0; >-} > > #endif /* __KERNEL__ */ > >-- >2.7.4 -- Wei Yang Help you, Help me ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] memblock: remove stale #else and the code it protects 2018-10-19 8:17 ` Wei Yang @ 2018-10-21 7:30 ` Mike Rapoport 2018-10-22 1:35 ` Wei Yang 0 siblings, 1 reply; 4+ messages in thread From: Mike Rapoport @ 2018-10-21 7:30 UTC (permalink / raw) To: Wei Yang, Mike Rapoport Cc: Andrew Morton, Alexander Duyck, Michal Hocko, linux-mm, linux-kernel On October 19, 2018 11:17:30 AM GMT+03:00, Wei Yang <richard.weiyang@gmail.com> wrote: >Which tree it applies? To mmotm of the end of September. >On Thu, Sep 27, 2018 at 08:03:45PM +0300, Mike Rapoport wrote: >>During removal of HAVE_MEMBLOCK definition, the #else clause of the >> >> #ifdef CONFIG_HAVE_MEMBLOCK >> ... >> #else >> ... >> #endif >> >>conditional was not removed. >> >>Remove it now. >> >>Signed-off-by: Mike Rapoport <rppt@linux.vnet.ibm.com> >>Reported-by: Alexander Duyck <alexander.duyck@gmail.com> >>Cc: Michal Hocko <mhocko@suse.com> >>--- >> include/linux/memblock.h | 5 ----- >> 1 file changed, 5 deletions(-) >> >>diff --git a/include/linux/memblock.h b/include/linux/memblock.h >>index d3bc270..d4d0e01 100644 >>--- a/include/linux/memblock.h >>+++ b/include/linux/memblock.h >>@@ -597,11 +597,6 @@ static inline void early_memtest(phys_addr_t >start, phys_addr_t end) >> { >> } >> #endif >>-#else >>-static inline phys_addr_t memblock_alloc(phys_addr_t size, >phys_addr_t align) >>-{ >>- return 0; >>-} >> >> #endif /* __KERNEL__ */ >> >>-- >>2.7.4 -- Sent from my Android device with K-9 Mail. Please excuse my brevity. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] memblock: remove stale #else and the code it protects 2018-10-21 7:30 ` Mike Rapoport @ 2018-10-22 1:35 ` Wei Yang 0 siblings, 0 replies; 4+ messages in thread From: Wei Yang @ 2018-10-22 1:35 UTC (permalink / raw) To: Mike Rapoport Cc: Wei Yang, Mike Rapoport, Andrew Morton, Alexander Duyck, Michal Hocko, linux-mm, linux-kernel On Sun, Oct 21, 2018 at 10:30:16AM +0300, Mike Rapoport wrote: > > >On October 19, 2018 11:17:30 AM GMT+03:00, Wei Yang <richard.weiyang@gmail.com> wrote: >>Which tree it applies? > >To mmotm of the end of September. > I may lost some background of this change. The file I am looking at is https://git.kernel.org/pub/scm/linux/kernel/git/mhocko/mm.git/tree/include/linux/memblock.h which has CONFIG_HAVE_MEMBLOCK in line 5. >>On Thu, Sep 27, 2018 at 08:03:45PM +0300, Mike Rapoport wrote: >>>During removal of HAVE_MEMBLOCK definition, the #else clause of the >>> >>> #ifdef CONFIG_HAVE_MEMBLOCK >>> ... >>> #else >>> ... >>> #endif >>> >>>conditional was not removed. >>> >>>Remove it now. >>> >>>Signed-off-by: Mike Rapoport <rppt@linux.vnet.ibm.com> >>>Reported-by: Alexander Duyck <alexander.duyck@gmail.com> >>>Cc: Michal Hocko <mhocko@suse.com> >>>--- >>> include/linux/memblock.h | 5 ----- >>> 1 file changed, 5 deletions(-) >>> >>>diff --git a/include/linux/memblock.h b/include/linux/memblock.h >>>index d3bc270..d4d0e01 100644 >>>--- a/include/linux/memblock.h >>>+++ b/include/linux/memblock.h >>>@@ -597,11 +597,6 @@ static inline void early_memtest(phys_addr_t >>start, phys_addr_t end) >>> { >>> } >>> #endif >>>-#else >>>-static inline phys_addr_t memblock_alloc(phys_addr_t size, >>phys_addr_t align) >>>-{ >>>- return 0; >>>-} >>> And in that file, here is memblock_reserved_memory_within. I guess this is not the version you are trying to fix. BTW, if you could put the commit SHA which removes the CONFIG_HAVE_MEMBLOCK, it would be easier for others to catch up. >>> #endif /* __KERNEL__ */ >>> >>>-- >>>2.7.4 > >-- >Sent from my Android device with K-9 Mail. Please excuse my brevity. -- Wei Yang Help you, Help me ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-10-22 1:35 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-09-27 17:03 [PATCH] memblock: remove stale #else and the code it protects Mike Rapoport 2018-10-19 8:17 ` Wei Yang 2018-10-21 7:30 ` Mike Rapoport 2018-10-22 1:35 ` Wei Yang
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.