[Bug 1705582] Comment bridged from LTC Bugzilla

[bugproxy]

------- Comment From kumar.ankit008 at in.ibm.com 2017-08-24 00:58 EDT-------
(In reply to comment #10)
> Ankit, the package is now uploading and waiting for review.  Please complete
> the package description how to test this update

This patch can be tested on system where,
Number of scsi device node (/dev/sg<X>) are more than total number of device node that can be opened at a time.(using open call)

The reason being is : In device_close call earlier we were just doing
unlink of opened device file (/tmp/nodexyz) without closing descriptor
which eventually will result in sort of file descriptor and database
creation will failed as we won't have free descriptor to be given for
creating database.

This issue can be observed only on system where scsi device node is more
than the number of device file can be opened at a time on system.

Procedure to test:
1. run vpdupdate (on system with above details)

and vpdupdate will fail.

~Ankit

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to lsvpd in Ubuntu.
https://bugs.launchpad.net/bugs/1705582

Title:
  Update Package request for lsvpd

Status in The Ubuntu-power-systems project:
  Incomplete
Status in lsvpd package in Ubuntu:
  Fix Released
Status in lsvpd source package in Xenial:
  In Progress

Bug description:
  SRU
  <SRU test description missing>

  ---Problem Description---
  Update Package request for lsvpd

  ---uname output---
  Linux tuleta4u-lp9 4.10.0-27-generic #30~16.04.2-Ubuntu SMP Thu Jun 29 16:06:52 UTC 2017 ppc64le ppc64le ppc64le GNU/Linux

  ---Steps to Reproduce---
   lsvpd

  Userspace tool common name: lsvpd

  Please pull below patches for lsvpd package.

  ------------------
  commit 7499dcc199da96befef7bb788d62e446833137b2
  Author: Ankit Kumar <ankit at linux.vnet.ibm.com>
  Date:   Mon Dec 7 11:59:05 2015 +0530

      lsvpd: Unique name for temp Device file given and file closed
  before deleting it.

      This piece of code used mkstemp to get unique name of file in that particular
      directory. As file name is not hardcoded, so parameter for device_close and
      device_open is changed to pass device location path.

      This patch also fixes the issue of opened file which is not closed by closing
      file descriptor.

      Signed-off-by: Ankit Kumar <ankit at linux.vnet.ibm.com>
      [Removed redundant variable - Vasant]
      Signed-off-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>

  -----

  Backported patch on top of 1.7.6 provided

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-power-systems/+bug/1705582/+subscriptions