/sys/class/scsi_host/hostN/partition_number and .../mad_version showing up BE on LE Ubuntu. (ibmvscsi)

[bugproxy]

== Comment: #0 - ERIC P. FRIED <email address hidden> - 2015-06-15 18:29:28 ==
---Problem Description---
/sys/class/scsi_host/host0/partition_number and .../mad_version showing up BE on LE Ubuntu.

---uname output---
Linux neo34 3.19.0-10-generic #10-Ubuntu SMP Mon Mar 23 16:18:35 UTC 2015 ppc64le ppc64le ppc64le GNU/Linux

---Additional Hardware Info---
Dual VIOS, Ubuntu partition is full virtual.

---Patches Installed---
VMC device driver for Neo project. Ask Steve Royer (seroyer@us) if it matters.

Machine Type = type_model=8247-22L

---Debugger---
A debugger is not configured

---Steps to Reproduce---
 neo@neo34:/sys/class/scsi_disk$ cat /sys/module/ibmvscsi/drivers/vio\:ibmvscsi/30000006/host1/scsi_host/host1/partition_number
50331648
neo@neo34:/sys/class/scsi_disk$ cat /sys/module/ibmvscsi/drivers/vio\:ibmvscsi/30000006/host1/scsi_host/host1/mad_version
16777216

Turns out these numbers are really big-endian versions of their real values.
In [60]: print("0x%x" % 50331648)
0x3000000

The actual partition ID of host1 is 3.
neo@neo34:/sys/class/scsi_disk$ lssyscfg -r lpar -F lpar_id --filter lpar_names=vios2
3

Userspace tool common name: Not sure what this means. Use 'cat' to reproduce the bug.

The userspace tool has the following bit modes: ?

Userspace rpm: Not sure what this means either. And I'm on Ubuntu, no RPMs.

Userspace tool obtained from project website: na

*Additional Instructions for Eric <email address hidden> Brian <email address hidden>:
-Post a private note with access information to the machine that the bug is occuring on.
-Attach ltrace and strace of userspace application.

== Comment: #1 - TYREL N. DATWYLER <email address hidden> - 2016-02-17 17:46:17 ==
Fix has been submitted upstream:

https://lkml.org/lkml/2016/2/10/792

== Comment: #2 - TYREL N. DATWYLER <email address hidden> - 2016-02-18 12:19:18 ==
Fix has been accepted upstream into Martin Petersen's 4.6/scsi-queue branch.

http://git.kernel.org/cgit/linux/kernel/git/mkp/scsi.git/commit/?h=4.6/scsi-queue&id=f9e3de0d6053ae89828188291a405c64d8d95a0e

[Ubuntu Foundations Team Bug Bot (crichton)]

Thank you for taking the time to report this bug and helping to make Ubuntu better. It seems that your bug report is not filed about a specific source package though, rather it is just filed against Ubuntu in general. It is important that bug reports be filed about source packages so that people interested in the package can find the bugs about it. You can find some hints about determining what package your bug might be about at https://wiki.ubuntu.com/Bugs/FindRightPackage. You might also ask for help in the #ubuntu-bugs irc channel on Freenode.

To change the source package that this bug is filed about visit https://bugs.launchpad.net/ubuntu/+bug/1547153/+editstatus and add the package name in the text box next to the word Package.

[This is an automated message. I apologize if it reached you inappropriately; please just reply to this message indicating so.]

[Launchpad Janitor (janitor)]

This bug was fixed in the package linux - 4.4.0-8.23

---------------
linux (4.4.0-8.23) xenial; urgency=low

  * cgroup namespace mounts broken in containers (LP: #1549398)
    - SAUCE: kernfs: Always set super block owner to init_user_ns

  * 4.4.0-7.22 no longer boots on arm64 (LP: #1547718)
    - arm64: mm: avoid calling apply_to_page_range on empty range
    - UBUNTU SAUCE: arm: mm: avoid calling apply_to_page_range on empty range

  * kernel install failed /bin/cp: cannot stat ‘/boot/initrd.img-4.3.0-7-generic’: No such file or directory (LP: #1536810)
    - [Config] postinst -- handle recreating symlinks when a real file is present

  * insecure overlayfs xattrs handling in copy_up (LP: #1534961)
    - SAUCE: cred: Add clone_cred() interface
    - SAUCE: overlayfs: Use mounter's credentials instead of selectively raising caps
    - SAUCE: overlayfs: Skip permission checking for trusted.overlayfs.* xattrs
    - SAUCE: overlayfs: Be more careful about copying up sxid files
    - SAUCE: overlayfs: Propogate nosuid from lower and upper mounts

  * overlayfs over fuse should refuse copy_up of files if uid/gid not mapped (LP: #1535150)
    - SAUCE: cred: Add clone_cred() interface
    - SAUCE: overlayfs: Use mounter's credentials instead of selectively raising caps
    - SAUCE: overlayfs: Skip permission checking for trusted.overlayfs.* xattrs
    - SAUCE: overlayfs: Be more careful about copying up sxid files
    - SAUCE: overlayfs: Propogate nosuid from lower and upper mounts

  * overlay: mkdir fails if directory exists in lowerdir in a user namespace (LP: #1531747)
    - SAUCE: cred: Add clone_cred() interface
    - SAUCE: overlayfs: Use mounter's credentials instead of selectively raising caps
    - SAUCE: overlayfs: Skip permission checking for trusted.overlayfs.* xattrs

  * Update Intel ethernet drivers to Fortville SW5 (LP: #1547674)
    - net: bulk free infrastructure for NAPI context, use napi_consume_skb
    - net: Add eth_platform_get_mac_address() helper.
    - i40e: Add mac_filter_element at the end of the list instead of HEAD
    - i40e/i40evf: Fix RSS rx-flow-hash configuration through ethtool
    - i40e: Replace X722 mac check in ethtool get_settings
    - i40evf: allow channel bonding of VFs
    - i40e: define function capabilities in only one place
    - i40evf: null out ring pointers on free
    - i40e: Cleanup the code with respect to restarting autoneg
    - i40e: update features with right offload
    - i40e: bump version to 1.4.10
    - i40e: add new device IDs for X722
    - i40e: Extend ethtool RSS hooks for X722
    - i40e/i40evf: Fix for UDP/TCP RSS for X722
    - i40evf: add new write-back mode
    - i40e/i40evf: Use private workqueue
    - i40e: add new proxy-wol bit for X722
    - i40e: Limit DCB FW version checks to X710/XL710 devices
    - i40e: AQ Add Run PHY Activity struct
    - i40e: AQ Geneve cloud tunnel type
    - i40e: AQ Add external power class to get link status
    - i40e: add 100Mb ethtool reporting
    - ixgbe: bulk free SKBs during TX completion cleanup cycle
    - igb: Remove unnecessary flag setting in igb_set_flag_queue_pairs()
    - igb: Unpair the queues when changing the number of queues...

[bugproxy (bugproxy)]

------- Comment From <email address hidden> 2016-06-24 16:49 EDT-------
Works as expected with 4.4.0-21-generic.