Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allocate executable storage properly #507

Open
wants to merge 1 commit into
base: v3.x/staging
Choose a base branch
from

Conversation

ifakhrutdinov
Copy link
Contributor

@ifakhrutdinov ifakhrutdinov commented Dec 12, 2024

Proposed changes

Some storage that is intended to be executed is not allocated with the EXECUTABLE=YES option, can lead to ABENDs when certain z/OS options are set.

This PR changes the code which allocates such storage to allocate it with EXECUTABLE=YES.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

PR Checklist

Please delete options that are not relevant.

  • If the changes in this PR are meant for the next release / mainline, this PR targets the "staging" branch.
  • My code follows the style guidelines of this project (see: Contributing guideline)
  • I have commented my code, particularly in hard-to-understand areas
  • Relevant update to CHANGELOG.md
  • My changes generate no new warnings

Testing

You will need a z/OS system where the special diagnostics for non-executable storage has been enabled.

Do the following:

  • Regression testing of ZIS;
  • Cancel a ZIS instance to trigger the address space resource manager; no ABENDs should occur.

@ifakhrutdinov ifakhrutdinov force-pushed the bugfix/non-exec-storage branch from d5019a1 to e62347e Compare December 12, 2024 12:35
@ifakhrutdinov ifakhrutdinov self-assigned this Dec 12, 2024
@ifakhrutdinov ifakhrutdinov marked this pull request as ready for review December 12, 2024 12:43
@ifakhrutdinov ifakhrutdinov added enhancement New feature or request bug Something isn't working labels Dec 13, 2024
Some storage that is intended to be executed is not allocated with the
EXECUTABLE=YES option, can lead to ABENDs when certain z/OS options are
set.

This commit changes the code which allocates such storage to allocate it
with EXECUTABLE=YES.

Signed-off-by: Irek Fakhrutdinov <[email protected]>
@ifakhrutdinov ifakhrutdinov force-pushed the bugfix/non-exec-storage branch from e62347e to 2a06c28 Compare December 20, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant