On 3/11/22 9:45 AM, Bart Van Assche wrote:

On 3/10/22 14:10, Jens Axboe wrote:

quoted

On 3/10/22 2:52 PM, Bean Huo (beanhuo) wrote:

quoted

Yes, in upstream linux and upstream android, there is no such code.
But as we know, mobile customers have used bio->bi_write_hint in their
products for years. And the group ID is set according to
bio->bi_write_hint before passing the CDB to UFS.


    lrbp = &hba->lrb[tag];
                 WARN_ON(lrbp->cmd);
              + if(cmd->cmnd[0] == WRITE_10)
               +{
                 +             cmd->cmnd[6] = (0x1f& cmd->request->bio->bi_write_hint);
               +}
               lrbp->cmd = cmd;
               lrbp->sense_bufflen = UFS_SENSE_SIZE;
               lrbp->sense_buffer = cmd->sense_buffer;

I don't know why they don't push these changes to the community, maybe
it's because changes across the file system and block layers are
unacceptable to the block layer and FS. but for sure we should now
warn them to push to the community as soon as possible.

If the code isn't upstream, it's a bit late to start thinking about
that now. This feature has existed for 5 years at this point, and the
only consumer was NVMe. The upstream kernel cares only about what is
in-tree, as that is the only part we can modify and fix. We
change/modify internal kernel APIs all the time, which is how tech debt
is removed and the long term sanity of the project is maintained. This
in turn means that out-of-tree code will break, that's just a natural
side effect and something we can't do anything about.

If at some point there's a desire to actually try and upstream this
support, then we'll be happy to review that patchset. Or you can
continue to stay out-of-tree and just patch in what you need. If you're
already modifying core code, then that shouldn't be a problem.

Hi Jens,

The "upstream first" policy applies to the Android kernel (see also
https://arstechnica.com/gadgets/2021/09/android-to-take-an-upstream-first-development-model-for-the-linux-kernel/).
If anyone requests inclusion in the Android kernel tree of a patch
that is not upstream, that request is rejected unless a very strong
reason can be provided why it should be included in the Android kernel
only instead of being sent upstream. It is not clear to me why the
patch Bean mentioned is not upstream nor in the upstream Android
kernel tree.

From a UFS vendor I received the feedback that the F2FS write hint
information helps to reduce write amplification significantly. If the
write hint information is retained in the upstream kernel I can help
with making sure that the UFS patch mentioned above is integrated in
the upstream Linux kernel.

I'm really not that interested at this point, and I don't want to gate
removal or inclusion of code on some potential future event that may
never happen.

That doesn't mean that the work and process can't continue on the
Android front, the only difference is what the baseline kernel looks
like for the submitted patchset.

Hence I do think we should go ahead as planned, and then we'll just
revisit the topic if/when some actual code comes up.

-- 
Jens Axboe