From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: lustre-devel@lists.lustre.org
Subject: [lustre-devel] [PATCH 07/10] staging: lustre: Handle nodemask on UMP machines
Date: Fri, 6 Nov 2015 17:12:29 -0800 [thread overview]
Message-ID: <20151107011229.GA14234@kroah.com> (raw)
In-Reply-To: <cafafd1c8db141899d9c9757422451dd@EXCHCS32.ornl.gov>
On Fri, Nov 06, 2015 at 10:18:17PM +0000, Simmons, James A. wrote:
> >All warnings (new ones prefixed by >>):
> >
> > In file included from include/linux/bitops.h:36:0,
> > from drivers/staging/lustre/lustre/libcfs/../../include/linux/libcfs/linux/libcfs.h:44,
> > from drivers/staging/lustre/lustre/libcfs/../../include/linux/libcfs/libcfs.h:40,
> > from drivers/staging/lustre/lustre/libcfs/libcfs_cpu.c:38:
> > drivers/staging/lustre/lustre/libcfs/libcfs_cpu.c: In function 'cfs_cpt_table_alloc':
> >>> arch/m68k/include/asm/bitops.h:64:5: warning: passing argument 2 of 'bset_mem_set_bit' from incompatible pointer type
> > bset_mem_set_bit(nr, vaddr) : \
> > ^
> >>> drivers/staging/lustre/lustre/libcfs/libcfs_cpu.c:61:3: note: in expansion of macro 'set_bit'
> > set_bit(0, &cptab->ctb_nodemask);
>
> Yep and additional patch exist to fix this. Should I just push the fix for this or drop this patch and create
> a new patch that is combo of both fixes.
>
> BTW Greg this new batch of patches are order independent. Sorry for not pointing that out. The rest of the
> patch appear to be okay.
Ok, I'll drop this one, and apply the rest, please resend this after you
have fixed it up.
thanks,
greg k-h
WARNING: multiple messages have this Message-ID (diff)
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: "Simmons, James A." <simmonsja@ornl.gov>
Cc: "'kbuild test robot'" <lkp@intel.com>,
James Simmons <jsimmons@infradead.org>,
"devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>,
James Simmons <uja.ornl@gmail.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Oleg Drokin <oleg.drokin@intel.com>,
"kbuild-all@01.org" <kbuild-all@01.org>,
"lustre-devel@lists.lustre.org" <lustre-devel@lists.lustre.org>
Subject: Re: [lustre-devel] [PATCH 07/10] staging: lustre: Handle nodemask on UMP machines
Date: Fri, 6 Nov 2015 17:12:29 -0800 [thread overview]
Message-ID: <20151107011229.GA14234@kroah.com> (raw)
In-Reply-To: <cafafd1c8db141899d9c9757422451dd@EXCHCS32.ornl.gov>
On Fri, Nov 06, 2015 at 10:18:17PM +0000, Simmons, James A. wrote:
> >All warnings (new ones prefixed by >>):
> >
> > In file included from include/linux/bitops.h:36:0,
> > from drivers/staging/lustre/lustre/libcfs/../../include/linux/libcfs/linux/libcfs.h:44,
> > from drivers/staging/lustre/lustre/libcfs/../../include/linux/libcfs/libcfs.h:40,
> > from drivers/staging/lustre/lustre/libcfs/libcfs_cpu.c:38:
> > drivers/staging/lustre/lustre/libcfs/libcfs_cpu.c: In function 'cfs_cpt_table_alloc':
> >>> arch/m68k/include/asm/bitops.h:64:5: warning: passing argument 2 of 'bset_mem_set_bit' from incompatible pointer type
> > bset_mem_set_bit(nr, vaddr) : \
> > ^
> >>> drivers/staging/lustre/lustre/libcfs/libcfs_cpu.c:61:3: note: in expansion of macro 'set_bit'
> > set_bit(0, &cptab->ctb_nodemask);
>
> Yep and additional patch exist to fix this. Should I just push the fix for this or drop this patch and create
> a new patch that is combo of both fixes.
>
> BTW Greg this new batch of patches are order independent. Sorry for not pointing that out. The rest of the
> patch appear to be okay.
Ok, I'll drop this one, and apply the rest, please resend this after you
have fixed it up.
thanks,
greg k-h
next prev parent reply other threads:[~2015-11-07 1:12 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-04 18:39 [lustre-devel] [PATCH 01/10] staging: lustre: wrong parameter to cfs_hash_keycpy James Simmons
2015-11-04 18:39 ` James Simmons
2015-11-04 18:39 ` [lustre-devel] [PATCH 02/10] staging: lustre: remove unnecessary EXPORT_SYMBOL in libcfs James Simmons
2015-11-04 18:39 ` James Simmons
2015-11-04 18:39 ` [lustre-devel] [PATCH 03/10] staging: lustre: remove libcfs_debug_set_level prototype from libcfs_private.h James Simmons
2015-11-04 18:39 ` James Simmons
2015-11-04 18:40 ` [lustre-devel] [PATCH 04/10] staging: lustre: fix buffer overflow of string buffer James Simmons
2015-11-04 18:40 ` James Simmons
2015-11-04 18:40 ` [lustre-devel] [PATCH 05/10] staging: lustre: Fix possible NULL pointer dereference in lprocfs_status.c James Simmons
2015-11-04 18:40 ` James Simmons
2015-11-04 18:40 ` [lustre-devel] [PATCH 06/10] staging: lustre: Update module author to OpenSFS James Simmons
2015-11-04 18:40 ` James Simmons
2015-11-04 18:40 ` [lustre-devel] [PATCH 07/10] staging: lustre: Handle nodemask on UMP machines James Simmons
2015-11-04 18:40 ` James Simmons
2015-11-04 22:06 ` [lustre-devel] " kbuild test robot
2015-11-04 22:06 ` kbuild test robot
2015-11-05 0:46 ` [lustre-devel] " kbuild test robot
2015-11-05 0:46 ` kbuild test robot
2015-11-06 22:18 ` [lustre-devel] " Simmons, James A.
2015-11-06 22:18 ` Simmons, James A.
2015-11-07 1:12 ` Greg Kroah-Hartman [this message]
2015-11-07 1:12 ` Greg Kroah-Hartman
2015-11-07 20:53 ` Dan Carpenter
2015-11-07 20:53 ` Dan Carpenter
2015-11-04 18:40 ` [lustre-devel] [PATCH 08/10] staging: lustre: race condition for check/use cfs_fail_val James Simmons
2015-11-04 18:40 ` James Simmons
2015-11-04 18:40 ` [lustre-devel] [PATCH 09/10] staging: lustre: remove page_collection::pc_lock in libcfs James Simmons
2015-11-04 18:40 ` James Simmons
2015-11-04 18:40 ` [lustre-devel] [PATCH 10/10] staging: lustre: fix 'error handling' issues for libcfs workitem.c James Simmons
2015-11-04 18:40 ` James Simmons
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151107011229.GA14234@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=lustre-devel@lists.lustre.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.