Re: Testing for specific PCI devices

2025-05-06 Thread Vladimir 'phcoder' Serbinenko
Patches should be in logical chunks. In your case squashing is the way to go Regards Vladimir 'phcoder' Serbinenko Le mar. 6 mai 2025, 17:36, Yair Yarom a écrit : > > I've made the changes, but I'm inexperienced with git send or the policy > here. Should I send the patches as is (several commit

Re: Testing for specific PCI devices

2025-05-06 Thread Yair Yarom
I've made the changes, but I'm inexperienced with git send or the policy here. Should I send the patches as is (several commits for the fixes above), or should I squash it, or commit per file, etc. Thanks, On Sat, 3 May 2025 at 17:01, Vladimir 'phcoder' Serbinenko < phco...@gmail.com> wrote: > G

Re: Testing for specific PCI devices

2025-05-03 Thread Vladimir 'phcoder' Serbinenko
Generally looks ok but there are few comments like using err_t as a boolean with inverted semantics is unintuitive. Check for grub_malloc return value is completely missing. Goto is discouraged. Can you send using git send to the ML so it can be reviewed properly? Le jeu. 24 avr. 2025, 15:44, Yair

Re: Testing for specific PCI devices

2025-04-24 Thread Yair Yarom
Hi, Sure, I've uploaded my changes to github: https://github.com/irush-cs/grub/tree/irush/testpci Generally, this is a 'testpci' command that accepts one or more PCI device IDs in the form ":" as arguments, or reads lines from a file given by the --file parameter, and returns true if one

Re: Testing for specific PCI devices

2025-04-23 Thread Vladimir 'phcoder' Serbinenko
Hello. Such a functionality would be beneficial for GRUB. For more details I'd have to see at least the proposed interface if not the actual patches Regards Vladimir 'phcoder' Serbinenko Le mer. 23 avr. 2025, 11:56, Yair Yarom a écrit : > Hi, > > We have a netboot environment, and for some mach