* [PATCH 01/12] ARM: mach-imx: iomux-imx31: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-20 6:50 ` Uwe Kleine-König
2015-05-21 1:09 ` Shawn Guo
2015-05-20 1:37 ` [PATCH 02/12] dmaengine: rcar-dmac: " Joe Perches
` (11 subsequent siblings)
12 siblings, 2 replies; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel, Shawn Guo, Sascha Hauer; +Cc: Russell King, linux-arm-kernel
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
arch/arm/mach-imx/iomux-imx31.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-imx/iomux-imx31.c b/arch/arm/mach-imx/iomux-imx31.c
index d6a3075..6dd22ca 100644
--- a/arch/arm/mach-imx/iomux-imx31.c
+++ b/arch/arm/mach-imx/iomux-imx31.c
@@ -40,7 +40,7 @@ static DEFINE_SPINLOCK(gpio_mux_lock);
#define IOMUX_REG_MASK (IOMUX_PADNUM_MASK & ~0x3)
-static unsigned long mxc_pin_alloc_map[NB_PORTS * 32 / BITS_PER_LONG];
+static DECLARE_BITMAP(mxc_pin_alloc_map, NB_PORTS * 32);
/*
* set the mode for a IOMUX pin.
*/
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH 01/12] ARM: mach-imx: iomux-imx31: Use DECLARE_BITMAP
2015-05-20 1:37 ` [PATCH 01/12] ARM: mach-imx: iomux-imx31: " Joe Perches
@ 2015-05-20 6:50 ` Uwe Kleine-König
2015-05-21 1:09 ` Shawn Guo
1 sibling, 0 replies; 26+ messages in thread
From: Uwe Kleine-König @ 2015-05-20 6:50 UTC (permalink / raw)
To: Joe Perches
Cc: linux-kernel, Shawn Guo, Sascha Hauer, Russell King, linux-arm-kernel
Hello,
On Tue, May 19, 2015 at 06:37:49PM -0700, Joe Perches wrote:
> Use the generic mechanism to declare a bitmap instead of unsigned long.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> arch/arm/mach-imx/iomux-imx31.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-imx/iomux-imx31.c b/arch/arm/mach-imx/iomux-imx31.c
> index d6a3075..6dd22ca 100644
> --- a/arch/arm/mach-imx/iomux-imx31.c
> +++ b/arch/arm/mach-imx/iomux-imx31.c
> @@ -40,7 +40,7 @@ static DEFINE_SPINLOCK(gpio_mux_lock);
>
> #define IOMUX_REG_MASK (IOMUX_PADNUM_MASK & ~0x3)
>
> -static unsigned long mxc_pin_alloc_map[NB_PORTS * 32 / BITS_PER_LONG];
> +static DECLARE_BITMAP(mxc_pin_alloc_map, NB_PORTS * 32);
> /*
> * set the mode for a IOMUX pin.
> */
Compile tested and
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH 01/12] ARM: mach-imx: iomux-imx31: Use DECLARE_BITMAP
2015-05-20 1:37 ` [PATCH 01/12] ARM: mach-imx: iomux-imx31: " Joe Perches
2015-05-20 6:50 ` Uwe Kleine-König
@ 2015-05-21 1:09 ` Shawn Guo
1 sibling, 0 replies; 26+ messages in thread
From: Shawn Guo @ 2015-05-21 1:09 UTC (permalink / raw)
To: Joe Perches; +Cc: linux-kernel, Sascha Hauer, Russell King, linux-arm-kernel
On Tue, May 19, 2015 at 06:37:49PM -0700, Joe Perches wrote:
> Use the generic mechanism to declare a bitmap instead of unsigned long.
>
> Signed-off-by: Joe Perches <joe@perches.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 02/12] dmaengine: rcar-dmac: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
2015-05-20 1:37 ` [PATCH 01/12] ARM: mach-imx: iomux-imx31: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-25 17:12 ` Vinod Koul
2015-05-20 1:37 ` [PATCH 03/12] drm/amdkfd: " Joe Perches
` (10 subsequent siblings)
12 siblings, 1 reply; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel; +Cc: Dan Williams, Vinod Koul, dmaengine
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/dma/sh/rcar-dmac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/sh/rcar-dmac.c b/drivers/dma/sh/rcar-dmac.c
index a18d16c..d5423a6 100644
--- a/drivers/dma/sh/rcar-dmac.c
+++ b/drivers/dma/sh/rcar-dmac.c
@@ -183,7 +183,7 @@ struct rcar_dmac {
unsigned int n_channels;
struct rcar_dmac_chan *channels;
- unsigned long modules[256 / BITS_PER_LONG];
+ DECLARE_BITMAP(modules, 256);
};
#define to_rcar_dmac(d) container_of(d, struct rcar_dmac, engine)
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 03/12] drm/amdkfd: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
2015-05-20 1:37 ` [PATCH 01/12] ARM: mach-imx: iomux-imx31: " Joe Perches
2015-05-20 1:37 ` [PATCH 02/12] dmaengine: rcar-dmac: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-20 7:32 ` Oded Gabbay
2015-05-20 1:37 ` [PATCH 04/12] drm/radeon: " Joe Perches
` (9 subsequent siblings)
12 siblings, 1 reply; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel, Oded Gabbay; +Cc: David Airlie, dri-devel
Use the generic mechanism to declare a bitmap instead of unsigned long.
It seems that "struct kfd_process.allocated_queue_bitmap" is unused.
Maybe it could be deleted instead.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
index f21fcce..aba3e5d 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
@@ -150,8 +150,7 @@ struct kfd_dev {
const struct kfd2kgd_calls *kfd2kgd;
struct mutex doorbell_mutex;
- unsigned long doorbell_available_index[DIV_ROUND_UP(
- KFD_MAX_NUM_OF_QUEUES_PER_PROCESS, BITS_PER_LONG)];
+ DECLARE_BITMAP(doorbell_available_index, KFD_MAX_NUM_OF_QUEUES_PER_PROCESS);
void *gtt_mem;
uint64_t gtt_start_gpu_addr;
@@ -473,7 +472,7 @@ struct kfd_process {
/* Size is queue_array_size, up to MAX_PROCESS_QUEUES. */
struct kfd_queue **queues;
- unsigned long allocated_queue_bitmap[DIV_ROUND_UP(KFD_MAX_NUM_OF_QUEUES_PER_PROCESS, BITS_PER_LONG)];
+ DECLARE_BITMAP(allocated_queue_bitmap, KFD_MAX_NUM_OF_QUEUES_PER_PROCESS);
/*Is the user space process 32 bit?*/
bool is_32bit_user_mode;
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH 03/12] drm/amdkfd: Use DECLARE_BITMAP
2015-05-20 1:37 ` [PATCH 03/12] drm/amdkfd: " Joe Perches
@ 2015-05-20 7:32 ` Oded Gabbay
0 siblings, 0 replies; 26+ messages in thread
From: Oded Gabbay @ 2015-05-20 7:32 UTC (permalink / raw)
To: Joe Perches
Cc: Linux-Kernel@Vger. Kernel. Org, Oded Gabbay, David Airlie,
Maling list - DRI developers
On Wed, May 20, 2015 at 4:37 AM, Joe Perches <joe@perches.com> wrote:
> Use the generic mechanism to declare a bitmap instead of unsigned long.
>
> It seems that "struct kfd_process.allocated_queue_bitmap" is unused.
> Maybe it could be deleted instead.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> index f21fcce..aba3e5d 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> @@ -150,8 +150,7 @@ struct kfd_dev {
>
> const struct kfd2kgd_calls *kfd2kgd;
> struct mutex doorbell_mutex;
> - unsigned long doorbell_available_index[DIV_ROUND_UP(
> - KFD_MAX_NUM_OF_QUEUES_PER_PROCESS, BITS_PER_LONG)];
> + DECLARE_BITMAP(doorbell_available_index, KFD_MAX_NUM_OF_QUEUES_PER_PROCESS);
>
> void *gtt_mem;
> uint64_t gtt_start_gpu_addr;
> @@ -473,7 +472,7 @@ struct kfd_process {
> /* Size is queue_array_size, up to MAX_PROCESS_QUEUES. */
> struct kfd_queue **queues;
>
> - unsigned long allocated_queue_bitmap[DIV_ROUND_UP(KFD_MAX_NUM_OF_QUEUES_PER_PROCESS, BITS_PER_LONG)];
> + DECLARE_BITMAP(allocated_queue_bitmap, KFD_MAX_NUM_OF_QUEUES_PER_PROCESS);
>
> /*Is the user space process 32 bit?*/
> bool is_32bit_user_mode;
> --
> 2.1.2
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
Thanks!
I indeed deleted the allocated_queue_bitmap and squashed it into your
patch. It was a residue from a very early version.
Applied to my -next branch.
Oded
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 04/12] drm/radeon: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
` (2 preceding siblings ...)
2015-05-20 1:37 ` [PATCH 03/12] drm/amdkfd: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-20 16:45 ` Alex Deucher
2015-05-20 1:37 ` [PATCH 05/12] IB/ehca: " Joe Perches
` (8 subsequent siblings)
12 siblings, 1 reply; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel, Alex Deucher, Christian König; +Cc: David Airlie, dri-devel
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/gpu/drm/radeon/radeon.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/radeon/radeon.h b/drivers/gpu/drm/radeon/radeon.h
index 46eb0fa..d556733 100644
--- a/drivers/gpu/drm/radeon/radeon.h
+++ b/drivers/gpu/drm/radeon/radeon.h
@@ -719,7 +719,7 @@ struct radeon_doorbell {
resource_size_t size;
u32 __iomem *ptr;
u32 num_doorbells; /* Number of doorbells actually reserved for radeon. */
- unsigned long used[DIV_ROUND_UP(RADEON_MAX_DOORBELLS, BITS_PER_LONG)];
+ DECLARE_BITMAP(used, RADEON_MAX_DOORBELLS);
};
int radeon_doorbell_get(struct radeon_device *rdev, u32 *page);
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH 04/12] drm/radeon: Use DECLARE_BITMAP
2015-05-20 1:37 ` [PATCH 04/12] drm/radeon: " Joe Perches
@ 2015-05-20 16:45 ` Alex Deucher
0 siblings, 0 replies; 26+ messages in thread
From: Alex Deucher @ 2015-05-20 16:45 UTC (permalink / raw)
To: Joe Perches
Cc: LKML, Alex Deucher, Christian König, Maling list - DRI developers
On Tue, May 19, 2015 at 9:37 PM, Joe Perches <joe@perches.com> wrote:
> Use the generic mechanism to declare a bitmap instead of unsigned long.
>
> Signed-off-by: Joe Perches <joe@perches.com>
Applied to my -next tree. thanks!
Alex
> ---
> drivers/gpu/drm/radeon/radeon.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon.h b/drivers/gpu/drm/radeon/radeon.h
> index 46eb0fa..d556733 100644
> --- a/drivers/gpu/drm/radeon/radeon.h
> +++ b/drivers/gpu/drm/radeon/radeon.h
> @@ -719,7 +719,7 @@ struct radeon_doorbell {
> resource_size_t size;
> u32 __iomem *ptr;
> u32 num_doorbells; /* Number of doorbells actually reserved for radeon. */
> - unsigned long used[DIV_ROUND_UP(RADEON_MAX_DOORBELLS, BITS_PER_LONG)];
> + DECLARE_BITMAP(used, RADEON_MAX_DOORBELLS);
> };
>
> int radeon_doorbell_get(struct radeon_device *rdev, u32 *page);
> --
> 2.1.2
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 05/12] IB/ehca: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
` (3 preceding siblings ...)
2015-05-20 1:37 ` [PATCH 04/12] drm/radeon: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-20 1:37 ` [PATCH 06/12] bcache: " Joe Perches
` (7 subsequent siblings)
12 siblings, 0 replies; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel, Hoang-Nam Nguyen, Christoph Raisch
Cc: Doug Ledford, Sean Hefty, Hal Rosenstock, linux-rdma
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/infiniband/hw/ehca/ipz_pt_fn.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/infiniband/hw/ehca/ipz_pt_fn.h b/drivers/infiniband/hw/ehca/ipz_pt_fn.h
index a801274..19bc344 100644
--- a/drivers/infiniband/hw/ehca/ipz_pt_fn.h
+++ b/drivers/infiniband/hw/ehca/ipz_pt_fn.h
@@ -65,7 +65,7 @@ struct ipz_page {
struct ipz_small_queue_page {
unsigned long page;
- unsigned long bitmap[IPZ_SPAGE_PER_KPAGE / BITS_PER_LONG];
+ DECLARE_BITMAP(bitmap, IPZ_SPAGE_PER_KPAGE);
int fill;
void *mapped_addr;
u32 mmap_count;
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 06/12] bcache: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
` (4 preceding siblings ...)
2015-05-20 1:37 ` [PATCH 05/12] IB/ehca: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-20 1:37 ` [PATCH 07/12] spider_net: " Joe Perches
` (6 subsequent siblings)
12 siblings, 0 replies; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel, Kent Overstreet; +Cc: Neil Brown, linux-bcache, linux-raid
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/md/bcache/journal.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/md/bcache/journal.c b/drivers/md/bcache/journal.c
index fe080ad..ce64fc8 100644
--- a/drivers/md/bcache/journal.c
+++ b/drivers/md/bcache/journal.c
@@ -157,7 +157,7 @@ int bch_journal_read(struct cache_set *c, struct list_head *list)
for_each_cache(ca, c, iter) {
struct journal_device *ja = &ca->journal;
- unsigned long bitmap[SB_JOURNAL_BUCKETS / BITS_PER_LONG];
+ DECLARE_BITMAP(bitmap, SB_JOURNAL_BUCKETS);
unsigned i, l, r, m;
uint64_t seq;
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 07/12] spider_net: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
` (5 preceding siblings ...)
2015-05-20 1:37 ` [PATCH 06/12] bcache: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-21 21:18 ` David Miller
2015-05-20 1:37 ` [PATCH 08/12] s390/sclp: " Joe Perches
` (5 subsequent siblings)
12 siblings, 1 reply; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel, Ishizaki Kou, Jens Osterkamp; +Cc: netdev
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/net/ethernet/toshiba/spider_net.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/toshiba/spider_net.c b/drivers/net/ethernet/toshiba/spider_net.c
index 8e9371a..3c54a2c 100644
--- a/drivers/net/ethernet/toshiba/spider_net.c
+++ b/drivers/net/ethernet/toshiba/spider_net.c
@@ -604,8 +604,7 @@ spider_net_set_multi(struct net_device *netdev)
int i;
u32 reg;
struct spider_net_card *card = netdev_priv(netdev);
- unsigned long bitmask[SPIDER_NET_MULTICAST_HASHES / BITS_PER_LONG] =
- {0, };
+ DECLARE_BITMAP(bitmask, SPIDER_NET_MULTICAST_HASHES) = {};
spider_net_set_promisc(card);
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 08/12] s390/sclp: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
` (6 preceding siblings ...)
2015-05-20 1:37 ` [PATCH 07/12] spider_net: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-20 7:03 ` Heiko Carstens
2015-05-20 1:37 ` [PATCH 09/12] [SCSI] qla4xxx: " Joe Perches
` (4 subsequent siblings)
12 siblings, 1 reply; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel; +Cc: Martin Schwidefsky, Heiko Carstens, linux390, linux-s390
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/s390/char/sclp_cmd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/s390/char/sclp_cmd.c b/drivers/s390/char/sclp_cmd.c
index a186918..f74c040 100644
--- a/drivers/s390/char/sclp_cmd.c
+++ b/drivers/s390/char/sclp_cmd.c
@@ -186,7 +186,7 @@ int sclp_cpu_deconfigure(u8 cpu)
static DEFINE_MUTEX(sclp_mem_mutex);
static LIST_HEAD(sclp_mem_list);
static u8 sclp_max_storage_id;
-static unsigned long sclp_storage_ids[256 / BITS_PER_LONG];
+static DECLARE_BITMAP(sclp_storage_ids, 256);
static int sclp_mem_state_changed;
struct memory_increment {
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH 08/12] s390/sclp: Use DECLARE_BITMAP
2015-05-20 1:37 ` [PATCH 08/12] s390/sclp: " Joe Perches
@ 2015-05-20 7:03 ` Heiko Carstens
0 siblings, 0 replies; 26+ messages in thread
From: Heiko Carstens @ 2015-05-20 7:03 UTC (permalink / raw)
To: Joe Perches; +Cc: linux-kernel, Martin Schwidefsky, linux390, linux-s390
On Tue, May 19, 2015 at 06:37:56PM -0700, Joe Perches wrote:
> Use the generic mechanism to declare a bitmap instead of unsigned long.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/s390/char/sclp_cmd.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied, thanks.
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 09/12] [SCSI] qla4xxx: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
` (7 preceding siblings ...)
2015-05-20 1:37 ` [PATCH 08/12] s390/sclp: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-20 13:04 ` Nilesh Javali
2015-05-20 1:37 ` [PATCH 10/12] scsi: " Joe Perches
` (3 subsequent siblings)
12 siblings, 1 reply; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel, QLogic-Storage-Upstream; +Cc: James E.J. Bottomley, linux-scsi
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/scsi/qla4xxx/ql4_def.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/qla4xxx/ql4_def.h b/drivers/scsi/qla4xxx/ql4_def.h
index 8f6d0fb..32341ac 100644
--- a/drivers/scsi/qla4xxx/ql4_def.h
+++ b/drivers/scsi/qla4xxx/ql4_def.h
@@ -795,7 +795,7 @@ struct scsi_qla_host {
#define CHAP_DMA_BLOCK_SIZE 512
struct workqueue_struct *task_wq;
- unsigned long ddb_idx_map[MAX_DDB_ENTRIES / BITS_PER_LONG];
+ DECLARE_BITMAP(ddb_idx_map, MAX_DDB_ENTRIES);
#define SYSFS_FLAG_FW_SEL_BOOT 2
struct iscsi_boot_kset *boot_kset;
struct ql4_boot_tgt_info boot_tgt;
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* RE: [PATCH 09/12] [SCSI] qla4xxx: Use DECLARE_BITMAP
2015-05-20 1:37 ` [PATCH 09/12] [SCSI] qla4xxx: " Joe Perches
@ 2015-05-20 13:04 ` Nilesh Javali
0 siblings, 0 replies; 26+ messages in thread
From: Nilesh Javali @ 2015-05-20 13:04 UTC (permalink / raw)
To: Joe Perches, linux-kernel, Dept-Eng QLogic Storage Upstream
Cc: James E.J. Bottomley, linux-scsi
> -----Original Message-----
> From: Joe Perches [mailto:joe@perches.com]
> Sent: Wednesday, May 20, 2015 7:08 AM
> To: linux-kernel; Dept-Eng QLogic Storage Upstream
> Cc: James E.J. Bottomley; linux-scsi
> Subject: [PATCH 09/12] [SCSI] qla4xxx: Use DECLARE_BITMAP
>
> Use the generic mechanism to declare a bitmap instead of unsigned long.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/scsi/qla4xxx/ql4_def.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/qla4xxx/ql4_def.h b/drivers/scsi/qla4xxx/ql4_def.h
> index 8f6d0fb..32341ac 100644
> --- a/drivers/scsi/qla4xxx/ql4_def.h
> +++ b/drivers/scsi/qla4xxx/ql4_def.h
> @@ -795,7 +795,7 @@ struct scsi_qla_host {
>
> #define CHAP_DMA_BLOCK_SIZE 512
> struct workqueue_struct *task_wq;
> - unsigned long ddb_idx_map[MAX_DDB_ENTRIES / BITS_PER_LONG];
> + DECLARE_BITMAP(ddb_idx_map, MAX_DDB_ENTRIES);
> #define SYSFS_FLAG_FW_SEL_BOOT 2
> struct iscsi_boot_kset *boot_kset;
> struct ql4_boot_tgt_info boot_tgt;
> --
> 2.1.2
Reviewed-by: Nilesh Javali <nilesh.javali@qlogic.com>
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 10/12] scsi: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
` (8 preceding siblings ...)
2015-05-20 1:37 ` [PATCH 09/12] [SCSI] qla4xxx: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-20 5:03 ` Bart Van Assche
2015-05-20 1:37 ` [PATCH 11/12] logfs: " Joe Perches
` (2 subsequent siblings)
12 siblings, 1 reply; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel, Jens Axboe, James E.J. Bottomley; +Cc: linux-scsi
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/scsi/sr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/sr.c b/drivers/scsi/sr.c
index 8bd54a6..f40c957 100644
--- a/drivers/scsi/sr.c
+++ b/drivers/scsi/sr.c
@@ -99,7 +99,7 @@ static struct scsi_driver sr_template = {
.done = sr_done,
};
-static unsigned long sr_index_bits[SR_DISKS / BITS_PER_LONG];
+static DECLARE_BITMAP(sr_index_bits, SR_DISKS);
static DEFINE_SPINLOCK(sr_index_lock);
/* This semaphore is used to mediate the 0->1 reference get in the
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH 10/12] scsi: Use DECLARE_BITMAP
2015-05-20 1:37 ` [PATCH 10/12] scsi: " Joe Perches
@ 2015-05-20 5:03 ` Bart Van Assche
0 siblings, 0 replies; 26+ messages in thread
From: Bart Van Assche @ 2015-05-20 5:03 UTC (permalink / raw)
To: Joe Perches, linux-kernel, Jens Axboe, James E.J. Bottomley; +Cc: linux-scsi
On 05/20/15 03:37, Joe Perches wrote:
> Use the generic mechanism to declare a bitmap instead of unsigned long.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/scsi/sr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/sr.c b/drivers/scsi/sr.c
> index 8bd54a6..f40c957 100644
> --- a/drivers/scsi/sr.c
> +++ b/drivers/scsi/sr.c
> @@ -99,7 +99,7 @@ static struct scsi_driver sr_template = {
> .done = sr_done,
> };
>
> -static unsigned long sr_index_bits[SR_DISKS / BITS_PER_LONG];
> +static DECLARE_BITMAP(sr_index_bits, SR_DISKS);
> static DEFINE_SPINLOCK(sr_index_lock);
>
> /* This semaphore is used to mediate the 0->1 reference get in the
Reviewed-by: Bart Van Assche <bart.vanassche@sandisk.com>
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 11/12] logfs: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
` (9 preceding siblings ...)
2015-05-20 1:37 ` [PATCH 10/12] scsi: " Joe Perches
@ 2015-05-20 1:37 ` Joe Perches
2015-05-20 1:38 ` [PATCH 12/12] sunrpc: " Joe Perches
[not found] ` <OF4147CCA9.838D5302-ONC1257E4B.000904D1-C1257E4B.000904DB@de.ibm.com>
12 siblings, 0 replies; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:37 UTC (permalink / raw)
To: linux-kernel, Joern Engel, Prasad Joshi; +Cc: logfs
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
fs/logfs/logfs.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/logfs/logfs.h b/fs/logfs/logfs.h
index 5f09376..9b8d8a6 100644
--- a/fs/logfs/logfs.h
+++ b/fs/logfs/logfs.h
@@ -301,7 +301,7 @@ struct logfs_block {
struct page *page;
struct inode *inode;
struct logfs_transaction *ta;
- unsigned long alias_map[LOGFS_BLOCK_FACTOR / BITS_PER_LONG];
+ DECLARE_BITMAP(alias_map, LOGFS_BLOCK_FACTOR);
struct logfs_block_ops *ops;
int full;
int partial;
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 12/12] sunrpc: Use DECLARE_BITMAP
2015-05-20 1:37 [PATCH 00/12] treewide: Use DECLARE_BITMAP Joe Perches
` (10 preceding siblings ...)
2015-05-20 1:37 ` [PATCH 11/12] logfs: " Joe Perches
@ 2015-05-20 1:38 ` Joe Perches
[not found] ` <OF4147CCA9.838D5302-ONC1257E4B.000904D1-C1257E4B.000904DB@de.ibm.com>
12 siblings, 0 replies; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:38 UTC (permalink / raw)
To: linux-kernel
Cc: J. Bruce Fields, Trond Myklebust, Anna Schumaker,
David S. Miller, linux-nfs, netdev
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by: Joe Perches <joe@perches.com>
---
net/sunrpc/auth_gss/svcauth_gss.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/sunrpc/auth_gss/svcauth_gss.c b/net/sunrpc/auth_gss/svcauth_gss.c
index 1095be9..e417476 100644
--- a/net/sunrpc/auth_gss/svcauth_gss.c
+++ b/net/sunrpc/auth_gss/svcauth_gss.c
@@ -320,7 +320,7 @@ struct gss_svc_seq_data {
int sd_max;
/* for i such that sd_max-GSS_SEQ_WIN < i <= sd_max, the i-th bit of
* sd_win is nonzero iff sequence number i has been seen already: */
- unsigned long sd_win[GSS_SEQ_WIN/BITS_PER_LONG];
+ DECLARE_BITMAP(sd_win, GSS_SEQ_WIN);
spinlock_t sd_lock;
};
--
2.1.2
^ permalink raw reply related [flat|nested] 26+ messages in thread[parent not found: <OF4147CCA9.838D5302-ONC1257E4B.000904D1-C1257E4B.000904DB@de.ibm.com>]
* [possible PATCH -next] MAINTAINERS: Jens Axboe has changed email address
[not found] ` <OF4147CCA9.838D5302-ONC1257E4B.000904D1-C1257E4B.000904DB@de.ibm.com>
@ 2015-05-20 1:50 ` Joe Perches
2015-05-20 3:48 ` Jens Axboe
0 siblings, 1 reply; 26+ messages in thread
From: Joe Perches @ 2015-05-20 1:50 UTC (permalink / raw)
To: Jens Axboe; +Cc: LKML, Andrew Morton
Jens has changed email address, update his entries in MAINTAINERS
Maybe it'd be better to use an address that won't expire when
job functions change.
Signed-off-by: Joe Perches <joe@perches.com>
---
On Tue, 2015-05-19 at 18:37 -0700, Postmaster@de.ibm.com wrote:
> Your message was not delivered to:
> jens@de.ibm.com
> because:
> User jens (jens@de.ibm.com) not listed in Domino Directory
MAINTAINERS | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 86d9398..fb1a38d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2087,7 +2087,7 @@ S: Maintained
F: drivers/leds/leds-blinkm.c
BLOCK LAYER
-M: Jens Axboe <axboe@kernel.dk>
+M: Jens Axboe <axboe@fb.com>
T: git git://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git
S: Maintained
F: block/
@@ -8232,7 +8232,7 @@ S: Maintained
F: drivers/net/wireless/rt2x00/
RAMDISK RAM BLOCK DEVICE DRIVER
-M: Jens Axboe <axboe@kernel.dk>
+M: Jens Axboe <axboe@fb.com>
S: Maintained
F: Documentation/blockdev/ramdisk.txt
F: drivers/block/brd.c
@@ -8765,7 +8765,7 @@ S: Supported
F: arch/score/
SCSI CDROM DRIVER
-M: Jens Axboe <axboe@kernel.dk>
+M: Jens Axboe <axboe@fb.com>
L: linux-scsi@vger.kernel.org
W: http://www.kernel.dk
S: Maintained
@@ -10301,7 +10301,7 @@ S: Maintained
F: scripts/unifdef.c
UNIFORM CDROM DRIVER
-M: Jens Axboe <axboe@kernel.dk>
+M: Jens Axboe <axboe@fb.com>
W: http://www.kernel.dk
S: Maintained
F: Documentation/cdrom/
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [possible PATCH -next] MAINTAINERS: Jens Axboe has changed email address
2015-05-20 1:50 ` [possible PATCH -next] MAINTAINERS: Jens Axboe has changed email address Joe Perches
@ 2015-05-20 3:48 ` Jens Axboe
2015-05-20 3:57 ` Joe Perches
0 siblings, 1 reply; 26+ messages in thread
From: Jens Axboe @ 2015-05-20 3:48 UTC (permalink / raw)
To: Joe Perches; +Cc: LKML, Andrew Morton
> On May 19, 2015, at 7:50 PM, Joe Perches <joe@perches.com> wrote:
>
> Jens has changed email address, update his entries in MAINTAINERS
Uhm no I didn't, I think you messed up with some IBM address.
> Maybe it'd be better to use an address that won't expire when
> job functions change.
And your patch changes it from just that to an employer email :-)
--
Jens Axboe
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [possible PATCH -next] MAINTAINERS: Jens Axboe has changed email address
2015-05-20 3:48 ` Jens Axboe
@ 2015-05-20 3:57 ` Joe Perches
2015-05-20 14:13 ` Jens Axboe
0 siblings, 1 reply; 26+ messages in thread
From: Joe Perches @ 2015-05-20 3:57 UTC (permalink / raw)
To: Jens Axboe; +Cc: LKML, Andrew Morton
On Wed, 2015-05-20 at 03:48 +0000, Jens Axboe wrote:
> > On May 19, 2015, at 7:50 PM, Joe Perches <joe@perches.com> wrote:
> >
> > Jens has changed email address, update his entries in MAINTAINERS
>
> Uhm no I didn't, I think you messed up with some IBM address.
>
> > Maybe it'd be better to use an address that won't expire when
> > job functions change.
>
> And your patch changes it from just that to an employer email :-)
Apologies, my mistake, it was a different Jens.
I saw Jens in the original bounce and just assumed it was
you because "there can be only one"...
It was Jens Osterkamp <jens@de.ibm.com>
That Jens hasn't signed a linux commit since 2008.
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [possible PATCH -next] MAINTAINERS: Jens Axboe has changed email address
2015-05-20 3:57 ` Joe Perches
@ 2015-05-20 14:13 ` Jens Axboe
0 siblings, 0 replies; 26+ messages in thread
From: Jens Axboe @ 2015-05-20 14:13 UTC (permalink / raw)
To: Joe Perches; +Cc: LKML, Andrew Morton
On 05/19/2015 09:57 PM, Joe Perches wrote:
> On Wed, 2015-05-20 at 03:48 +0000, Jens Axboe wrote:
>>> On May 19, 2015, at 7:50 PM, Joe Perches <joe@perches.com> wrote:
>>>
>>> Jens has changed email address, update his entries in MAINTAINERS
>>
>> Uhm no I didn't, I think you messed up with some IBM address.
>>
>>> Maybe it'd be better to use an address that won't expire when
>>> job functions change.
>>
>> And your patch changes it from just that to an employer email :-)
>
> Apologies, my mistake, it was a different Jens.
>
> I saw Jens in the original bounce and just assumed it was
> you because "there can be only one"...
I'm not saying I had anything to do with Jens@IBM disappearance, but
yes, there can only be one.
--
Jens Axboe
^ permalink raw reply [flat|nested] 26+ messages in thread