Turning COMPAT_32BIT_TIME off on riscv64

Bug #2038584 reported by Dimitri John Ledkov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Undecided
Unassigned
linux-allwinner (Ubuntu)
New
Undecided
Unassigned
linux-riscv (Ubuntu)
Fix Released
Undecided
Unassigned
linux-starfive (Ubuntu)
New
Undecided
Unassigned

Bug Description

This will prevent (?!) binaries to operate correctly, if they are still using time32.... but maybe riscv32 ABI uses 64-bit time_t anyway?

24.04 LTS is likely to be used for 10 years. And if allowed to overrun and remain active in the field in 2038 can lead to catastrophic failure in the field due to these syscalls enabled and used.

I would like to request if we can turn off COMPAT_32BIT_TIME on every architecture, thus this will be arch by arch bug report, and arch by arch decision.

This needs to be a per-arch decision, potentially taking into consideration bi-arch userspace support.

config COMPAT_32BIT_TIME
 bool "Provide system calls for 32-bit time_t"
 default !64BIT || COMPAT
 help
   This enables 32 bit time_t support in addition to 64 bit time_t support.
   This is relevant on all 32-bit architectures, and 64-bit architectures
   as part of compat syscall handling.

Revision history for this message
Ubuntu Kernel Bot (ubuntu-kernel-bot) wrote : Missing required logs.

This bug is missing log files that will aid in diagnosing the problem. While running an Ubuntu kernel (not a mainline or third-party kernel) please enter the following command in a terminal window:

apport-collect 2038584

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu):
status: New → Incomplete
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Consensus from maintainers is that it's a mistake that this option is on for riscv64, especially since compat is off.

Revision history for this message
Dimitri John Ledkov (xnox) wrote :
Changed in linux (Ubuntu):
status: Incomplete → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package linux - 6.8.0-11.11

---------------
linux (6.8.0-11.11) noble; urgency=medium

  * noble/linux: 6.8.0-11.11 -proposed tracker (LP: #2053094)

  * Miscellaneous Ubuntu changes
    - [Packaging] riscv64: disable building unnecessary binary debs

 -- Paolo Pisati <email address hidden> Wed, 14 Feb 2024 00:04:31 +0100

Changed in linux (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (6.1 KiB)

This bug was fixed in the package linux-riscv - 6.8.0-20.20.1

---------------
linux-riscv (6.8.0-20.20.1) noble; urgency=medium

  * noble/linux-riscv: 6.8.0-20.20.1 -proposed tracker (LP: #2058921)

  * Packaging resync (LP: #1786013)
    - debian.riscv/dkms-versions -- update from kernel-versions (main/d2024.02.29)

  * Enable StarFive VisionFive 2 board (LP: #2013232)
    - dt-bindings: pwm: Add bindings for OpenCores PWM Controller
    - riscv: dts: starfive: jh7110: Add PWM node and pins configuration
    - SAUCE: pwm: opencores: Add PWM driver support
    - SAUCE: dt-bindings: PCI: Add PLDA XpressRICH PCIe host common properties
    - SAUCE: PCI: microchip: Move pcie-microchip-host.c to plda directory
    - SAUCE: PCI: microchip: Move PLDA IP register macros to pcie-plda.h
    - SAUCE: PCI: microchip: Add bridge_addr field to struct mc_pcie
    - SAUCE: PCI: microchip: Rename two PCIe data structures
    - SAUCE: PCI: microchip: Move PCIe host data structures to plda-pcie.h
    - SAUCE: PCI: microchip: Rename two setup functions
    - SAUCE: PCI: microchip: Change the argument of plda_pcie_setup_iomems()
    - SAUCE: PCI: microchip: Move setup functions to pcie-plda-host.c
    - SAUCE: PCI: microchip: Rename interrupt related functions
    - SAUCE: PCI: microchip: Add num_events field to struct plda_pcie_rp
    - SAUCE: PCI: microchip: Add request_event_irq() callback function
    - SAUCE: PCI: microchip: Add INTx and MSI event num to struct plda_event
    - SAUCE: PCI: microchip: Add get_events() callback and add PLDA get_event()
    - SAUCE: PCI: microchip: Add event irqchip field to host port and add PLDA
      irqchip
    - SAUCE: PCI: microchip: Move IRQ functions to pcie-plda-host.c
    - SAUCE: PCI: plda: Add event bitmap field to struct plda_pcie_rp
    - SAUCE: PCI: plda: Add host init/deinit and map bus functions
    - SAUCE: dt-bindings: PCI: Add StarFive JH7110 PCIe controller
    - SAUCE: PCI: Add PCIE_RESET_CONFIG_DEVICE_WAIT_MS waiting time value
    - SAUCE: PCI: starfive: Add JH7110 PCIe controller
    - SAUCE: PCI: starfive: Offload the NVMe timeout workaround to host drivers.
    - SAUCE: riscv: dts: starfive: add PCIe dts configuration for JH7110
    - SAUCE: clk: starfive: jh7110-sys: Fix lower rate of CPUfreq by setting PLL0
      rate to 1.5GHz
    - SAUCE: [HACK] riscv: dts: starfive: Add VisionFive 2 reserved memory node
    - SAUCE: riscv: dts: starfive: Disable JH7110 crypto peripheral
    - [Config] riscv: updateconfigs for VisionFive 2 patches

  * Enable Nezha board (LP: #1975592)
    - [Config] Enable CONFIG_REGULATOR_FIXED_VOLTAGE on riscv64
    - SAUCE: dt-bindings: pwm: Add binding for Allwinner D1/T113-S3/R329 PWM
      controller
    - SAUCE: pwm: Add Allwinner's D1/T113-S3/R329 SoCs PWM support
    - SAUCE: riscv: dts: allwinner: d1: Add pwm node
    - SAUCE: riscv: dts: allwinner: d1: Add PWM pins
    - SAUCE: mmc: sunxi-mmc: Correct the maximum segment size
    - SAUCE: riscv: dts: allwinner: d1: Add misc nodes
    - SAUCE: riscv: dts: allwinner: Keep aldo regulator on
    - SAUCE: riscv: dts: allwinner: Add button on the Nezha board
    - SAUCE: riscv: dts: allwinner: Add button on the Lichee RV ...

Read more...

Changed in linux-riscv (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.