Messages in this thread | | | From | KY Srinivasan <> | Subject | RE: [PATCH 1/1] Drivers: base: memory: Export symbols for onlining memory blocks | Date | Tue, 23 Jul 2013 16:54:50 +0000 |
| |
> -----Original Message----- > From: Greg KH [mailto:gregkh@linuxfoundation.org] > Sent: Tuesday, July 23, 2013 12:02 PM > To: KY Srinivasan > Cc: linux-kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de; > apw@canonical.com; andi@firstfloor.org; akpm@linux-foundation.org; linux- > mm@kvack.org; kamezawa.hiroyuki@gmail.com; mhocko@suse.cz; > hannes@cmpxchg.org; yinghan@google.com; jasowang@redhat.com; > kay@vrfy.org > Subject: Re: [PATCH 1/1] Drivers: base: memory: Export symbols for onlining > memory blocks > > On Fri, Jul 19, 2013 at 12:23:05PM -0700, K. Y. Srinivasan wrote: > > The current machinery for hot-adding memory requires having udev > > rules to bring the memory segments online. Export the necessary functionality > > to to bring the memory segment online without involving user space code. > > > > Signed-off-by: K. Y. Srinivasan <kys@microsoft.com> > > --- > > drivers/base/memory.c | 5 ++++- > > include/linux/memory.h | 4 ++++ > > 2 files changed, 8 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/base/memory.c b/drivers/base/memory.c > > index 2b7813e..a8204ac 100644 > > --- a/drivers/base/memory.c > > +++ b/drivers/base/memory.c > > @@ -328,7 +328,7 @@ static int > __memory_block_change_state_uevent(struct memory_block *mem, > > return ret; > > } > > > > -static int memory_block_change_state(struct memory_block *mem, > > +int memory_block_change_state(struct memory_block *mem, > > unsigned long to_state, unsigned long from_state_req, > > int online_type) > > { > > @@ -341,6 +341,8 @@ static int memory_block_change_state(struct > memory_block *mem, > > > > return ret; > > } > > +EXPORT_SYMBOL(memory_block_change_state); > > EXPORT_SYMBOL_GPL() for all of these please.
Will do. > > And as others have pointed out, I can't export symbols without a user of > those symbols going into the tree at the same time. So I'll drop this > patch for now and wait for your consumer of these symbols to be > submitted.
I will submit the consumer as well.
Thanks,
K. Y
> greg k-h > >
| |