Bug#853122: [Regression] Changes to i2c-piix4.c initialisation prevent loading of sp5100_tco watchdog driver on AMD SB800 chipset

2017-03-03 Thread Paul Menzel
Dear Linux folks,


Unfortunately, commit 2fee61d22e (i2c: piix4: Add support for
multiplexed main adapter in SB800) [1] caused a regression. Tim
reported that to the Linux Kernel Bugtracker as bug #170741 last
September [2], but it looks like the affected subsystems don’t use it.

So I just copy his report in here, and put all the people in CC
mentioned in the commit, the bug report, and in the subsystems
I2C/SMBUS CONTROLLER DRIVERS FOR PC and WATCHDOG DEVICE DRIVERS
mentioned in the file `MAINTAINERS`.

> On the AMD Turion N40L and other related SoCs, the i2c-piix4 driver
> now claims the 0xcd6 ioport, preventing the sp5100_tco watchdog
> driver from loading:
> 
> piix4_smbus :00:14.0: SMBus Host Controller at 0xb00, revision 0
> piix4_smbus :00:14.0: Using register 0x2c for SMBus port selection
> piix4_smbus :00:14.0: Auxiliary SMBus Host Controller at 0xb20
> sp5100_tco: SP5100/SB800 TCO WatchDog Timer Driver v0.05
> sp5100_tco: PCI Vendor ID: 0x1002, Device ID: 0x4385, Revision ID: 0x42
> sp5100_tco: I/O address 0x0cd6 already in use
> 
> 
> This breaks watchdog operation on existing systems on upgrade and new
> deployments unless the i2c-piix4 driver is blacklisted.
> 
> See:
> 
> drivers/watchdog/sp5100_tco.c
> 
> tco_timer_enable(void)
> 
> (SB800_IO_PM_INDEX_REG is defined in drivers/watchdog/sp5100_tco.h)
> 
> This is the commit which prevents the watchdog driver from loading:
> 
> http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=2fee61d22e606fc99ade9079fda15fdee83ec33e
> 
> See also AMD docs
> 
> 45483_sb800_bdg_pub_3.03
> 
> Perhaps a fix is to switch both drivers from static to dynamic
> allocation of the IO ports in question, since the watchdog driver
> only accesses the port during initialisation (with backoff/retry
> maybe to avoid races?).

Is there somebody having the resources to implement the dynamic
allocation to solve this regression?


Thanks,

Paul


[1] 
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=2fee61d22e606fc99ade9079fda15fdee83ec33e
[2] https://bugzilla.kernel.org/show_bug.cgi?id=170741

signature.asc
Description: This is a digitally signed message part


Bug#853122: [Regression] Changes to i2c-piix4.c initialisation prevent loading of sp5100_tco watchdog driver on AMD SB800 chipset

2017-03-03 Thread Wolfram Sang

> Unfortunately, commit 2fee61d22e (i2c: piix4: Add support for
> multiplexed main adapter in SB800) [1] caused a regression. Tim
> reported that to the Linux Kernel Bugtracker as bug #170741 last
> September [2], but it looks like the affected subsystems don’t use it.

Jean Delvare pointed out this issue amongst others[1] last year already.
Let me quote:

===

5* The I/O ports used for SMBus configuration and port switching are
also needed by a watchdog driver, sp5100_tco. Both drivers request the
region, so the first one wins, and the other driver can't be loaded.
sp5100_tco was there first, so the changes done to the i2c-piix4 driver
recently will cause a regression for some users by preventing them
from using the sp5100_tco and i2c-piix4 drivers at the same time. In
the long run I guess we will need a helper module to handle this shared
resource. Unless IORESOURCE_MUXED can be used for that. Either way,
that's more work than I can put into this before kernel v4.5 is
released. For the time being, I think we should simply make it
non-fatal if the I/O ports can't be requested, and continue without
multiplexing (as before.)

===

Seems nobody had the resources, so far. I don't have the HW and not much
experience with non-embedded platforms. I wonder, though, if we really
need to convert the drivers to MFD ones, or if we could use the simpler
MFD_SYSCON mechanism which helps in exactly such cases for embedded
platforms. But I am really lacking details here and am afraid this is
probably all the input I can give currently.

Regards,

   Wolfram

[1] http://www.spinics.net/lists/linux-i2c/msg23437.html



signature.asc
Description: PGP signature


Re: linux_4.9.13-1~bpo8+1_amd64.changes is NEW

2017-03-03 Thread Harald Dunkel
> 
> If you have any questions, you may reply to this email.
> 

Is there a new linux-latest for bpo8 as well?

Regards
Harri



Processed: Re: module hpsa no longer detects MSL2024 tape changer

2017-03-03 Thread Debian Bug Tracking System
Processing control commands:

> fixed -1 4.9.2-2~bpo8+1
Bug #767389 [src:linux] linux-image-3.16-0.bpo.2-amd64 module hpsa does not 
recognize /dev/changer
Marked as fixed in versions linux/4.9.2-2~bpo8+1.

-- 
767389: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=767389
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#767389: module hpsa no longer detects MSL2024 tape changer

2017-03-03 Thread Sergio Gelato
control: fixed -1 4.9.2-2~bpo8+1

I confirm that the latest kernel in jessie-backports does not suffer from
this problem.

Am building a patched 3.16.39 (it compiles OK) but won't be able to test it
until the next maintenance window for the affected system.



Re: linux_4.9.13-1~bpo8+1_amd64.changes is NEW

2017-03-03 Thread Ben Hutchings
On Fri, 2017-03-03 at 13:17 +0100, Harald Dunkel wrote:
> > 
> > If you have any questions, you may reply to this email.
> > 
> 
> Is there a new linux-latest for bpo8 as well?

linux is still in NEW for all other architectures, and linux-latest and
linux-signed are waiting for that.

Ben.

-- 
Ben Hutchings
Hoare's Law of Large Problems:
Inside every large problem is a small problem struggling to get
out.



signature.asc
Description: This is a digitally signed message part


Re: linux_4.9.13-1~bpo8+1_amd64.changes is NEW

2017-03-03 Thread Ben Hutchings
On Fri, 2017-03-03 at 15:54 +, Ben Hutchings wrote:
> On Fri, 2017-03-03 at 13:17 +0100, Harald Dunkel wrote:
> > > 
> > > If you have any questions, you may reply to this email.
> > > 
> > 
> > Is there a new linux-latest for bpo8 as well?
> 
> linux is still in NEW for all other architectures, and linux-latest and
> linux-signed are waiting for that.

Actually, it just got accepted, so the other two can be uploaded.

Ben.

-- 
Ben Hutchings
Hoare's Law of Large Problems:
Inside every large problem is a small problem struggling to get
out.



signature.asc
Description: This is a digitally signed message part


Processing of linux-signed_4.3~bpo8+1_multi.changes

2017-03-03 Thread Debian FTP Masters
linux-signed_4.3~bpo8+1_multi.changes uploaded successfully to localhost
along with the files:
  linux-signed_4.3~bpo8+1.dsc
  linux-signed_4.3~bpo8+1.tar.xz
  linux-signed_4.3~bpo8+1_source.buildinfo
  linux-image-4.9.0-0.bpo.2-amd64_4.9.13-1~bpo8+1_amd64.deb
  linux-image-4.9.0-0.bpo.2-rt-amd64_4.9.13-1~bpo8+1_amd64.deb
  kernel-image-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  nic-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  nic-wireless-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  nic-shared-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  serial-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  usb-serial-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  ppp-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  pata-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  cdrom-core-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  firewire-core-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  scsi-core-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  scsi-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  loop-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  btrfs-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  ext4-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  isofs-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  jfs-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  ntfs-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  xfs-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  fat-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  md-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  multipath-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  usb-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  usb-storage-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  pcmcia-storage-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  fb-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  input-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  event-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  mouse-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  nic-pcmcia-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  pcmcia-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  nic-usb-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  sata-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  acpi-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  i2c-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  crc-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  crypto-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  crypto-dm-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  efi-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  ata-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  mmc-core-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  mmc-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  nbd-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  squashfs-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  speakup-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  virtio-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  uinput-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  sound-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  hyperv-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  udf-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb
  fuse-modules-4.9.0-0.bpo.2-amd64-di_4.9.13-1~bpo8+1_amd64.udeb

Greetings,

Your Debian queue daemon (running on host usper.debian.org)



linux-signed_4.3~bpo8+1_multi.changes is NEW

2017-03-03 Thread Debian FTP Masters
binary:acpi-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:acpi-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:acpi-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:affs-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:affs-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:ata-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:ata-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:ata-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:ata-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:ata-modules-4.9.0-0.bpo.2-armmp-di is NEW.
binary:ata-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:ata-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:ata-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:btrfs-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:btrfs-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:btrfs-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:btrfs-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:btrfs-modules-4.9.0-0.bpo.2-armmp-di is NEW.
binary:btrfs-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:btrfs-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:btrfs-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:btrfs-modules-4.9.0-0.bpo.2-s390x-di is NEW.
binary:cdrom-core-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:cdrom-core-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:cdrom-core-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:cdrom-core-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:cdrom-core-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:cdrom-core-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:cdrom-core-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:crc-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:crc-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:crc-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:crc-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:crc-modules-4.9.0-0.bpo.2-armmp-di is NEW.
binary:crc-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:crc-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:crc-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:crc-modules-4.9.0-0.bpo.2-s390x-di is NEW.
binary:crypto-dm-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:crypto-dm-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:crypto-dm-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:crypto-dm-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:crypto-dm-modules-4.9.0-0.bpo.2-armmp-di is NEW.
binary:crypto-dm-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:crypto-dm-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:crypto-dm-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:crypto-dm-modules-4.9.0-0.bpo.2-s390x-di is NEW.
binary:crypto-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:crypto-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:crypto-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:crypto-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:crypto-modules-4.9.0-0.bpo.2-armmp-di is NEW.
binary:crypto-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:crypto-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:crypto-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:crypto-modules-4.9.0-0.bpo.2-s390x-di is NEW.
binary:dasd-extra-modules-4.9.0-0.bpo.2-s390x-di is NEW.
binary:dasd-modules-4.9.0-0.bpo.2-s390x-di is NEW.
binary:efi-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:efi-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:efi-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:efi-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:efi-modules-4.9.0-0.bpo.2-armmp-di is NEW.
binary:event-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:event-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:event-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:event-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:event-modules-4.9.0-0.bpo.2-armmp-di is NEW.
binary:event-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:event-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:event-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:ext4-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:ext4-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:ext4-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:ext4-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:ext4-modules-4.9.0-0.bpo.2-armmp-di is NEW.
binary:ext4-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:ext4-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:ext4-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:ext4-modules-4.9.0-0.bpo.2-s390x-di is NEW.
binary:fancontrol-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:fancontrol-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:fat-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:fat-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:fat-modules-4.9.0-0.bpo.2-amd64-di is NEW.
binary:fat-modules-4.9.0-0.bpo.2-arm64-di is NEW.
binary:fat-modules-4.9.0-0.bpo.2-armmp-di is NEW.
binary:fat-modules-4.9.0-0.bpo.2-powerpc-di is NEW.
binary:fat-modules-4.9.0-0.bpo.2-powerpc64-di is NEW.
binary:fat-modules-4.9.0-0.bpo.2-powerpc64le-di is NEW.
binary:fat-modules-4.9.0-0.bpo.2-s390x-di is NEW.
binary:fb-modules-4.9.0-0.bpo.2-686-di is NEW.
binary:fb-modules-4.9.0-0.bpo.2-686-pae-di is NEW.
binary:fb-modules-4.9.0-0.b

Processed: Re: Bug#856474: stap: include runtime_defines.h not found

2017-03-03 Thread Debian Bug Tracking System
Processing control commands:

> reassign -1 linux
Bug #856474 [systemtap] stap: include runtime_defines.h not found
Bug reassigned from package 'systemtap' to 'linux'.
No longer marked as found in versions systemtap/3.0-7.
Ignoring request to alter fixed versions of bug #856474 to the same values 
previously set

-- 
856474: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=856474
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Processed: reassign 856474 to src:linux, severity of 856474 is important, affects 856474

2017-03-03 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> reassign 856474 src:linux 4.9.13-1
Bug #856474 [linux] stap: include runtime_defines.h not found
Bug reassigned from package 'linux' to 'src:linux'.
Ignoring request to alter found versions of bug #856474 to the same values 
previously set
Ignoring request to alter fixed versions of bug #856474 to the same values 
previously set
Bug #856474 [src:linux] stap: include runtime_defines.h not found
Marked as found in versions linux/4.9.13-1.
> severity 856474 important
Bug #856474 [src:linux] stap: include runtime_defines.h not found
Severity set to 'important' from 'grave'
> affects 856474 systemtap
Bug #856474 [src:linux] stap: include runtime_defines.h not found
Added indication that 856474 affects systemtap
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
856474: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=856474
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#856474: stap: include runtime_defines.h not found

2017-03-03 Thread Gerald Turner
I modified systemtap buildrun.cxx so that the Makefile it produces
during Pass 3 doesn't have quotes around the include path, i.e.:

Before:

  EXTRA_CFLAGS += -I"/usr/share/systemtap/runtime"

After:

  EXTRA_CFLAGS += -I/usr/share/systemtap/runtime

Then stap works fine against the Debian kernel, running 4.10-1~exp1 at
the moment.

I cannot fathom how Debian's linux-headers packages would affect this
argument, but for the record, here's systemtap running gcc in Pass 3,
elided:

  gcc-6 ... 
-I/usr/src/linux-headers-4.10.0-trunk-common/"/usr/share/systemtap/runtime"   
...

And with buildrun.cxx modified so that quotations are removed from the
Makefile:

  gcc-6 ... -I/usr/share/systemtap/runtime ...

How is /usr/src/linux-headers-4.10.0-trunk-common/ getting interpolated
into that path argument?  Attached a Makefile produced by stap - maybe
somebody could guess why that interpolation occurs at line 125?

Apologies if I'm adding more noise than signal to the bug report.  Frank
Ch. Eigler already addressed the quotation issue earlier in this bug
report.

-- 
Gerald Turner Encrypted mail preferred!
OpenPGP: 4096R / CA89 B27A 30FA 66C5 1B80  3858 EC94 2276 FDB8 716D
--- systemtap-3.1.orig/buildrun.cxx
+++ systemtap-3.1/buildrun.cxx
@@ -495,7 +495,7 @@ compile_pass (systemtap_session& s)
   #if CHECK_POINTER_ARITH_PR5947
   o << "EXTRA_CFLAGS += -Wpointer-arith" << endl;
   #endif
-  o << "EXTRA_CFLAGS += -I\"" << s.runtime_path << "\"" << endl;
+  o << "EXTRA_CFLAGS += -I" << s.runtime_path << endl;
   // XXX: this may help ppc toc overflow
   // o << "CFLAGS := $(subst -Os,-O2,$(CFLAGS)) -fminimal-toc" << endl;
   o << "obj-m := " << s.module_name << ".o" << endl;
_KBUILD_CFLAGS := $(call flags,KBUILD_CFLAGS)
stap_check_gcc = $(shell set -x; if $(CC) $(1) -S -o /dev/null -xc /dev/null > 
/dev/null 2>&1; then echo "$(1)"; else echo "$(2)"; fi)
CHECK_BUILD := $(CC) $(NOSTDINC_FLAGS) $(KBUILD_CPPFLAGS) $(CPPFLAGS) 
$(LINUXINCLUDE) $(_KBUILD_CFLAGS) $(CFLAGS_KERNEL) $(EXTRA_CFLAGS) $(CFLAGS) 
-DKBUILD_BASENAME=\"stap_1008\" -Werror -S -o /dev/null -xc 
stap_check_build = $(shell set -x; if $(CHECK_BUILD) $(1) > /dev/null 2>&1 ; 
then echo "$(2)"; else echo "$(3)"; fi)
SYSTEMTAP_RUNTIME = "/usr/share/systemtap/runtime"
CONFIG_MODULE_SIG := n
EXTRA_CFLAGS :=
EXTRA_CFLAGS += -Iinclude2/asm/mach-default
EXTRA_CFLAGS += -I/lib/modules/4.10.0-trunk-amd64/build
STAPCONF_HEADER := 
/tmp/stapmZGYID/stapconf_0c48b85f89be3e4d6b5ccca7b814cb38_755.h
$(STAPCONF_HEADER):
@> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-hrtimer-rel.c > 
/dev/null 2>&1; then echo "#define STAPCONF_HRTIMER_REL 1"; fi >> $@
>> $@
if $(CHECK_BUILD) 
$(SYSTEMTAP_RUNTIME)/linux/autoconf-generated-compile.c > /dev/null 2>&1; then 
echo "#define STAPCONF_GENERATED_COMPILE 1"; fi >> $@
if $(CHECK_BUILD) 
$(SYSTEMTAP_RUNTIME)/linux/autoconf-hrtimer-getset-expires.c > /dev/null 2>&1; 
then echo "#define STAPCONF_HRTIMER_GETSET_EXPIRES 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-inode-private.c > 
/dev/null 2>&1; then echo "#define STAPCONF_INODE_PRIVATE 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-inode-rwsem.c > 
/dev/null 2>&1; then echo "#define STAPCONF_INODE_RWSEM 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-constant-tsc.c > 
/dev/null 2>&1; then echo "#define STAPCONF_CONSTANT_TSC 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-ktime-get-real.c 
> /dev/null 2>&1; then echo "#define STAPCONF_KTIME_GET_REAL 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-x86-uniregs.c > 
/dev/null 2>&1; then echo "#define STAPCONF_X86_UNIREGS 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-nameidata.c > 
/dev/null 2>&1; then echo "#define STAPCONF_NAMEIDATA_CLEANUP 1"; fi >> $@
echo "#define STAPCONF_UNREGISTER_KPROBES 1">> $@
if $(CHECK_BUILD) 
$(SYSTEMTAP_RUNTIME)/linux/autoconf-kprobe-symbol-name.c > /dev/null 2>&1; then 
echo "#define STAPCONF_KPROBE_SYMBOL_NAME 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-real-parent.c > 
/dev/null 2>&1; then echo "#define STAPCONF_REAL_PARENT 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-uaccess.c > 
/dev/null 2>&1; then echo "#define STAPCONF_LINUX_UACCESS_H 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-oneachcpu-retry.c 
> /dev/null 2>&1; then echo "#define STAPCONF_ONEACHCPU_RETRY 1"; fi >> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-dpath-path.c > 
/dev/null 2>&1; then echo "#define STAPCONF_DPATH_PATH 1"; fi >> $@
>> $@
>> $@
echo "#define STAPCONF_SYNCHRONIZE_SCHED 1">> $@
if $(CHECK_BUILD) $(SYSTEMTAP_RUNTIME)/linux/autoconf-task-uid.c > 
/dev/null 2>&1; then echo "#define STAPCONF_TASK_UID 1"; fi >> $@
if $(CHECK_BUILD) 
$(SYSTEMTA

Processed: Re: initramfs-tools: mkinitramfs fails to run on systems with tmpfs on /

2017-03-03 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> tags 856589 + patch
Bug #856589 [initramfs-tools] initramfs-tools: mkinitramfs fails to run on 
systems with tmpfs on /
Added tag(s) patch.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
856589: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=856589
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#856589: initramfs-tools: mkinitramfs fails to run on systems with tmpfs on /

2017-03-03 Thread Benjamin Drung
tags 856589 + patch
thanks

The solution is quite simple: Just return from dep_add_modules_mount()
when tmpfs is detected as fstype. A tested patch is attached.

-- 
Benjamin Drung
System Developer
Debian & Ubuntu Developer

ProfitBricks GmbH
Greifswalder Str. 207
D - 10405 Berlin

Email: benjamin.dr...@profitbricks.com
URL:  http://www.profitbricks.com

Sitz der Gesellschaft: Berlin.
Registergericht: Amtsgericht Charlottenburg, HRB 125506B.
Geschäftsführer: Andreas Gauger, Achim Weiss.From 7b6c9871e04e4d2304b99b14fe313d7b4ab35698 Mon Sep 17 00:00:00 2001
From: Benjamin Drung 
Date: Fri, 3 Mar 2017 17:08:38 +0100
Subject: [PATCH] Fix failure when / uses tmpfs

When initramfs-tools is run on a system with a tmpfs based root mount
point, it will fail:

  root@host:~$ /etc/kernel/postinst.d/kdump-tools $(uname -r)
  kdump-tools: Generating /var/lib/kdump/initrd.img-4.4.36-2
  mkinitramfs: failed to determine device for /
  mkinitramfs: workaround is MODULES=most, check:
  grep -r MODULES /etc/initramfs-tools/

  Error please report bug on initramfs-tools
  Include the output of 'mount' and 'cat /proc/mounts'
  update-initramfs: failed for  with 1.

Bug-Debian: #856589
---
 hook-functions | 5 +
 1 file changed, 5 insertions(+)

diff --git a/hook-functions b/hook-functions
index fa6862f..3fc6c5b 100644
--- a/hook-functions
+++ b/hook-functions
@@ -297,6 +297,11 @@ dep_add_modules_mount()
 		return
 	fi
 
+	# Handle tmpfs which uses memory instead of a block devices.
+	if [ "${FSTYPE}" = "tmpfs" ]; then
+		return
+	fi
+
 	if [ "$dir" = / ] && [ "${dev_node}" = "/dev/root" ] ; then
 		if [ -b "${dev_node}" ]; then
 			# Match it to the canonical device name by UUID
-- 
2.9.3



Bug#856474: stap: include runtime_defines.h not found

2017-03-03 Thread Ben Hutchings
Control: tag -1 upstream patch

On Fri, 03 Mar 2017 23:10:56 +0530 Ritesh Raj Sarraf  wrote:
[...]
> Dear (Linux) Kernel Team:
> 
> We currently have systemtap 3.0 in Debian testing  where this problem was
> reported.
> 
> In Experimental, I have also pushed the 3.1 release. The 3.1 release is 
> reported
> to work perfectly in our tests with the custom built kernel. The problem is 
> only
> seen with the Debian built Linux kernels.
[...]

I investigated this and found that it occurs when the kernel source and
object trees are separate (an "out-of-tree" build, not to be confused
with out-of-tree modules).  We separate them in Debian kernel header
packages to avoid duplicating source files for each flavour.

When this is the case, the compiler is called in the root of the
object tree, and the kernel build system adjusts -I options in the
compiler flags to refer to subdirectories of the source tree if
necessary.  Any directory name beginning with /, ./ or ../ is excluded
from this adjustment.

systemtap uses -I"/usr/share/systemtap/runtime", which ought to be
excluded... but make has no understanding of shell quoting, so it is
wrongly adjusted to something like
-I/usr/src/linux-headers-4.9.0-2-common/"/usr/share/systemtap/runtime"
and runtime_defines.h cannot be found.

So my fix is to delete quotes before checking for the exclusions:

--- a/scripts/Kbuild.include
+++ b/scripts/Kbuild.include
@@ -206,7 +206,7 @@ hdr-inst := -f $(srctree)/scripts/Makefile.headersinst obj
 # Prefix -I with $(srctree) if it is not an absolute path.
 # skip if -I has no parameter
 addtree = $(if $(patsubst -I%,%,$(1)), \
-$(if $(filter-out -I/% -I./% -I../%,$(1)),$(patsubst 
-I%,-I$(srctree)/%,$(1)),$(1)))
+$(if $(filter-out -I/% -I./% -I../%,$(subst $(quote),,$(subst 
$(squote),,$(1,$(patsubst -I%,-I$(srctree)/%,$(1)),$(1)))
 
 # Find all -I options and call addtree
 flags = $(foreach o,$($(1)),$(if $(filter -I%,$(o)),$(call addtree,$(o)),$(o)))
--- END ---

Ben.

-- 
Ben Hutchings
All the simple programs have been written, and all the good names
taken.


signature.asc
Description: This is a digitally signed message part


Processed: Re: Bug#856474: stap: include runtime_defines.h not found

2017-03-03 Thread Debian Bug Tracking System
Processing control commands:

> tag -1 upstream patch
Bug #856474 [src:linux] stap: include runtime_defines.h not found
Added tag(s) patch and upstream.

-- 
856474: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=856474
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#856474: stap: include runtime_defines.h not found

2017-03-03 Thread Ben Hutchings
On Sat, 2017-03-04 at 01:39 +, Ben Hutchings wrote:
[...]
> I investigated this and found that it occurs when the kernel source and
> object trees are separate (an "out-of-tree" build, not to be confused
> with out-of-tree modules).  We separate them in Debian kernel header
> packages to avoid duplicating source files for each flavour.
> 
> When this is the case, the compiler is called in the root of the
> object tree, and the kernel build system adjusts -I options in the
> compiler flags to refer to subdirectories of the source tree if
> necessary.  Any directory name beginning with /, ./ or ../ is excluded
> from this adjustment.
> 
> systemtap uses -I"/usr/share/systemtap/runtime", which ought to be
> excluded... but make has no understanding of shell quoting, so it is
> wrongly adjusted to something like
> -I/usr/src/linux-headers-4.9.0-2-common/"/usr/share/systemtap/runtime"
> and runtime_defines.h cannot be found.

By the way, kbuild has been doing this ever since it started supporting
out-of-tree builds (around Linux 2.6.0).  However, both the adjusted
and original -I options were used.  Since Linux 4.8 only the adjusted
option is used.

Ben.

-- 
Ben Hutchings
All the simple programs have been written, and all the good names
taken.


signature.asc
Description: This is a digitally signed message part


Bug#856474: stap: include runtime_defines.h not found

2017-03-03 Thread Jack Henschel
On Sat, 04 Mar 2017 01:39:58 + Ben Hutchings  wrote:
> So my fix is to delete quotes before checking for the exclusions:
Wow Ben, that was a really quick fix!

I'll test it on Monday and report back.




signature.asc
Description: OpenPGP digital signature