Skip to content

Add PR write permissions for sync-with-master #27

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Add PR write permissions for sync-with-master #27

wants to merge 1 commit into from
+2 −0

Conversation

prakashsurya
Copy link

No description provided.

@prakashsurya
Copy link
Author

@sdimitro I don't know how to test this.... do you?

@prakashsurya
Copy link
Author

I'm thinking this might be needed based on this document, the fact that we're getting a 403 error currently:

 + hub pull-request -b 6.0/stage -h master -m 'Merge branch '\''master'\'' into '\''6.0/stage'\'''
Error creating pull request: Forbidden (HTTP 403)

and the fact that we appear to be using read permissions:

GITHUB_TOKEN Permissions
  Contents: read
  Metadata: read

Beyond that, though, I'm sort of guessing that this might help fix the issue.

delphix-devops-bot pushed a commit that referenced this pull request Oct 9, 2025
@osandov
drgn.helpers.linux.mm: add for_each_valid_page_range() helper
22c80e2 
Now that we have the necessary SPARSEMEM and FLATMEM infrastructure, we
can add a helper for getting ranges of valid struct pages. This has a
few use cases:

* Fixing #27, a long-standing issue with for_each_page() returning
  invalid pages.
* The crash kmem -p command.
* Adding page identification to identify_address().
* Big batched reads of pages.

Returning the PFN ranges instead of individual PFNs+pages is less
convenient for simpler use cases but especially useful for the latter
two.

Signed-off-by: Omar Sandoval <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@prakashsurya