treewide: Use DECLARE_BITMAP
Subsystems (18)
cover treewide: Use DECLARE_BITMAP
Use the standard method to declare a bitmap array.
Joe Perches (12):
ARM: mach-imx: iomux-imx31: Use DECLARE_BITMAP
dmaengine: rcar-dmac: Use DECLARE_BITMAP
drm/amdkfd: Use DECLARE_BITMAP
drm/radeon: Use DECLARE_BITMAP
IB/ehca: Use DECLARE_BITMAP
bcache: Use DECLARE_BITMAP
spider_net: Use DECLARE_BITMAP
s390/sclp: Use DECLARE_BITMAP
[SCSI] qla4xxx: Use DECLARE_BITMAP
scsi: Use DECLARE_BITMAP
logfs: Use DECLARE_BITMAP
sunrpc: Use DECLARE_BITMAP
arch/arm/mach-imx/iomux-imx31.c | 2 +- drivers/dma/sh/rcar-dmac.c | 2 +- drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 5 ++--- drivers/gpu/drm/radeon/radeon.h | 2 +- drivers/infiniband/hw/ehca/ipz_pt_fn.h | 2 +- drivers/md/bcache/journal.c | 2 +- drivers/net/ethernet/toshiba/spider_net.c | 3 +-- drivers/s390/char/sclp_cmd.c | 2 +- drivers/scsi/qla4xxx/ql4_def.h | 2 +- drivers/scsi/sr.c | 2 +- fs/logfs/logfs.h | 2 +- net/sunrpc/auth_gss/svcauth_gss.c | 2 +- 12 files changed, 13 insertions(+), 15 deletions(-)
Signature · 1 line
2.1.2
PATCH 1/12 ARM: mach-imx: iomux-imx31: Use DECLARE_BITMAP
Use the generic mechanism to declare a bitmap instead of unsigned long.
--- arch/arm/mach-imx/iomux-imx31.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/mach-imx/iomux-imx31.c +1 -1
@@ -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. */
Signature · 1 line
2.1.2
Re: [PATCH 01/12] ARM: mach-imx: iomux-imx31: Use DECLARE_BITMAP
Hello,
Quoted 12 lines · Joe Perches
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by Joe Perches <[email protected]>--- arch/arm/mach-imx/iomux-imx31.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)arch/arm/mach-imx/iomux-imx31.c +0
-0
arch/arm/mach-imx/iomux-imx31.c static DEFINE_SPINLOCK(gpio_mux_lock); · line 46
Quoted 8 lines
arch/arm/mach-imx/iomux-imx31.c +1
-1@@ -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
Best regards
Uwe
Signature · 2 lines
Pengutronix e.K. | Uwe Kleine-K?nig | Industrial Linux Solutions | http://www.pengutronix.de/ |
Shawn Guo
AppliedRe: [PATCH 01/12] ARM: mach-imx: iomux-imx31: Use DECLARE_BITMAP
In reply to Joe PerchesQuoted 3 lines · Joe Perches
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by Joe Perches <[email protected]>
Applied, thanks.
PATCH 2/12 dmaengine: rcar-dmac: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- drivers/dma/sh/rcar-dmac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/dma/sh/rcar-dmac.c +1 -1
@@ -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)
Signature · 1 line
2.1.2
Vinod Koul
AppliedRe: [PATCH 02/12] dmaengine: rcar-dmac: Use DECLARE_BITMAP
Quoted 2 lines · Joe Perches
Use the generic mechanism to declare a bitmap instead of unsigned long.
Applied, thanks
Signature · 1 line
~Vinod
PATCH 3/12 drm/amdkfd: Use DECLARE_BITMAP
In reply to Joe PerchesUse 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.
--- drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
drivers/gpu/drm/amd/amdkfd/kfd_priv.h +2 -3
@@ -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;
Signature · 1 line
2.1.2
Re: [PATCH 03/12] drm/amdkfd: Use DECLARE_BITMAP
Quoted 36 lines · Joe Perches
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 <[email protected]>--- drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)drivers/gpu/drm/amd/amdkfd/kfd_priv.h +2
-4@@ -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
Quoted 5 lines
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
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
PATCH 4/12 drm/radeon: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- drivers/gpu/drm/radeon/radeon.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/gpu/drm/radeon/radeon.h +1 -1
@@ -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);
Signature · 1 line
2.1.2
Alex Deucher
AppliedRe: [PATCH 04/12] drm/radeon: Use DECLARE_BITMAP
Quoted 3 lines · Joe Perches
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by Joe Perches <[email protected]>
Applied to my -next tree. thanks!
Alex
Quoted 20 lines
--- drivers/gpu/drm/radeon/radeon.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)drivers/gpu/drm/radeon/radeon.h +1
-2@@ -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
Quoted 4 lines
_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel
PATCH 5/12 IB/ehca: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- drivers/infiniband/hw/ehca/ipz_pt_fn.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/infiniband/hw/ehca/ipz_pt_fn.h +1 -1
@@ -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;
Signature · 1 line
2.1.2
PATCH 6/12 bcache: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- drivers/md/bcache/journal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/md/bcache/journal.c +1 -1
@@ -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;
Signature · 1 line
2.1.2
PATCH 7/12 spider_net: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- drivers/net/ethernet/toshiba/spider_net.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/net/ethernet/toshiba/spider_net.c +1 -2
@@ -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);
Signature · 1 line
2.1.2
Re: [PATCH 07/12] spider_net: Use DECLARE_BITMAP
From: Joe Perches <[email protected]>
Date: Tue, 19 May 2015 18:37:55 -0700
Quoted 3 lines
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by Joe Perches <[email protected]>
Applied to net-next, thanks Joe.
PATCH 8/12 s390/sclp: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- drivers/s390/char/sclp_cmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/s390/char/sclp_cmd.c +1 -1
@@ -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 {
Signature · 1 line
2.1.2
Heiko Carstens
AppliedRe: [PATCH 08/12] s390/sclp: Use DECLARE_BITMAP
Quoted 6 lines · Joe Perches
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by Joe Perches <[email protected]>--- drivers/s390/char/sclp_cmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Applied, thanks.
PATCH 9/12 qla4xxx: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- drivers/scsi/qla4xxx/ql4_def.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/scsi/qla4xxx/ql4_def.h +1 -1
@@ -795,7 +795,7 @@ struct scsi_qla_host {#define CHAP_DMA_BLOCK_SIZE 512struct 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 2struct iscsi_boot_kset *boot_kset;struct ql4_boot_tgt_info boot_tgt;
Signature · 1 line
2.1.2
RE: [PATCH 09/12] [SCSI] qla4xxx: Use DECLARE_BITMAP
Quoted 28 lines
-----Original Message-----
From: Joe Perches [mailto:[email protected]]
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: UseDECLARE_BITMAPUse the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by Joe Perches <[email protected]>--- drivers/scsi/qla4xxx/ql4_def.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)drivers/scsi/qla4xxx/ql4_def.h +1
-2@@ -795,7 +795,7 @@ struct scsi_qla_host {#define CHAP_DMA_BLOCK_SIZE 512struct 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 2struct iscsi_boot_kset *boot_kset;struct ql4_boot_tgt_info boot_tgt; --
Quoted 1 line
2.1.2
PATCH 10/12 scsi: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- drivers/scsi/sr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/scsi/sr.c +1 -1
@@ -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
Signature · 1 line
2.1.2
Re: [PATCH 10/12] scsi: Use DECLARE_BITMAP
Quoted 12 lines · Joe Perches
Use the generic mechanism to declare a bitmap instead of unsigned long.
Signed-off-by Joe Perches <[email protected]>--- drivers/scsi/sr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)drivers/scsi/sr.c +0
-0
drivers/scsi/sr.c static struct scsi_driver sr_template = { · line 105
Quoted 8 lines
drivers/scsi/sr.c +1
-1@@ -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
PATCH 11/12 logfs: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- fs/logfs/logfs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
fs/logfs/logfs.h +1 -1
@@ -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;
Signature · 1 line
2.1.2
PATCH 12/12 sunrpc: Use DECLARE_BITMAP
In reply to Joe PerchesUse the generic mechanism to declare a bitmap instead of unsigned long.
--- net/sunrpc/auth_gss/svcauth_gss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
net/sunrpc/auth_gss/svcauth_gss.c +1 -1
@@ -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; };
Signature · 1 line
2.1.2
A message that is not in the archive (sent off-list or not yet received): OF4147CCA9.838D5302-ONC1257E4B.000904D1-C1257E4B.000904DB@de.ibm.com
PATCH 1/1 MAINTAINERS: Jens Axboe has changed email address
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.
---
Quoted 4 lines · On Tue, 2015-05-19 at 18:37 -0700, [email protected] wrote:
Your message was not delivered to:
[email protected]
because:
User jens ([email protected]) not listed in Domino Directory
MAINTAINERS | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
MAINTAINERS +4 -4
@@ -2087,7 +2087,7 @@ S: Maintained F: drivers/leds/leds-blinkm.c BLOCK LAYER -M: Jens Axboe <[email protected]> +M: Jens Axboe <[email protected]> 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 <[email protected]> +M: Jens Axboe <[email protected]> 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 <[email protected]> +M: Jens Axboe <[email protected]> L: [email protected] W: http://www.kernel.dk S: Maintained
@@ -10301,7 +10301,7 @@ S: Maintained F: scripts/unifdef.c UNIFORM CDROM DRIVER -M: Jens Axboe <[email protected]> +M: Jens Axboe <[email protected]> W: http://www.kernel.dk S: Maintained F: Documentation/cdrom/
Re: [possible PATCH -next] MAINTAINERS: Jens Axboe has changed email address
Quoted 4 lines · Joe Perches
Jens has changed email address, update his entries in
MAINTAINERS
Uhm no I didn't, I think you messed up with some IBM address.
Quoted 2 lines
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 :-)
Signature · 1 line
Jens Axboe
Re: [possible PATCH -next] MAINTAINERS: Jens Axboe has changed email address
Quoted 10 lines · Jens Axboe
Quoted 3 lines · Joe Perches
Jens has changed email address, update his entries in
MAINTAINERSUhm no I didn't, I think you messed up with some IBM address.
Quoted 2 lines
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 <[email protected]>
That Jens hasn't signed a linux commit since 2008.
Re: [possible PATCH -next] MAINTAINERS: Jens Axboe has changed email address
Quoted 10 lines · Joe Perches
Quoted 9 lines · Jens Axboe
Quoted 3 lines · Joe Perches
Jens has changed email address, update his entries in
MAINTAINERSUhm no I didn't, I think you messed up with some IBM address.
Quoted 2 lines
Maybe it'd be better to use an address that won't expire when job functions change.
Quoted 6 lines
Quoted 1 line
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.
Signature · 1 line
Jens Axboe