From: "Martin K. Petersen" <martin.petersen@oracle.com>
To: Roman Mamedov <rm@romanrm.net>
Cc: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>,
Logan Gunthorpe <logang@deltatee.com>,
linux-raid@vger.kernel.org, Jes Sorensen <jes@trained-monkey.org>,
Guoqing Jiang <guoqing.jiang@linux.dev>, Xiao Ni <xni@redhat.com>,
Coly Li <colyli@suse.de>,
Chaitanya Kulkarni <chaitanyak@nvidia.com>,
Jonmichael Hands <jm@chia.net>,
Stephen Bates <sbates@raithlin.com>,
Martin Oliveira <Martin.Oliveira@eideticom.com>,
David Sloan <David.Sloan@eideticom.com>
Subject: Re: [PATCH mdadm v2 1/2] mdadm: Add --discard option for Create
Date: Mon, 12 Sep 2022 13:43:33 -0400 [thread overview]
Message-ID: <yq1a674bie5.fsf@ca-mkp.ca.oracle.com> (raw)
In-Reply-To: <20220909165417.4161b5a8@nvm> (Roman Mamedov's message of "Fri, 9 Sep 2022 16:54:17 +0500")
Roman,
> In the ATA layer there is the sysfs file "discard_zeroes_data", which
> tells whether or not to expect zeroes after a discard. That flag does
> not appear to be always correct, e.g. I have some SSDs which do
> observably read back zeroes after a discard, but the flag still says
> "0".
It is hardwired to "0" for backwards compatibility. We don't support
discards for clearing block ranges anymore. There is an explicit zeroing
operation for that use case.
--
Martin K. Petersen Oracle Linux Engineering
next prev parent reply other threads:[~2022-09-12 17:44 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-08 23:08 [PATCH mdadm v2 0/2] Discard Option for Creating Arrays Logan Gunthorpe
2022-09-08 23:08 ` [PATCH mdadm v2 1/2] mdadm: Add --discard option for Create Logan Gunthorpe
2022-09-09 9:57 ` Mariusz Tkaczyk
2022-09-09 11:54 ` Roman Mamedov
[not found] ` <CABdXBANrJNWjq4237k9DPRoxLVmiAUoKMZxaaLUrcMHsODwvmA@mail.gmail.com>
2022-09-09 15:31 ` Roman Mamedov
2022-09-12 17:43 ` Martin K. Petersen [this message]
2022-09-09 15:47 ` Logan Gunthorpe
2022-09-13 7:35 ` Mariusz Tkaczyk
2022-09-13 15:43 ` Logan Gunthorpe
2022-09-14 12:01 ` Mariusz Tkaczyk
2022-09-14 16:29 ` Logan Gunthorpe
2022-09-14 17:39 ` Mariusz Tkaczyk
2022-09-19 8:41 ` Xiao Ni
2022-09-21 18:45 ` Logan Gunthorpe
2022-09-08 23:08 ` [PATCH mdadm v2 2/2] manpage: Add --discard option to manpage Logan Gunthorpe
2022-09-12 17:40 ` [PATCH mdadm v2 0/2] Discard Option for Creating Arrays Martin K. Petersen
[not found] ` <CABdXBAP0LeQMmhSLUMZ_TmnSp5xmZ4xJBkNa7HUm7094m_x9xA@mail.gmail.com>
2022-09-13 3:47 ` Martin K. Petersen
2022-09-13 15:38 ` Logan Gunthorpe
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=yq1a674bie5.fsf@ca-mkp.ca.oracle.com \
--to=martin.petersen@oracle.com \
--cc=David.Sloan@eideticom.com \
--cc=Martin.Oliveira@eideticom.com \
--cc=chaitanyak@nvidia.com \
--cc=colyli@suse.de \
--cc=guoqing.jiang@linux.dev \
--cc=jes@trained-monkey.org \
--cc=jm@chia.net \
--cc=linux-raid@vger.kernel.org \
--cc=logang@deltatee.com \
--cc=mariusz.tkaczyk@linux.intel.com \
--cc=rm@romanrm.net \
--cc=sbates@raithlin.com \
--cc=xni@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.