On 4/10/21 8:52 AM, Christoph Hellwig wrote:
On Fri, Apr 09, 2021 at 05:46:09PM +0200, Steffen Maier wrote:
Is it OK to remove a long standing sysfs attribute?
Was there any deprecation phase?
I just noticed it in our CI reporting fails due to this change since at
least linux-next 20210409. Suppose it came via
https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git/commit/?h=for-5.13/block&id=aaff5ebaa2694f283b7d07fdd55fb287ffc4f1e9
Wanted to ask before I adapt the test cases.
What does your CI do with it?
We check almost all zfcp-related sysfs attributes for returning expected values
(depending on possible error inject state transitions as well as steady state).
The Scsi_Host's unchecked_isa_dma is just checked to return '0'.
It would be kinda of sad to carry around this baggage, but if real
userspace is broken by the chance we'll have to do it. I'm just not
entirely sure testcases qualify as real userspace yet.
I was more pondering whether sysfs is considered stable API.
I'm not aware of any issues with real userspace if unchecked_isa_dma is absent.
Have adapted our test case to accept a missing unchecked_isa_dma sysfs attribute.
--
Mit freundlichen Gruessen / Kind regards
Steffen Maier
Linux on IBM Z Development
https://www.ibm.com/privacy/us/en/
IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Matthias Hartmann
Geschaeftsfuehrung: Dirk Wittkopp
Sitz der Gesellschaft: Boeblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294
