Re: [PATCH v2 1/3] zram: Replace bit spinlocks with a spinlock_t.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Sergey Senozhatsky <senozhatsky@xxxxxxxxxxxx>
Subject: Re: [PATCH v2 1/3] zram: Replace bit spinlocks with a spinlock_t.
From: Sergey Senozhatsky <senozhatsky@xxxxxxxxxxxx>
Date: Mon, 8 Jul 2024 12:32:28 +0900
Cc: Alexander Lobakin <aleksander.lobakin@xxxxxxxxx>, Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>, linux-block@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Minchan Kim <minchan@xxxxxxxxxx>, Jens Axboe <axboe@xxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Mike Galbraith <umgwanakikbuti@xxxxxxxxx>
In-reply-to: <20240708030330.GA797471@google.com>
References: <20240620153556.777272-1-bigeasy@linutronix.de> <20240620153556.777272-2-bigeasy@linutronix.de> <27fb4f62-d656-449c-9f3c-5d0b61a88cca@intel.com> <20240704121908.GjH4p40u@linutronix.de> <801cac51-1bd3-4f79-8474-251a7a81ca08@intel.com> <20240708030330.GA797471@google.com>

On (24/07/08 12:03), Sergey Senozhatsky wrote:
[..]
> > I meant
> > 
> > 	for (size_t index = 0; index < num_pages; index++)
> > 
> > It's allowed and even recommended for a couple years already.
> 
> I wonder since when?  Do gcc 5.1 and clang 13.0.1 support this?

Since C99.  gcc 5.1/clang 13.0.1 are fine with that.  TIL.

References:
- [PATCH v2 0/3] zram: Replace bit spinlocks with a spinlock_t.
  - From: Sebastian Andrzej Siewior
- [PATCH v2 1/3] zram: Replace bit spinlocks with a spinlock_t.
  - From: Sebastian Andrzej Siewior
- Re: [PATCH v2 1/3] zram: Replace bit spinlocks with a spinlock_t.
  - From: Alexander Lobakin
- Re: [PATCH v2 1/3] zram: Replace bit spinlocks with a spinlock_t.
  - From: Sebastian Andrzej Siewior
- Re: [PATCH v2 1/3] zram: Replace bit spinlocks with a spinlock_t.
  - From: Alexander Lobakin
- Re: [PATCH v2 1/3] zram: Replace bit spinlocks with a spinlock_t.
  - From: Sergey Senozhatsky

Prev by Date: Re: [PATCH v2 1/3] zram: Replace bit spinlocks with a spinlock_t.
Next by Date: Re: blk-cgroup: add spin_lock for u64_stats_update
Previous by thread: Re: [PATCH v2 1/3] zram: Replace bit spinlocks with a spinlock_t.
Next by thread: [PATCH v2 2/3] zram: Remove ZRAM_LOCK
Index(es):
- Date
- Thread

[Index of Archives] [Linux RAID] [Linux SCSI] [Linux ATA RAID] [IDE] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Device Mapper]