2023-04-20 17:42:50

by Sarthak Kukreti

[permalink] [raw]
Subject: Re: [PATCH v4 1/4] block: Introduce provisioning primitives

Dropped in v5.

Thanks!
Sarthak

On Tue, Apr 18, 2023 at 3:43 PM Bart Van Assche <[email protected]> wrote:
>
> On 4/18/23 15:12, Sarthak Kukreti wrote:
> > /* Fail if we don't recognize the flags. */
> > - if (mode & ~BLKDEV_FALLOC_FL_SUPPORTED)
> > + if (mode != 0 && mode & ~BLKDEV_FALLOC_FL_SUPPORTED)
> > return -EOPNOTSUPP;
>
> Is this change necessary? Doesn't (mode & ~BLKDEV_FALLOC_FL_SUPPORTED)
> != 0 imply that mode != 0?
>
> Thanks,
>
> Bart.
>