[0/2] block size limit cleanups

Message

Luis Chamberlain Dec. 18, 2024, 2:02 a.m. UTC

This spins off two change which introduces no functional changes from the
bs > ps block device patch series [0]. These are just cleanups.

[0] https://lkml.kernel.org/r/[e-mail]

Luis Chamberlain (2):
  block/bdev: use helper for max block size check
  nvme: use blk_validate_block_size() for max LBA check

 block/bdev.c             | 3 +--
 drivers/nvme/host/core.c | 2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

Comments

Keith Busch Dec. 18, 2024, 2:09 a.m. UTC | #1

On Tue, Dec 17, 2024 at 06:02:10PM -0800, Luis Chamberlain wrote:
> This spins off two change which introduces no functional changes from the
> bs > ps block device patch series [0]. These are just cleanups.

Looks good.

Reviewed-by: Keith Busch <[e-mail]>

Christoph Hellwig Dec. 18, 2024, 7:21 a.m. UTC | #2

Looks good:

Reviewed-by: Christoph Hellwig <[e-mail]>

(the CC list feels a little excessive, though)

Jens Axboe Dec. 18, 2024, 2:22 p.m. UTC | #3

On Tue, 17 Dec 2024 18:02:10 -0800, Luis Chamberlain wrote:
> This spins off two change which introduces no functional changes from the
> bs > ps block device patch series [0]. These are just cleanups.
> 
> [0] https://lkml.kernel.org/r/[e-mail]
> 
> Luis Chamberlain (2):
>   block/bdev: use helper for max block size check
>   nvme: use blk_validate_block_size() for max LBA check
> 
> [...]

Applied, thanks!

[1/2] block/bdev: use helper for max block size check
      commit: 26fff8a4432ffd03409346b7dae1e1a2c5318b7c
[2/2] nvme: use blk_validate_block_size() for max LBA check
      commit: 51588b1b77b65cd0fb3440f78f37bef7178a2715

Best regards,

Jens Axboe Dec. 18, 2024, 2:25 p.m. UTC | #4

On 12/18/24 7:22 AM, Jens Axboe wrote:
> 
> On Tue, 17 Dec 2024 18:02:10 -0800, Luis Chamberlain wrote:
>> This spins off two change which introduces no functional changes from the
>> bs > ps block device patch series [0]. These are just cleanups.
>>
>> [0] https://lkml.kernel.org/r/[e-mail]
>>
>> Luis Chamberlain (2):
>>   block/bdev: use helper for max block size check
>>   nvme: use blk_validate_block_size() for max LBA check
>>
>> [...]
> 
> Applied, thanks!
> 
> [1/2] block/bdev: use helper for max block size check
>       commit: 26fff8a4432ffd03409346b7dae1e1a2c5318b7c
> [2/2] nvme: use blk_validate_block_size() for max LBA check
>       commit: 51588b1b77b65cd0fb3440f78f37bef7178a2715

JFYI, gmail puts all your emails into spam, because it can't
authenticate them. This seemingly happens to everybody that uses
kernel.org as the email, if they haven't updated to the newer setup. You
should fix it, I only saw the emails because others replied.

And please trim the way excessive CC lists on patches like this.
Cleanups don't need to go to tons of unrelated lists or people.

Max Gurtovoy Dec. 18, 2024, 5:14 p.m. UTC | #5

On 18/12/2024 4:02, Luis Chamberlain wrote:
> This spins off two change which introduces no functional changes from the
> bs > ps block device patch series [0]. These are just cleanups.
>
> [0] https://lkml.kernel.org/r/[e-mail]
>
> Luis Chamberlain (2):
>    block/bdev: use helper for max block size check
>    nvme: use blk_validate_block_size() for max LBA check
>
>   block/bdev.c             | 3 +--
>   drivers/nvme/host/core.c | 2 +-
>   2 files changed, 2 insertions(+), 3 deletions(-)

Looks good,

Reviewed-by: Max Gurtovoy <[e-mail]>