linux-kernel - Re: [PATCH V5 2/2] scsi: ufs: Add configfs support for ufs provisioning

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 17 Jul 2018 00:04:34 +0000
From:   Bart Van Assche <Bart.VanAssche@....com>
To:     "sayalil@...eaurora.org" <sayalil@...eaurora.org>,
        "evgreen@...omium.org" <evgreen@...omium.org>
CC:     "vinholikatti@...il.com" <vinholikatti@...il.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "asutoshd@...eaurora.org" <asutoshd@...eaurora.org>,
        "riteshh@...eaurora.org" <riteshh@...eaurora.org>,
        "cang@...eaurora.org" <cang@...eaurora.org>,
        "martin.petersen@...cle.com" <martin.petersen@...cle.com>,
        "subhashj@...eaurora.org" <subhashj@...eaurora.org>,
        "linux-scsi@...r.kernel.org" <linux-scsi@...r.kernel.org>,
        "vivek.gautam@...eaurora.org" <vivek.gautam@...eaurora.org>,
        "rnayak@...eaurora.org" <rnayak@...eaurora.org>,
        "jejb@...ux.vnet.ibm.com" <jejb@...ux.vnet.ibm.com>
Subject: Re: [PATCH V5 2/2] scsi: ufs: Add configfs support for ufs
 provisioning

On Mon, 2018-07-16 at 16:46 -0700, Evan Green wrote:
> I see Bart has chimed in on the next series with a suggestion to break
> out each field into individual files within configfs. Bart, what are
> your feelings about converting to a binary attribute? I remember when
> I did my sysfs equivalent of this patch, somebody chimed in indicating
> a "commit" file might be needed so that the new configuration could be
> written in one fell swoop. One advantage of the binary attribute is
> that it writes the configuration atomically.

Hello Evan,

I may be missing some UFS background information. But since a configfs interface
is being added I think the same rule applies as to all Linux kernel user space
interfaces, namely that it should be backwards compatible. Additionally, if
anyone ever will want to use this interface from a shell script, I think that
it will be much easier to write multiple ASCII attributes than a single binary
attribute.

Bart.