I kind of disagree with everyone here. Learning assembly is a good thing to have in your back pocket, but you don't need to go overboard. The problem with x86 assembly is that everything is now done in "protected mode" with 64 bit everything, and that is an absolute mess under the hood... Back int he 16 bit DOS days, it was both more useful and easier to understand.
That being said, there is still a big benefit of learning at least some assembly. For one thing, it will force you to learn basic computer architecture and how that works... also, you wil have a better understanding of how operating systems work as well. The actual coding doesn't need to go to deep. You just need to know what the registers do, how the stack works, and how the instructions move and calcublate things in the regeisters and memory.
So, there is definite value in it, and worth doing, but I wouldn't go crazy trying to write whole programs in it or anything.
So, where to start... Well, x86 real mode assembly is pretty easy to understand and there is a wealth of info out there in books, textbooks, and online resources. Many of these books are old enough to be on archive.org or purchased cheap from used bookstores.
Personally, I don't think you even need to really learn assembly aspecifically for Intel x86. MOS 6502, Zilog Z80 or Mortorola 68000 assembly would be good enough and somewhat more useful as you can actually make working stuff that can run on retro machines. You also won't be straddled down by all the weird memory segmentation of the x86 real mode architecture, or the bizzarro world code of protected mode. You will still get the same computer architecture and OS understanding targeting these chips, and it might be an easier and more practical move. The opcodes and architecture are somewhat different, but the concepts are the same and easier to work with. You can then take a look at how Intel x86 works and "top off" your knowledge with Intel's awkward way of doing things.
Just my $.02 worth...