* [kvm-unit-tests PATCH 1/3] scripts/checkpatch.pl: Add empty codespell and const_structs configs
2026-04-23 14:16 [kvm-unit-tests PATCH 0/3] scripts: Configure and update tools to allow basic b4 usage Christoph Schlameuss
@ 2026-04-23 14:16 ` Christoph Schlameuss
2026-04-24 7:00 ` Thomas Huth
2026-04-23 14:16 ` [kvm-unit-tests PATCH 2/3] scripts/checkpatch.pl: Copy kernel codespell file Christoph Schlameuss
` (2 subsequent siblings)
3 siblings, 1 reply; 10+ messages in thread
From: Christoph Schlameuss @ 2026-04-23 14:16 UTC (permalink / raw)
To: kvm
Cc: Janosch Frank, Paolo Bonzini, Thomas Huth, Andrew Jones,
Nicholas Piggin, Christoph Schlameuss
Without these files each patch is flagged with these errors providing no
value and swamping actually found warnings and errors:
● xxxxxxxxxxxx: commit title
● checkpatch.pl: No typos will be found - file '.../kvm-unit-tests/scripts/spelling.txt': No such file or directory
● checkpatch.pl: No structs that should be const will be found - file '.../kvm-unit-tests/scripts/const_structs.checkpatch': No such file or directory
Fixes: 846737f068d9 ("checkpatch support")
Signed-off-by: Christoph Schlameuss <schlameuss@linux.ibm.com>
---
scripts/const_structs.checkpatch | 0
1 file changed, 0 insertions(+), 0 deletions(-)
diff --git a/scripts/const_structs.checkpatch b/scripts/const_structs.checkpatch
new file mode 100644
index 000000000000..e69de29bb2d1
--
2.53.0
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [kvm-unit-tests PATCH 1/3] scripts/checkpatch.pl: Add empty codespell and const_structs configs
2026-04-23 14:16 ` [kvm-unit-tests PATCH 1/3] scripts/checkpatch.pl: Add empty codespell and const_structs configs Christoph Schlameuss
@ 2026-04-24 7:00 ` Thomas Huth
2026-04-24 7:36 ` Christoph Schlameuss
0 siblings, 1 reply; 10+ messages in thread
From: Thomas Huth @ 2026-04-24 7:00 UTC (permalink / raw)
To: Christoph Schlameuss, kvm
Cc: Janosch Frank, Paolo Bonzini, Andrew Jones, Nicholas Piggin
On 23/04/2026 16.16, Christoph Schlameuss wrote:
> Without these files each patch is flagged with these errors providing no
> value and swamping actually found warnings and errors:
>
> ● xxxxxxxxxxxx: commit title
> ● checkpatch.pl: No typos will be found - file '.../kvm-unit-tests/scripts/spelling.txt': No such file or directory
> ● checkpatch.pl: No structs that should be const will be found - file '.../kvm-unit-tests/scripts/const_structs.checkpatch': No such file or directory
Very weird, we've got the checkpatch.pl script in the tree since a while,
but I cannot remember having seen these warnings in the past. However, I
just tried to run checkpatch.pl with a patch, and I now get them, too ...
> diff --git a/scripts/const_structs.checkpatch b/scripts/const_structs.checkpatch
> new file mode 100644
> index 000000000000..e69de29bb2d1
Maybe add "virtio_config_ops" to the file so that it does not look so empty?
Anyway:
Reviewed-by: Thomas Huth <thuth@redhat.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kvm-unit-tests PATCH 1/3] scripts/checkpatch.pl: Add empty codespell and const_structs configs
2026-04-24 7:00 ` Thomas Huth
@ 2026-04-24 7:36 ` Christoph Schlameuss
0 siblings, 0 replies; 10+ messages in thread
From: Christoph Schlameuss @ 2026-04-24 7:36 UTC (permalink / raw)
To: Thomas Huth, Christoph Schlameuss, kvm
Cc: Janosch Frank, Paolo Bonzini, Andrew Jones, Nicholas Piggin
On Fri Apr 24, 2026 at 9:00 AM CEST, Thomas Huth wrote:
> On 23/04/2026 16.16, Christoph Schlameuss wrote:
>> Without these files each patch is flagged with these errors providing no
>> value and swamping actually found warnings and errors:
>>
>> ● xxxxxxxxxxxx: commit title
>> ● checkpatch.pl: No typos will be found - file '.../kvm-unit-tests/scripts/spelling.txt': No such file or directory
>> ● checkpatch.pl: No structs that should be const will be found - file '.../kvm-unit-tests/scripts/const_structs.checkpatch': No such file or directory
>
> Very weird, we've got the checkpatch.pl script in the tree since a while,
> but I cannot remember having seen these warnings in the past. However, I
> just tried to run checkpatch.pl with a patch, and I now get them, too ...
>
Maybe you did have a git setting modifying the arguments? IMO this just should
at least somewhat work OOTB.
>> diff --git a/scripts/const_structs.checkpatch b/scripts/const_structs.checkpatch
>> new file mode 100644
>> index 000000000000..e69de29bb2d1
>
> Maybe add "virtio_config_ops" to the file so that it does not look so empty?
>
> Anyway:
> Reviewed-by: Thomas Huth <thuth@redhat.com>
I agree that it does look strange completely empty. Will add virtio_config_ops.
Thanks
^ permalink raw reply [flat|nested] 10+ messages in thread
* [kvm-unit-tests PATCH 2/3] scripts/checkpatch.pl: Copy kernel codespell file
2026-04-23 14:16 [kvm-unit-tests PATCH 0/3] scripts: Configure and update tools to allow basic b4 usage Christoph Schlameuss
2026-04-23 14:16 ` [kvm-unit-tests PATCH 1/3] scripts/checkpatch.pl: Add empty codespell and const_structs configs Christoph Schlameuss
@ 2026-04-23 14:16 ` Christoph Schlameuss
2026-04-23 14:16 ` [kvm-unit-tests PATCH 3/3] scripts/get_maintainer.pl: Ignore --nogit-chief-penguins option Christoph Schlameuss
2026-04-24 7:19 ` [kvm-unit-tests PATCH 0/3] scripts: Configure and update tools to allow basic b4 usage Thomas Huth
3 siblings, 0 replies; 10+ messages in thread
From: Christoph Schlameuss @ 2026-04-23 14:16 UTC (permalink / raw)
To: kvm
Cc: Janosch Frank, Paolo Bonzini, Thomas Huth, Andrew Jones,
Nicholas Piggin, Christoph Schlameuss
Provide a codespell config to actually let it do something.
Obviously codespell is not happy about the addition of these misspellings
here.
Signed-off-by: Christoph Schlameuss <schlameuss@linux.ibm.com>
---
scripts/spelling.txt | 1802 ++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 1802 insertions(+)
diff --git a/scripts/spelling.txt b/scripts/spelling.txt
new file mode 100644
index 000000000000..1abb6ae15890
--- /dev/null
+++ b/scripts/spelling.txt
@@ -0,0 +1,1802 @@
+# Originally from Debian's Lintian tool. Various false positives have been
+# removed, and various additions have been made as they've been discovered
+# in the kernel source.
+#
+# License: GPLv2
+#
+# The format of each line is:
+# mistake||correction
+#
+# Copied with this comment change from linux kernel 028ef9c96e96 ("Linux 7.0")
+abandonning||abandoning
+abigious||ambiguous
+abitrary||arbitrary
+abitrate||arbitrate
+abnornally||abnormally
+abnrormal||abnormal
+abord||abort
+aboslute||absolute
+abov||above
+abreviated||abbreviated
+absense||absence
+absolut||absolute
+absoulte||absolute
+acccess||access
+acceess||access
+accelaration||acceleration
+accelearion||acceleration
+acceleratoin||acceleration
+accelleration||acceleration
+accelrometer||accelerometer
+accesing||accessing
+accesnt||accent
+accessable||accessible
+accesss||access
+accidentaly||accidentally
+accidentually||accidentally
+acclerated||accelerated
+accoding||according
+accomodate||accommodate
+accomodates||accommodates
+accordign||according
+accoring||according
+accout||account
+accquire||acquire
+accquired||acquired
+accross||across
+accumalate||accumulate
+accumalator||accumulator
+acessable||accessible
+acess||access
+acessing||accessing
+achitecture||architecture
+acient||ancient
+acitions||actions
+acitve||active
+acknowldegement||acknowledgment
+acknowledgement||acknowledgment
+ackowledge||acknowledge
+ackowledged||acknowledged
+acording||according
+activete||activate
+actived||activated
+actualy||actually
+actvie||active
+acumulating||accumulating
+acumulative||accumulative
+acumulator||accumulator
+acutally||actually
+adapater||adapter
+adderted||asserted
+addional||additional
+additionaly||additionally
+additonal||additional
+addres||address
+adddress||address
+addreses||addresses
+addresss||address
+addrress||address
+aditional||additional
+aditionally||additionally
+aditionaly||additionally
+adminstrative||administrative
+adress||address
+adresses||addresses
+adrresses||addresses
+advertisment||advertisement
+adviced||advised
+afecting||affecting
+againt||against
+agaist||against
+aggreataon||aggregation
+aggreation||aggregation
+ajust||adjust
+albumns||albums
+alegorical||allegorical
+algined||aligned
+algorith||algorithm
+algorithmical||algorithmically
+algoritm||algorithm
+algoritms||algorithms
+algorithmn||algorithm
+algorrithm||algorithm
+algorritm||algorithm
+aligment||alignment
+alignement||alignment
+allign||align
+alligned||aligned
+alllocate||allocate
+alloated||allocated
+allocatote||allocate
+allocatrd||allocated
+allocte||allocate
+allocted||allocated
+allpication||application
+alocate||allocate
+alogirhtms||algorithms
+alogrithm||algorithm
+alot||a lot
+alow||allow
+alows||allows
+alreay||already
+alredy||already
+altough||although
+alue||value
+ambigious||ambiguous
+ambigous||ambiguous
+amoung||among
+amount of times||number of times
+amout||amount
+amplifer||amplifier
+amplifyer||amplifier
+an union||a union
+an user||a user
+an userspace||a userspace
+an one||a one
+analysator||analyzer
+ang||and
+anniversery||anniversary
+annoucement||announcement
+anomolies||anomalies
+anomoly||anomaly
+anonynous||anonymous
+anway||anyway
+aplication||application
+apeared||appeared
+appearence||appearance
+applicaion||application
+appliction||application
+applictions||applications
+applys||applies
+appplications||applications
+appropiate||appropriate
+appropriatly||appropriately
+approriate||appropriate
+approriately||appropriately
+apropriate||appropriate
+aquainted||acquainted
+aquired||acquired
+aquisition||acquisition
+aquires||acquires
+arbitary||arbitrary
+architechture||architecture
+archtecture||architecture
+arguement||argument
+arguements||arguments
+arithmatic||arithmetic
+aritmetic||arithmetic
+arne't||aren't
+arraival||arrival
+artifical||artificial
+artillary||artillery
+asign||assign
+asser||assert
+assertation||assertion
+assertting||asserting
+assgined||assigned
+assiged||assigned
+assigment||assignment
+assigments||assignments
+assistent||assistant
+assocaited||associated
+assocated||associated
+assocating||associating
+assocation||association
+assocative||associative
+associcated||associated
+assotiated||associated
+asssert||assert
+assum||assume
+assumtpion||assumption
+asume||assume
+asuming||assuming
+asycronous||asynchronous
+asychronous||asynchronous
+asynchnous||asynchronous
+asynchrnous||asynchronous
+asynchronus||asynchronous
+asynchromous||asynchronous
+asymetric||asymmetric
+asymmeric||asymmetric
+atleast||at least
+atomatically||automatically
+atomicly||atomically
+atempt||attempt
+atrributes||attributes
+attachement||attachment
+attatch||attach
+attched||attached
+attemp||attempt
+attemps||attempts
+attemping||attempting
+attepmpt||attempt
+attnetion||attention
+attruibutes||attributes
+authentification||authentication
+authenicated||authenticated
+automaticaly||automatically
+automaticly||automatically
+automatize||automate
+automatized||automated
+automatizes||automates
+autonymous||autonomous
+auxillary||auxiliary
+auxilliary||auxiliary
+avaiable||available
+avaialable||available
+avaible||available
+availabe||available
+availabled||available
+availablity||availability
+availaible||available
+availale||available
+availavility||availability
+availble||available
+availiable||available
+availible||available
+avalable||available
+avaliable||available
+aysnc||async
+backgroud||background
+backword||backward
+backwords||backwards
+bahavior||behavior
+bakup||backup
+baloon||balloon
+baloons||balloons
+bandwith||bandwidth
+banlance||balance
+batery||battery
+battey||battery
+beacuse||because
+becasue||because
+becomming||becoming
+becuase||because
+beeing||being
+befor||before
+begining||beginning
+beter||better
+betweeen||between
+bianries||binaries
+bitmast||bitmask
+bitwiedh||bitwidth
+boardcast||broadcast
+borad||board
+boundry||boundary
+brievely||briefly
+brigde||bridge
+broadcase||broadcast
+broadcat||broadcast
+bufer||buffer
+bufferred||buffered
+bufferur||buffer
+bufufer||buffer
+cacluated||calculated
+caculate||calculate
+caculation||calculation
+cadidate||candidate
+cahces||caches
+calcluate||calculate
+calender||calendar
+calescing||coalescing
+calibraiton||calibration
+calle||called
+callibration||calibration
+callled||called
+callser||caller
+calucate||calculate
+calulate||calculate
+cancelation||cancellation
+cancle||cancel
+cant||can't
+cant'||can't
+canot||cannot
+cann't||can't
+cannnot||cannot
+capabiity||capability
+capabilites||capabilities
+capabilties||capabilities
+capabilty||capability
+capabitilies||capabilities
+capablity||capability
+capatibilities||capabilities
+capapbilities||capabilities
+captuer||capture
+caputure||capture
+carefuly||carefully
+cariage||carriage
+casued||caused
+catagory||category
+cehck||check
+challange||challenge
+challanges||challenges
+chache||cache
+chanell||channel
+changable||changeable
+chanined||chained
+channle||channel
+channnel||channel
+charachter||character
+charachters||characters
+charactor||character
+charater||character
+charaters||characters
+charcter||character
+chcek||check
+chck||check
+checksumed||checksummed
+checksuming||checksumming
+childern||children
+childs||children
+chiled||child
+chked||checked
+chnage||change
+chnages||changes
+chnange||change
+chnnel||channel
+choosen||chosen
+chouse||chose
+circumvernt||circumvent
+claread||cleared
+clared||cleared
+clearify||clarify
+closeing||closing
+clustred||clustered
+cnfiguration||configuration
+coexistance||coexistence
+colescing||coalescing
+collapsable||collapsible
+colorfull||colorful
+comand||command
+comit||commit
+commerical||commercial
+comming||coming
+comminucation||communication
+commited||committed
+commiting||committing
+committ||commit
+commmand||command
+commnunication||communication
+commoditiy||commodity
+comsume||consume
+comsumer||consumer
+comsuming||consuming
+comaptible||compatible
+compability||compatibility
+compaibility||compatibility
+comparsion||comparison
+compatability||compatibility
+compatable||compatible
+compatibililty||compatibility
+compatibiliy||compatibility
+compatibilty||compatibility
+compatiblity||compatibility
+competion||completion
+compilant||compliant
+compleatly||completely
+completition||completion
+completly||completely
+complient||compliant
+componnents||components
+compoment||component
+comppatible||compatible
+compres||compress
+compresion||compression
+compresser||compressor
+comression||compression
+comsumed||consumed
+comunicate||communicate
+comunication||communication
+conbination||combination
+concurent||concurrent
+conditionaly||conditionally
+conditon||condition
+condtion||condition
+condtional||conditional
+conected||connected
+conector||connector
+configed||configured
+configration||configuration
+configred||configured
+configuartion||configuration
+configuation||configuration
+configued||configured
+configuratoin||configuration
+configuraton||configuration
+configuretion||configuration
+configutation||configuration
+congiuration||configuration
+conider||consider
+conjuction||conjunction
+connction||connection
+connecetd||connected
+connectinos||connections
+connetor||connector
+connnection||connection
+connnections||connections
+consistancy||consistency
+consistant||consistent
+consits||consists
+constructred||constructed
+containes||contains
+containts||contains
+contaisn||contains
+contant||contact
+contence||contents
+contiguos||contiguous
+continious||continuous
+continous||continuous
+continously||continuously
+continueing||continuing
+contiuous||continuous
+contraints||constraints
+contruct||construct
+contol||control
+contoller||controller
+controled||controlled
+controler||controller
+controll||control
+contruction||construction
+contry||country
+conuntry||country
+convertion||conversion
+convertor||converter
+convienient||convenient
+convinient||convenient
+corected||corrected
+correponding||corresponding
+correponds||corresponds
+correspoding||corresponding
+cotrol||control
+cound||could
+couter||counter
+coutner||counter
+creationg||creating
+cryptocraphic||cryptographic
+cummulative||cumulative
+cunter||counter
+curent||current
+curently||currently
+cylic||cyclic
+dafault||default
+deactive||deactivate
+deafult||default
+deamon||daemon
+debouce||debounce
+decendant||descendant
+decendants||descendants
+decompres||decompress
+decsribed||described
+decrese||decrease
+decription||description
+detault||default
+dectected||detected
+defailt||default
+deferal||deferral
+deffered||deferred
+defferred||deferred
+definate||definite
+definately||definitely
+definiation||definition
+definiton||definition
+defintion||definition
+defintions||definitions
+defualt||default
+defult||default
+deintializing||deinitializing
+deintialize||deinitialize
+deintialized||deinitialized
+deivce||device
+delared||declared
+delare||declare
+delares||declares
+delaring||declaring
+delemiter||delimiter
+deley||delay
+delibrately||deliberately
+delievered||delivered
+demodualtor||demodulator
+demension||dimension
+dependancies||dependencies
+dependancy||dependency
+dependant||dependent
+dependend||dependent
+depreacted||deprecated
+depreacte||deprecate
+desactivate||deactivate
+desciptor||descriptor
+desciptors||descriptors
+descritpor||descriptor
+descripto||descriptor
+descripton||description
+descrition||description
+descritptor||descriptor
+desctiptor||descriptor
+desriptor||descriptor
+desriptors||descriptors
+desination||destination
+destionation||destination
+destoried||destroyed
+destory||destroy
+destoryed||destroyed
+destorys||destroys
+destroied||destroyed
+detabase||database
+deteced||detected
+detecion||detection
+detectt||detect
+detroyed||destroyed
+develope||develop
+developement||development
+developped||developed
+developpement||development
+developper||developer
+developpment||development
+deveolpment||development
+devided||divided
+deviece||device
+devision||division
+diable||disable
+diabled||disabled
+dicline||decline
+dictionnary||dictionary
+didnt||didn't
+diferent||different
+differrence||difference
+diffrent||different
+differenciate||differentiate
+diffreential||differential
+diffrentiate||differentiate
+difinition||definition
+digial||digital
+dimention||dimension
+dimesions||dimensions
+diconnected||disconnected
+disabed||disabled
+disasembler||disassembler
+disble||disable
+disgest||digest
+disired||desired
+dispalying||displaying
+dissable||disable
+dissapeared||disappeared
+diplay||display
+directon||direction
+direcly||directly
+direectly||directly
+diregard||disregard
+disassocation||disassociation
+disassocative||disassociative
+disapear||disappear
+disapeared||disappeared
+disappared||disappeared
+disbale||disable
+disbaled||disabled
+disble||disable
+disbled||disabled
+disconnet||disconnect
+discontinous||discontinuous
+disharge||discharge
+disnabled||disabled
+dispertion||dispersion
+dissapears||disappears
+dissconect||disconnect
+distiction||distinction
+divisable||divisible
+divsiors||divisors
+dsiabled||disabled
+docuentation||documentation
+documantation||documentation
+documentaion||documentation
+documment||document
+doesnt||doesn't
+donwload||download
+donwloading||downloading
+dorp||drop
+dosen||doesn
+downlad||download
+downlads||downloads
+droped||dropped
+droput||dropout
+druing||during
+dyanmic||dynamic
+dynmaic||dynamic
+eanable||enable
+eanble||enable
+easilly||easily
+ecspecially||especially
+edditable||editable
+editting||editing
+efective||effective
+effectivness||effectiveness
+efficently||efficiently
+ehther||ether
+eigth||eight
+elementry||elementary
+eletronic||electronic
+embeded||embedded
+emtpy||empty
+enabledi||enabled
+enbale||enable
+enble||enable
+enchanced||enhanced
+encorporating||incorporating
+encrupted||encrypted
+encrypiton||encryption
+encryped||encrypted
+encryptio||encryption
+endianess||endianness
+enpoint||endpoint
+enhaced||enhanced
+enlightnment||enlightenment
+enqueing||enqueuing
+entires||entries
+entites||entities
+entrys||entries
+enocded||encoded
+enought||enough
+enterily||entirely
+enviroiment||environment
+enviroment||environment
+environement||environment
+environent||environment
+eqivalent||equivalent
+equiped||equipped
+equivelant||equivalent
+equivilant||equivalent
+eror||error
+errorr||error
+errror||error
+estbalishment||establishment
+etsablishment||establishment
+etsbalishment||establishment
+evalute||evaluate
+evalutes||evaluates
+evalution||evaluation
+evaulated||evaluated
+excecutable||executable
+excceed||exceed
+exceded||exceeded
+exceds||exceeds
+exceeed||exceed
+excellant||excellent
+exchnage||exchange
+execeeded||exceeded
+execeeds||exceeds
+exeed||exceed
+exeeds||exceeds
+exeuction||execution
+existance||existence
+existant||existent
+exixt||exist
+exsits||exists
+exlcude||exclude
+exlcuding||excluding
+exlcusive||exclusive
+exlusive||exclusive
+exlicitly||explicitly
+exmaple||example
+expecially||especially
+experies||expires
+explicite||explicit
+explicity||explicitly
+explicitely||explicitly
+explict||explicit
+explictely||explicitly
+explictly||explicitly
+expresion||expression
+exprienced||experienced
+exprimental||experimental
+extened||extended
+exteneded||extended
+extensability||extensibility
+extention||extension
+extenstion||extension
+extracter||extractor
+faied||failed
+faield||failed
+faild||failed
+failded||failed
+failer||failure
+faill||fail
+failied||failed
+faillure||failure
+failue||failure
+failuer||failure
+failng||failing
+faireness||fairness
+falied||failed
+faliure||failure
+fallbck||fallback
+familar||familiar
+fatser||faster
+feauture||feature
+feautures||features
+fetaure||feature
+fetaures||features
+fetcing||fetching
+fileystem||filesystem
+fimrware||firmware
+fimware||firmware
+firmare||firmware
+firmaware||firmware
+firtly||firstly
+firware||firmware
+firwmare||firmware
+finanize||finalize
+findn||find
+finilizes||finalizes
+finsih||finish
+fliter||filter
+flusing||flushing
+folloing||following
+followign||following
+followings||following
+follwing||following
+fonud||found
+forcebly||forcibly
+forseeable||foreseeable
+forse||force
+fortan||fortran
+forwardig||forwarding
+forwared||forwarded
+frambuffer||framebuffer
+framming||framing
+framwork||framework
+frequence||frequency
+frequncy||frequency
+frequancy||frequency
+frome||from
+fronend||frontend
+fucntion||function
+fuction||function
+fuctions||functions
+fullill||fulfill
+funcation||function
+funcion||function
+functionallity||functionality
+functionaly||functionally
+functionnality||functionality
+functonality||functionality
+funtion||function
+funtions||functions
+furthur||further
+futhermore||furthermore
+futrue||future
+gatable||gateable
+gateing||gating
+gauage||gauge
+gaurenteed||guaranteed
+generiously||generously
+genereate||generate
+genereted||generated
+genric||generic
+gerenal||general
+geting||getting
+globel||global
+grabing||grabbing
+grahical||graphical
+grahpical||graphical
+granularty||granularity
+grapic||graphic
+grranted||granted
+grups||groups
+guage||gauge
+guarenteed||guaranteed
+guarentee||guarantee
+halfs||halves
+hander||handler
+handfull||handful
+hanlde||handle
+hanled||handled
+happend||happened
+hardare||hardware
+harware||hardware
+hardward||hardware
+havind||having
+heigth||height
+heirarchically||hierarchically
+heirarchy||hierarchy
+heirachy||hierarchy
+helpfull||helpful
+hearbeat||heartbeat
+heterogenous||heterogeneous
+hexdecimal||hexadecimal
+hybernate||hibernate
+hiearchy||hierarchy
+hierachy||hierarchy
+hierarchie||hierarchy
+homogenous||homogeneous
+horizental||horizontal
+howver||however
+hsould||should
+hypervior||hypervisor
+hypter||hyper
+idel||idle
+identidier||identifier
+iligal||illegal
+illigal||illegal
+illgal||illegal
+iomaped||iomapped
+imblance||imbalance
+immeadiately||immediately
+immedaite||immediate
+immedate||immediate
+immediatelly||immediately
+immediatly||immediately
+immidiate||immediate
+immutible||immutable
+impelentation||implementation
+impementated||implemented
+implemantation||implementation
+implemenation||implementation
+implementaiton||implementation
+implementated||implemented
+implemention||implementation
+implementd||implemented
+implemetation||implementation
+implemntation||implementation
+implentation||implementation
+implmentation||implementation
+implmenting||implementing
+incative||inactive
+incomming||incoming
+incompaitiblity||incompatibility
+incompatabilities||incompatibilities
+incompatable||incompatible
+incompatble||incompatible
+inconsistant||inconsistent
+increas||increase
+incremeted||incremented
+incrment||increment
+incuding||including
+inculde||include
+indendation||indentation
+indended||intended
+independant||independent
+independantly||independently
+independed||independent
+indiate||indicate
+indicat||indicate
+inexpect||inexpected
+infalte||inflate
+inferface||interface
+infinit||infinite
+infomation||information
+informatiom||information
+informations||information
+informtion||information
+infromation||information
+ingore||ignore
+inheritence||inheritance
+inital||initial
+initalized||initialized
+initalised||initialized
+initalise||initialize
+initalize||initialize
+initation||initiation
+initators||initiators
+initialiazation||initialization
+initializationg||initialization
+initializiation||initialization
+initializtion||initialization
+initialze||initialize
+initialzed||initialized
+initialzing||initializing
+initilization||initialization
+initilize||initialize
+initliaze||initialize
+initilized||initialized
+inofficial||unofficial
+inrerface||interface
+insititute||institute
+instace||instance
+instal||install
+instanciate||instantiate
+instanciated||instantiated
+instuments||instruments
+insufficent||insufficient
+intead||instead
+inteface||interface
+integreated||integrated
+integrety||integrity
+integrey||integrity
+intendet||intended
+intented||intended
+interal||internal
+interanl||internal
+interchangable||interchangeable
+interferring||interfering
+interger||integer
+intergrated||integrated
+intermittant||intermittent
+internel||internal
+interoprability||interoperability
+interuupt||interrupt
+interupt||interrupt
+interupts||interrupts
+interurpt||interrupt
+interrface||interface
+interrrupt||interrupt
+interrup||interrupt
+interrups||interrupts
+interruptted||interrupted
+interupted||interrupted
+intiailized||initialized
+intial||initial
+intialisation||initialisation
+intialised||initialised
+intialise||initialise
+intialization||initialization
+intialized||initialized
+intialize||initialize
+intregral||integral
+intrerrupt||interrupt
+intrrupt||interrupt
+intterrupt||interrupt
+intuative||intuitive
+inavlid||invalid
+invaid||invalid
+invaild||invalid
+invailid||invalid
+invald||invalid
+invalde||invalid
+invalide||invalid
+invalidiate||invalidate
+invalud||invalid
+invididual||individual
+invokation||invocation
+invokations||invocations
+ireelevant||irrelevant
+irrelevent||irrelevant
+isnt||isn't
+isssue||issue
+issus||issues
+iteraions||iterations
+iternations||iterations
+itertation||iteration
+itslef||itself
+ivalid||invalid
+jave||java
+jeffies||jiffies
+jumpimng||jumping
+juse||just
+jus||just
+kown||known
+lable||label
+langage||language
+langauage||language
+langauge||language
+langugage||language
+lauch||launch
+layed||laid
+legnth||length
+leightweight||lightweight
+lengh||length
+lenght||length
+lenth||length
+lesstiff||lesstif
+libaries||libraries
+libary||library
+librairies||libraries
+libraris||libraries
+licenceing||licencing
+limted||limited
+logaritmic||logarithmic
+loggging||logging
+loggin||login
+logile||logfile
+loobpack||loopback
+loosing||losing
+losted||lost
+maangement||management
+machinary||machinery
+maibox||mailbox
+maintainance||maintenance
+maintainence||maintenance
+maintan||maintain
+makeing||making
+mailformed||malformed
+malplaced||misplaced
+malplace||misplace
+managable||manageable
+managament||management
+managment||management
+mangement||management
+manger||manager
+manoeuvering||maneuvering
+manufaucturing||manufacturing
+mappping||mapping
+maping||mapping
+matchs||matches
+mathimatical||mathematical
+mathimatic||mathematic
+mathimatics||mathematics
+maxmium||maximum
+maximium||maximum
+maxium||maximum
+mechamism||mechanism
+mechanim||mechanism
+meetign||meeting
+memeory||memory
+memmber||member
+memoery||memory
+memroy||memory
+ment||meant
+mergable||mergeable
+mesage||message
+mesages||messages
+messags||messages
+messgaes||messages
+messsage||message
+messsages||messages
+metdata||metadata
+micropone||microphone
+microprocesspr||microprocessor
+migrateable||migratable
+miliseconds||milliseconds
+millenium||millennium
+milliseonds||milliseconds
+minimim||minimum
+minium||minimum
+minimam||minimum
+minimun||minimum
+miniumum||minimum
+minumum||minimum
+misalinged||misaligned
+miscelleneous||miscellaneous
+misformed||malformed
+mispelled||misspelled
+mispelt||misspelt
+mising||missing
+mismactch||mismatch
+missign||missing
+missmanaged||mismanaged
+missmatch||mismatch
+misssing||missing
+miximum||maximum
+mmnemonic||mnemonic
+mnay||many
+modfiy||modify
+modifer||modifier
+modul||module
+modulues||modules
+momery||memory
+memomry||memory
+monitring||monitoring
+monochorome||monochrome
+monochromo||monochrome
+monocrome||monochrome
+mopdule||module
+mroe||more
+mulitplied||multiplied
+muliple||multiple
+multipler||multiplier
+multidimensionnal||multidimensional
+multipe||multiple
+multple||multiple
+mumber||number
+muticast||multicast
+mutilcast||multicast
+mutiple||multiple
+mutli||multi
+nams||names
+navagating||navigating
+nead||need
+neccecary||necessary
+neccesary||necessary
+neccessary||necessary
+necesary||necessary
+neded||needed
+negaive||negative
+negoitation||negotiation
+negotation||negotiation
+nerver||never
+nescessary||necessary
+nessessary||necessary
+none existent||non-existent
+noticable||noticeable
+notication||notification
+notications||notifications
+notifcations||notifications
+notifed||notified
+notifer||notifier
+notity||notify
+notfify||notify
+nubmer||number
+numebr||number
+numer||number
+numner||number
+nunber||number
+obtaion||obtain
+obusing||abusing
+occassionally||occasionally
+occationally||occasionally
+occurance||occurrence
+occurances||occurrences
+occurd||occurred
+occured||occurred
+occurence||occurrence
+occure||occurred
+occuring||occurring
+ocurrence||occurrence
+offser||offset
+offet||offset
+offlaod||offload
+offloded||offloaded
+offseting||offsetting
+oflload||offload
+omited||omitted
+omiting||omitting
+omitt||omit
+ommiting||omitting
+ommitted||omitted
+onself||oneself
+onthe||on the
+ony||only
+openning||opening
+operatione||operation
+opertaions||operations
+opportunies||opportunities
+optionnal||optional
+optmizations||optimizations
+orientatied||orientated
+orientied||oriented
+orignal||original
+originial||original
+orphanded||orphaned
+otherise||otherwise
+ouput||output
+oustanding||outstanding
+overaall||overall
+overhread||overhead
+overlaping||overlapping
+oveflow||overflow
+overflw||overflow
+overlfow||overflow
+overide||override
+overrided||overridden
+overriden||overridden
+overrrun||overrun
+overun||overrun
+overwritting||overwriting
+overwriten||overwritten
+pacakge||package
+pachage||package
+packacge||package
+packege||package
+packge||package
+packtes||packets
+pakage||package
+paket||packet
+pallette||palette
+paln||plan
+palne||plane
+paramameters||parameters
+paramaters||parameters
+paramater||parameter
+paramenters||parameters
+parametes||parameters
+parametised||parametrised
+paramter||parameter
+paramters||parameters
+parmaters||parameters
+particuarly||particularly
+particularily||particularly
+partion||partition
+partions||partitions
+partiton||partition
+pased||passed
+passin||passing
+pathes||paths
+pattrns||patterns
+pecularities||peculiarities
+peformance||performance
+peforming||performing
+peice||piece
+pendantic||pedantic
+peprocessor||preprocessor
+perfomance||performance
+perfoming||performing
+perfomring||performing
+periperal||peripheral
+peripherial||peripheral
+permissons||permissions
+permited||permitted
+peroid||period
+persistance||persistence
+persistant||persistent
+phoneticly||phonetically
+pipline||pipeline
+plaform||platform
+plalform||platform
+platfoem||platform
+platfomr||platform
+platfrom||platform
+plattform||platform
+pleaes||please
+ploting||plotting
+plugable||pluggable
+poinnter||pointer
+pointeur||pointer
+poiter||pointer
+posible||possible
+positon||position
+possibilites||possibilities
+postion||position
+potocol||protocol
+powerfull||powerful
+pramater||parameter
+preambule||preamble
+preamle||preamble
+preample||preamble
+preapre||prepare
+preceeded||preceded
+preceeding||preceding
+preceed||precede
+precendence||precedence
+precission||precision
+predicition||prediction
+preemptable||preemptible
+prefered||preferred
+prefferably||preferably
+prefitler||prefilter
+preform||perform
+previleged||privileged
+previlege||privilege
+premption||preemption
+prepaired||prepared
+prepate||prepare
+preperation||preparation
+preprare||prepare
+pressre||pressure
+presuambly||presumably
+previosuly||previously
+previsously||previously
+primative||primitive
+princliple||principle
+priorty||priority
+priting||printing
+privilaged||privileged
+privilage||privilege
+priviledge||privilege
+priviledged||privileged
+priviledges||privileges
+privleges||privileges
+probaly||probably
+probabalistic||probabilistic
+procceed||proceed
+proccesors||processors
+procesed||processed
+proces||process
+procesing||processing
+processessing||processing
+processess||processes
+processpr||processor
+processsed||processed
+processsing||processing
+procteted||protected
+prodecure||procedure
+progamming||programming
+progams||programs
+progess||progress
+programable||programmable
+programers||programmers
+programm||program
+programms||programs
+progres||progress
+progresss||progress
+prohibitted||prohibited
+prohibitting||prohibiting
+promiscous||promiscuous
+promps||prompts
+pronnounced||pronounced
+prononciation||pronunciation
+pronouce||pronounce
+pronunce||pronounce
+propery||property
+propigate||propagate
+propigation||propagation
+propogation||propagation
+propogate||propagate
+prosess||process
+protable||portable
+protcol||protocol
+protecion||protection
+protedcted||protected
+protocoll||protocol
+promixity||proximity
+psudo||pseudo
+psuedo||pseudo
+psychadelic||psychedelic
+purgable||purgeable
+pwoer||power
+queing||queuing
+quering||querying
+querrying||querying
+queus||queues
+randomally||randomly
+raoming||roaming
+readyness||readiness
+reasearcher||researcher
+reasearchers||researchers
+reasearch||research
+recalcualte||recalculate
+receieve||receive
+recepient||recipient
+recevied||received
+receving||receiving
+recievd||received
+recieved||received
+recieve||receive
+reciever||receiver
+recieves||receives
+recieving||receiving
+recogniced||recognised
+recognizeable||recognizable
+recompte||recompute
+recommanded||recommended
+recyle||recycle
+redect||reject
+redircet||redirect
+redirectrion||redirection
+redundacy||redundancy
+reename||rename
+refcounf||refcount
+refence||reference
+refered||referred
+referencce||reference
+referenace||reference
+refererence||reference
+refering||referring
+refernces||references
+refernnce||reference
+refrence||reference
+regiser||register
+registed||registered
+registerd||registered
+registeration||registration
+registeresd||registered
+registerred||registered
+registes||registers
+registraration||registration
+regsiter||register
+regster||register
+regualar||regular
+reguator||regulator
+regulamentations||regulations
+reigstration||registration
+releated||related
+relevent||relevant
+reloade||reload
+remoote||remote
+remore||remote
+removeable||removable
+repective||respective
+repectively||respectively
+replacable||replaceable
+replacments||replacements
+replys||replies
+reponse||response
+representaion||representation
+repsonse||response
+reqested||requested
+reqeust||request
+reqister||register
+requed||requeued
+requestied||requested
+requiere||require
+requieres||requires
+requirment||requirement
+requred||required
+requried||required
+requst||request
+requsted||requested
+reregisteration||reregistration
+reseting||resetting
+reseved||reserved
+reseverd||reserved
+resizeable||resizable
+resonable||reasonable
+resotre||restore
+resouce||resource
+resouces||resources
+resoures||resources
+responce||response
+resrouce||resource
+ressizes||resizes
+ressource||resource
+ressources||resources
+restesting||retesting
+resumbmitting||resubmitting
+retransmited||retransmitted
+retreived||retrieved
+retreive||retrieve
+retreiving||retrieving
+retrive||retrieve
+retrived||retrieved
+retrun||return
+retun||return
+retuned||returned
+reudce||reduce
+reuest||request
+reuqest||request
+reutnred||returned
+revsion||revision
+rewritting||rewriting
+rmeoved||removed
+rmeove||remove
+rmeoves||removes
+rountine||routine
+routins||routines
+rquest||request
+runing||running
+runned||ran
+runnnig||running
+runnning||running
+runtine||runtime
+sacrifying||sacrificing
+safly||safely
+safty||safety
+satify||satisfy
+satisifed||satisfied
+savable||saveable
+scaleing||scaling
+scaned||scanned
+scaning||scanning
+scarch||search
+schdule||schedule
+seach||search
+searchs||searches
+secion||section
+secquence||sequence
+secund||second
+segement||segment
+seleted||selected
+semaphone||semaphore
+senario||scenario
+senarios||scenarios
+sentivite||sensitive
+separatly||separately
+sepcify||specify
+seperated||separated
+seperately||separately
+seperate||separate
+seperatly||separately
+seperator||separator
+sepperate||separate
+seqeunce||sequence
+seqeuncer||sequencer
+seqeuencer||sequencer
+sequece||sequence
+sequemce||sequence
+sequencial||sequential
+serivce||service
+serveral||several
+servive||service
+sesion||session
+setts||sets
+settting||setting
+shapshot||snapshot
+shoft||shift
+shotdown||shutdown
+shoud||should
+shouldnt||shouldn't
+shoule||should
+shrinked||shrunk
+siginificantly||significantly
+signabl||signal
+significanly||significantly
+similary||similarly
+similiar||similar
+simlar||similar
+simliar||similar
+simpified||simplified
+simultaneusly||simultaneously
+simultanous||simultaneous
+singaled||signaled
+singal||signal
+singed||signed
+slect||select
+sleeped||slept
+sliped||slipped
+softwade||software
+softwares||software
+soley||solely
+soluation||solution
+souce||source
+speach||speech
+specfic||specific
+specfication||specification
+specfield||specified
+speciefied||specified
+specifc||specific
+specifed||specified
+specificatin||specification
+specificaton||specification
+specificed||specified
+specifing||specifying
+specifiy||specify
+specifiying||specifying
+speficied||specified
+speicify||specify
+speling||spelling
+spinlcok||spinlock
+spinock||spinlock
+splitted||split
+spreaded||spread
+spurrious||spurious
+sructure||structure
+stablilization||stabilization
+staically||statically
+staion||station
+standardss||standards
+standartization||standardization
+standart||standard
+standy||standby
+stardard||standard
+staticly||statically
+statisitcs||statistics
+statuss||status
+stoped||stopped
+stoping||stopping
+stoppped||stopped
+straming||streaming
+struc||struct
+structres||structures
+stuct||struct
+strucuture||structure
+stucture||structure
+sturcture||structure
+subdirectoires||subdirectories
+suble||subtle
+substract||subtract
+submited||submitted
+submition||submission
+succeded||succeeded
+suceed||succeed
+succesfuly||successfully
+succesfully||successfully
+succesful||successful
+successed||succeeded
+successfull||successful
+successfuly||successfully
+sucessfully||successfully
+sucessful||successful
+sucess||success
+superflous||superfluous
+superseeded||superseded
+suplied||supplied
+suported||supported
+suport||support
+supportet||supported
+suppored||supported
+supporing||supporting
+supportin||supporting
+suppoted||supported
+suppported||supported
+suppport||support
+supprot||support
+supress||suppress
+surpressed||suppressed
+surpresses||suppresses
+susbsystem||subsystem
+suspeneded||suspended
+suspsend||suspend
+suspicously||suspiciously
+swaping||swapping
+switchs||switches
+swith||switch
+swithable||switchable
+swithc||switch
+swithced||switched
+swithcing||switching
+swithed||switched
+swithing||switching
+swtich||switch
+syfs||sysfs
+symetric||symmetric
+synax||syntax
+synchonized||synchronized
+sychronization||synchronization
+sychronously||synchronously
+synchronuously||synchronously
+syncronize||synchronize
+syncronized||synchronized
+syncronizing||synchronizing
+syncronus||synchronous
+syste||system
+sytem||system
+sythesis||synthesis
+tagert||target
+taht||that
+tained||tainted
+tarffic||traffic
+tansmit||transmit
+targetted||targeted
+targetting||targeting
+taskelt||tasklet
+teh||the
+temeprature||temperature
+temorary||temporary
+temproarily||temporarily
+temperture||temperature
+theads||threads
+therfore||therefore
+thier||their
+threds||threads
+threee||three
+threshhold||threshold
+thresold||threshold
+throtting||throttling
+throught||through
+tansition||transition
+trackling||tracking
+troughput||throughput
+trys||tries
+thses||these
+tiggers||triggers
+tiggered||triggered
+tiggerring||triggering
+tipically||typically
+timeing||timing
+timming||timing
+timout||timeout
+tmis||this
+tolarance||tolerance
+toogle||toggle
+torerable||tolerable
+torlence||tolerance
+traget||target
+traking||tracking
+tramsmitted||transmitted
+tramsmit||transmit
+tranasction||transaction
+tranceiver||transceiver
+tranfer||transfer
+tranmission||transmission
+tranport||transport
+transcevier||transceiver
+transciever||transceiver
+transferd||transferred
+transfered||transferred
+transfering||transferring
+transision||transition
+transistioned||transitioned
+transmittd||transmitted
+transormed||transformed
+trasaction||transaction
+trasfer||transfer
+trasmission||transmission
+trasmitter||transmitter
+treshold||threshold
+trigged||triggered
+triggerd||triggered
+trigerred||triggered
+trigerring||triggering
+trun||turn
+tunning||tuning
+ture||true
+tyep||type
+udpate||update
+updtes||updates
+uesd||used
+unknwon||unknown
+uknown||unknown
+usccess||success
+uncommited||uncommitted
+uncompatible||incompatible
+uncomressed||uncompressed
+unconditionaly||unconditionally
+undeflow||underflow
+undelying||underlying
+underun||underrun
+unecessary||unnecessary
+unexecpted||unexpected
+unexepected||unexpected
+unexpcted||unexpected
+unexpectd||unexpected
+unexpeted||unexpected
+unexpexted||unexpected
+unfortunatelly||unfortunately
+unifiy||unify
+uniterrupted||uninterrupted
+uninterruptable||uninterruptible
+unintialized||uninitialized
+unitialized||uninitialized
+unkmown||unknown
+unknonw||unknown
+unknouwn||unknown
+unknow||unknown
+unkown||unknown
+unamed||unnamed
+uneeded||unneeded
+unneded||unneeded
+unneccecary||unnecessary
+unneccesary||unnecessary
+unneccessary||unnecessary
+unnecesary||unnecessary
+unneedingly||unnecessarily
+unnsupported||unsupported
+unuspported||unsupported
+unmached||unmatched
+unprecise||imprecise
+unpriviledged||unprivileged
+unpriviliged||unprivileged
+unregester||unregister
+unresgister||unregister
+unrgesiter||unregister
+unsinged||unsigned
+unstabel||unstable
+unsolicted||unsolicited
+unsolicitied||unsolicited
+unsuccessfull||unsuccessful
+unsuported||unsupported
+untill||until
+ununsed||unused
+unuseful||useless
+unvalid||invalid
+upate||update
+upsupported||unsupported
+upto||up to
+useable||usable
+usefule||useful
+usefull||useful
+usege||usage
+usera||users
+usualy||usually
+usupported||unsupported
+utilites||utilities
+utillities||utilities
+utilties||utilities
+utiltity||utility
+utitity||utility
+utitlty||utility
+vaid||valid
+vaild||valid
+validationg||validating
+valide||valid
+variantions||variations
+varible||variable
+varient||variant
+vaule||value
+verbse||verbose
+veify||verify
+verfication||verification
+veriosn||version
+versoin||version
+verisons||versions
+verison||version
+veritical||vertical
+verson||version
+vicefersa||vice-versa
+virtal||virtual
+virtaul||virtual
+virtiual||virtual
+visiters||visitors
+vitual||virtual
+vunerable||vulnerable
+wakeus||wakeups
+was't||wasn't
+wathdog||watchdog
+wating||waiting
+wiat||wait
+wether||whether
+whataver||whatever
+whcih||which
+whenver||whenever
+wheter||whether
+whe||when
+wierd||weird
+wihout||without
+wiil||will
+wirte||write
+withing||within
+wnat||want
+wont||won't
+workarould||workaround
+writeing||writing
+writting||writing
+wtih||with
+zombe||zombie
+zomebie||zombie
--
2.53.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [kvm-unit-tests PATCH 3/3] scripts/get_maintainer.pl: Ignore --nogit-chief-penguins option
2026-04-23 14:16 [kvm-unit-tests PATCH 0/3] scripts: Configure and update tools to allow basic b4 usage Christoph Schlameuss
2026-04-23 14:16 ` [kvm-unit-tests PATCH 1/3] scripts/checkpatch.pl: Add empty codespell and const_structs configs Christoph Schlameuss
2026-04-23 14:16 ` [kvm-unit-tests PATCH 2/3] scripts/checkpatch.pl: Copy kernel codespell file Christoph Schlameuss
@ 2026-04-23 14:16 ` Christoph Schlameuss
2026-04-24 7:31 ` Thomas Huth
2026-04-24 7:19 ` [kvm-unit-tests PATCH 0/3] scripts: Configure and update tools to allow basic b4 usage Thomas Huth
3 siblings, 1 reply; 10+ messages in thread
From: Christoph Schlameuss @ 2026-04-23 14:16 UTC (permalink / raw)
To: kvm
Cc: Janosch Frank, Paolo Bonzini, Thomas Huth, Andrew Jones,
Nicholas Piggin, Christoph Schlameuss
b4 by default calls the get_maintainer.pl script with the
'--nogit-chief-penguins' option letting the 'b4 prep --auto-to-cc' fail.
Adding parsing of that option without any related functionality to allow
simple usage of b4 in the development workflow.
Signed-off-by: Christoph Schlameuss <schlameuss@linux.ibm.com>
---
Sorry, I am not too versed with perl, so if there are better ways to fix
this please comment.
---
scripts/get_maintainer.pl | 2 ++
1 file changed, 2 insertions(+)
diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 117b69145f0e..9b99f6218467 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -31,6 +31,7 @@ my $email_fixes = 1;
my $email_list = 1;
my $email_moderated_list = 1;
my $email_subscriber_list = 0;
+my $email_git_chief_penguins = 0;
my $email_git = 0;
my $email_git_all_signature_types = 0;
my $email_git_blame = 0;
@@ -228,6 +229,7 @@ if (!GetOptions(
'git-blame!' => \$email_git_blame,
'git-blame-signatures!' => \$email_git_blame_signatures,
'git-fallback!' => \$email_git_fallback,
+ 'git-chief-penguins!' => \$email_git_chief_penguins,
'git-min-signatures=i' => \$email_git_min_signatures,
'git-max-maintainers=i' => \$email_git_max_maintainers,
'git-min-percent=i' => \$email_git_min_percent,
--
2.53.0
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [kvm-unit-tests PATCH 3/3] scripts/get_maintainer.pl: Ignore --nogit-chief-penguins option
2026-04-23 14:16 ` [kvm-unit-tests PATCH 3/3] scripts/get_maintainer.pl: Ignore --nogit-chief-penguins option Christoph Schlameuss
@ 2026-04-24 7:31 ` Thomas Huth
2026-04-24 7:41 ` Christoph Schlameuss
0 siblings, 1 reply; 10+ messages in thread
From: Thomas Huth @ 2026-04-24 7:31 UTC (permalink / raw)
To: Christoph Schlameuss, kvm
Cc: Janosch Frank, Paolo Bonzini, Andrew Jones, Nicholas Piggin
On 23/04/2026 16.16, Christoph Schlameuss wrote:
> b4 by default calls the get_maintainer.pl script with the
> '--nogit-chief-penguins' option letting the 'b4 prep --auto-to-cc' fail.
>
> Adding parsing of that option without any related functionality to allow
> simple usage of b4 in the development workflow.
>
> Signed-off-by: Christoph Schlameuss <schlameuss@linux.ibm.com>
>
> ---
>
> Sorry, I am not too versed with perl, so if there are better ways to fix
> this please comment.
> ---
> scripts/get_maintainer.pl | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
> index 117b69145f0e..9b99f6218467 100755
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -31,6 +31,7 @@ my $email_fixes = 1;
> my $email_list = 1;
> my $email_moderated_list = 1;
> my $email_subscriber_list = 0;
> +my $email_git_chief_penguins = 0;
The variable is called email_git_penguin_chiefs in the kernel version of
this script ... Could you please use that here, too, so we stay at least a
little bit in sync with the kernel script?
Thanks,
Thomas
> my $email_git = 0;
> my $email_git_all_signature_types = 0;
> my $email_git_blame = 0;
> @@ -228,6 +229,7 @@ if (!GetOptions(
> 'git-blame!' => \$email_git_blame,
> 'git-blame-signatures!' => \$email_git_blame_signatures,
> 'git-fallback!' => \$email_git_fallback,
> + 'git-chief-penguins!' => \$email_git_chief_penguins,
> 'git-min-signatures=i' => \$email_git_min_signatures,
> 'git-max-maintainers=i' => \$email_git_max_maintainers,
> 'git-min-percent=i' => \$email_git_min_percent,
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kvm-unit-tests PATCH 3/3] scripts/get_maintainer.pl: Ignore --nogit-chief-penguins option
2026-04-24 7:31 ` Thomas Huth
@ 2026-04-24 7:41 ` Christoph Schlameuss
0 siblings, 0 replies; 10+ messages in thread
From: Christoph Schlameuss @ 2026-04-24 7:41 UTC (permalink / raw)
To: Thomas Huth, Christoph Schlameuss, kvm
Cc: Janosch Frank, Paolo Bonzini, Andrew Jones, Nicholas Piggin
On Fri Apr 24, 2026 at 9:31 AM CEST, Thomas Huth wrote:
> On 23/04/2026 16.16, Christoph Schlameuss wrote:
>> b4 by default calls the get_maintainer.pl script with the
>> '--nogit-chief-penguins' option letting the 'b4 prep --auto-to-cc' fail.
>>
>> Adding parsing of that option without any related functionality to allow
>> simple usage of b4 in the development workflow.
>>
>> Signed-off-by: Christoph Schlameuss <schlameuss@linux.ibm.com>
>>
>> ---
>>
>> Sorry, I am not too versed with perl, so if there are better ways to fix
>> this please comment.
>> ---
>> scripts/get_maintainer.pl | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
>> index 117b69145f0e..9b99f6218467 100755
>> --- a/scripts/get_maintainer.pl
>> +++ b/scripts/get_maintainer.pl
>> @@ -31,6 +31,7 @@ my $email_fixes = 1;
>> my $email_list = 1;
>> my $email_moderated_list = 1;
>> my $email_subscriber_list = 0;
>> +my $email_git_chief_penguins = 0;
>
> The variable is called email_git_penguin_chiefs in the kernel version of
> this script ... Could you please use that here, too, so we stay at least a
> little bit in sync with the kernel script?
>
> Thanks,
> Thomas
>
>
Oh wow, good eye. That was my intend to have it the same as in the kernel
version. But obviously I can mess that up. Will fix. Thx.
>> my $email_git = 0;
>> my $email_git_all_signature_types = 0;
>> my $email_git_blame = 0;
>> @@ -228,6 +229,7 @@ if (!GetOptions(
>> 'git-blame!' => \$email_git_blame,
>> 'git-blame-signatures!' => \$email_git_blame_signatures,
>> 'git-fallback!' => \$email_git_fallback,
>> + 'git-chief-penguins!' => \$email_git_chief_penguins,
>> 'git-min-signatures=i' => \$email_git_min_signatures,
>> 'git-max-maintainers=i' => \$email_git_max_maintainers,
>> 'git-min-percent=i' => \$email_git_min_percent,
>>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kvm-unit-tests PATCH 0/3] scripts: Configure and update tools to allow basic b4 usage
2026-04-23 14:16 [kvm-unit-tests PATCH 0/3] scripts: Configure and update tools to allow basic b4 usage Christoph Schlameuss
` (2 preceding siblings ...)
2026-04-23 14:16 ` [kvm-unit-tests PATCH 3/3] scripts/get_maintainer.pl: Ignore --nogit-chief-penguins option Christoph Schlameuss
@ 2026-04-24 7:19 ` Thomas Huth
2026-04-24 7:42 ` Christoph Schlameuss
3 siblings, 1 reply; 10+ messages in thread
From: Thomas Huth @ 2026-04-24 7:19 UTC (permalink / raw)
To: Christoph Schlameuss, kvm
Cc: Janosch Frank, Paolo Bonzini, Andrew Jones, Nicholas Piggin
On 23/04/2026 16.16, Christoph Schlameuss wrote:
> Provide some basic configuration for checkpatch.pl so that use on the
> command line is less confusing and actually useful.
> Checking new patches with i.e. 'b4 prep --check' will now not just give you
> dummy errors.
>
> Modify get_maintainer.pl to work when called from 'b4 prep --auto-to-cc'.
>
> ---
> The second patch also copies the current codespell config from the kernel.
> I think this would be nice to have but I would also understand if nobody
> wants to add that dead weight here. In that case I would also agree drop
> that second patch.
IMHO the dictionary that comes along with the codespell utility should
already be good enough to cover most of these typos, so I'm inclined to
rather say "no" to the second patch. Instead, maybe you could introduce a
k-u-t specific spelling.txt to get rid of the warning message, with some few
entries like:
kmv||kvm
RDSMR||RDMSR
WRSMR||WRMSR
?
(the latter two entries were fixed by commit 784dd85d25d484)
WDYT?
Thomas
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [kvm-unit-tests PATCH 0/3] scripts: Configure and update tools to allow basic b4 usage
2026-04-24 7:19 ` [kvm-unit-tests PATCH 0/3] scripts: Configure and update tools to allow basic b4 usage Thomas Huth
@ 2026-04-24 7:42 ` Christoph Schlameuss
0 siblings, 0 replies; 10+ messages in thread
From: Christoph Schlameuss @ 2026-04-24 7:42 UTC (permalink / raw)
To: Thomas Huth, Christoph Schlameuss, kvm
Cc: Janosch Frank, Paolo Bonzini, Andrew Jones, Nicholas Piggin
On Fri Apr 24, 2026 at 9:19 AM CEST, Thomas Huth wrote:
> On 23/04/2026 16.16, Christoph Schlameuss wrote:
>> Provide some basic configuration for checkpatch.pl so that use on the
>> command line is less confusing and actually useful.
>> Checking new patches with i.e. 'b4 prep --check' will now not just give you
>> dummy errors.
>>
>> Modify get_maintainer.pl to work when called from 'b4 prep --auto-to-cc'.
>>
>> ---
>> The second patch also copies the current codespell config from the kernel.
>> I think this would be nice to have but I would also understand if nobody
>> wants to add that dead weight here. In that case I would also agree drop
>> that second patch.
>
> IMHO the dictionary that comes along with the codespell utility should
> already be good enough to cover most of these typos, so I'm inclined to
> rather say "no" to the second patch. Instead, maybe you could introduce a
> k-u-t specific spelling.txt to get rid of the warning message, with some few
> entries like:
>
> kmv||kvm
> RDSMR||RDMSR
> WRSMR||WRMSR
>
> ?
>
> (the latter two entries were fixed by commit 784dd85d25d484)
>
> WDYT?
>
> Thomas
Fine with that. I will have a look around if there are some other things like
that we can add.
^ permalink raw reply [flat|nested] 10+ messages in thread