public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] staging/lustre/llite: Fix obd name after c&p error
@ 2015-03-03  7:39 green
  2015-03-03  8:28 ` Dan Carpenter
  0 siblings, 1 reply; 3+ messages in thread
From: green @ 2015-03-03  7:39 UTC (permalink / raw)
  To: Greg Kroah-Hartman, devel, Andreas Dilger
  Cc: Linux Kernel Mailing List, Oleg Drokin, Navya Sri Nizamkari,
	Julia Lawall

From: Oleg Drokin <green@linuxhacker.ru>

commit 95745e9b1de2 ("staging: lustre: Use kasprintf.") introduced
a copy and paste error causing two different obd types to be assigned
same content causing lustre to fail on mount with a warning from procfs
followed by a bizzare error about OST not having enough MDS
capabilities.

This patch unbreaks Lustre client again.

Signed-off-by: Oleg Drokin <green@linuxhacker.ru>
CC: Navya Sri Nizamkari <navyasri.tech@gmail.com>
CC: Julia Lawall <julia.lawall@lip6.fr>
---
This really came from nowhere and I do not see this patch in any archives
of mailinglists I am subscribed to.
In fact Google cannot find it either.

 drivers/staging/lustre/lustre/llite/llite_lib.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/llite/llite_lib.c b/drivers/staging/lustre/lustre/llite/llite_lib.c
index ee4bd89..bf1ec27 100644
--- a/drivers/staging/lustre/lustre/llite/llite_lib.c
+++ b/drivers/staging/lustre/lustre/llite/llite_lib.c
@@ -983,7 +983,7 @@ int ll_fill_super(struct super_block *sb, struct vfsmount *mnt)
 		goto out_free;
 	}
 
-	md = kasprintf(GFP_NOFS, "%s-%p", lprof->lp_dt, cfg->cfg_instance);
+	md = kasprintf(GFP_NOFS, "%s-%p", lprof->lp_md, cfg->cfg_instance);
 	if (!md) {
 		err = -ENOMEM;
 		goto out_free;
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] staging/lustre/llite: Fix obd name after c&p error
  2015-03-03  7:39 [PATCH] staging/lustre/llite: Fix obd name after c&p error green
@ 2015-03-03  8:28 ` Dan Carpenter
  2015-03-03 12:53   ` Greg Kroah-Hartman
  0 siblings, 1 reply; 3+ messages in thread
From: Dan Carpenter @ 2015-03-03  8:28 UTC (permalink / raw)
  To: green
  Cc: Greg Kroah-Hartman, devel, Andreas Dilger, Julia Lawall,
	Navya Sri Nizamkari, Linux Kernel Mailing List

On Tue, Mar 03, 2015 at 02:39:36AM -0500, green@linuxhacker.ru wrote:

> This really came from nowhere and I do not see this patch in any archives
> of mailinglists I am subscribed to.
> In fact Google cannot find it either.

I was going to say something similar.  It's not in linux-next either.

regards,
dan carpenter



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] staging/lustre/llite: Fix obd name after c&p error
  2015-03-03  8:28 ` Dan Carpenter
@ 2015-03-03 12:53   ` Greg Kroah-Hartman
  0 siblings, 0 replies; 3+ messages in thread
From: Greg Kroah-Hartman @ 2015-03-03 12:53 UTC (permalink / raw)
  To: Dan Carpenter
  Cc: green, devel, Andreas Dilger, Linux Kernel Mailing List,
	Julia Lawall, Navya Sri Nizamkari

On Tue, Mar 03, 2015 at 11:28:25AM +0300, Dan Carpenter wrote:
> On Tue, Mar 03, 2015 at 02:39:36AM -0500, green@linuxhacker.ru wrote:
> 
> > This really came from nowhere and I do not see this patch in any archives
> > of mailinglists I am subscribed to.
> > In fact Google cannot find it either.
> 
> I was going to say something similar.  It's not in linux-next either.

It was sent to the outreachy kernel mailing list, as part of the intern
application process.

Sorry for missing this cut and paste error, my fault in reviewing
things.

It's not in linux-next as it's in my "staging-testing" branch, which I
like to keep around for a day or so before mergeing to a non-rebasable
branch that shows up in linux-next.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-03-03 12:53 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-03  7:39 [PATCH] staging/lustre/llite: Fix obd name after c&p error green
2015-03-03  8:28 ` Dan Carpenter
2015-03-03 12:53   ` Greg Kroah-Hartman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox